Clarify that marking certain member variables const can hinder performance
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Documentación
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- cpp
- Área
- documentation
Línea de trabajo
Comienza localizando el documento 03-Style y revisando la orientación citada sobre marcar como const las variables miembro que no cambian. Aclara el compromiso documentado entre la corrección de const y mover contenedores, incluida la opción del getter para el campo private descrita en el issue, y luego verifica el documento renderizado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
In the 03-Style document, I read the following sentence:
If the member variable is not expected to change after the initialization, then mark it const.
However, we know that if, for example, we mark a container as const, we prevent it from being moved, possibly hindering performance. Marking fields as const is useful to enforce invariants; however, it "interferes" with performance. If both const correctness and performance are important, one could leave the field non-const, make it private and define a field getter.
- Lenguaje dominante
- Sin datos de lenguaje
- Estrellas
- 8.9k
- Forks
- 903
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de cpp-best-practices/cppbestpractices
-
Broken Link Chapter 2: Use_the_Tools_AvailableQuizá libre de nuevo @vishesharma20 la tomó hace 172 días y no hay ningún pull request abierto. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 64/100
cpp-best-practices/cppbestpractices#169 · 1 comentario ·
-
Broken Link - 11 Further ReadingPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 55/100
cpp-best-practices/cppbestpractices#168 · 1 comentario ·
-
C++ PracticeAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 10/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 35/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
cpp-best-practices/cppbestpractices#160 · 1 comentario ·
Todos los issues de cpp-best-practices/cppbestpractices
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
-
[Sync EN] Document the timezone of DateTime(Immutable)::createFromTimestamp() (#6074)Posiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertosync-en
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día
-
website
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
damodharg6/payledger#7 ·
-
area/mobile good first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100