21
¿Por qué es malo verificar las líneas con espacios en blanco al control de origen? ¿Qué tipo de problemas podría causar eso?¿Por qué es malo comprometer líneas con espacios en blanco al control de fuente?
A
Respuesta
22
Falsas diferencias, básicamente. Es útil si los diffs solo muestran cambios "reales". Algunos programas de diferencias ignorarán el espacio en blanco, pero sería mejor evitar el cambio ficticio en primer lugar.
Por supuesto, tampoco ayuda si hace que la línea se ajuste a la máquina de un colega.
+3
También es una razón para no registrarse código comentado. –
+1
¡Sí! Odio eso :) ¡Elimina el código si ya está en control de fuente! –
+7
Pero ... si su repositorio ya TIENE espacios en blanco, y usted, tratando de ser útil, lo saca, entonces USTED es el que está creando una diferencia falsa. ;) – Pistos
2
Como muchas personas los eliminan, aparecerán como líneas modificadas en herramientas diff si no usa todas las opciones (por ejemplo, un cvs diff antiguo) lo que significa que la gente ve su línea sin ningún motivo justificado.
En teoría, también podría tener cadenas que envuelven líneas donde el espacio en blanco sería realmente malo, pero ... probablemente no sea su problema.
1
Sí, estoy de acuerdo con las otras publicaciones, pero me gustaría añadir que no es malo per se. No es una gran práctica, pero ese es el tipo de cosas que suceden y simplemente suspiras y sigues con las cosas.
+0
La mala práctica es una mala práctica. No es lo más importante, pero si se puede arreglar fácilmente antes de comprometerse, ¿por qué no? –
3
Es como pintar las paredes, pero no terminar los bordes correctamente, y pasar directamente al zócalo.
0
Las utilidades modernas de diff no se cuelgan en el espacio en blanco.
2
Algunos editores eliminan automáticamente los espacios en blanco al final, otros no. Esto crea ruido dif y puede causar conflictos de fusión.
+0
Debería coordinar con las personas con las que está trabajando (colegas, público de código abierto) qué estrategia utilizan todos y asegurarse de que todos utilicen la misma estrategia. –
Cuestiones relacionadas
- 1. Xcode 4.5 - espacios en blanco al final
- 2. ¿Por qué es malo?
- 3. división por espacios en blanco o múltiples espacios en blanco
- 4. ¿Por qué Vim agrega espacios al unir líneas?
- 5. Mostrar espacios en blanco en emacs sólo en líneas
- 6. ¿Por qué es JFormattedTextField malo?
- 7. ¿Por qué es "import *" malo?
- 8. Al usar Sesiones es malo, y ¿qué tiene de malo?
- 9. Extracción espacios en blanco finales sólo para líneas editadas
- 10. MSTest Projects en Visual Studio: ¿qué archivos comprometer al control de versiones?
- 11. Ignorar espacios en blanco con PEG.js
- 12. Hacer {e} ignorar líneas que contienen solo espacios en blanco
- 13. ¿Cómo identificar y eliminar líneas nuevas y espacios en blanco?
- 14. ¿Por qué CoffeeScript requiere espacios en blanco después del mapa?
- 15. ¿Qué es "matemática de cuerdas" y por qué es malo?
- 16. Eliminar todos los espacios en blanco y las líneas vacías
- 17. ¿Por qué es REGISTER_GLOBALS tan malo?
- 18. ¿Por qué GLUT es tan malo?
- 19. Fusión sin conflictos de espacios en blanco
- 20. ¿Por qué los valores de cookie con espacios en blanco llegan al lado del cliente con comillas?
- 21. ¿Por qué el bloqueo (esto) {...} es malo?
- 22. HTML/CSS espacios en blanco romper
- 23. Bash: buscar archivos con espacios al final de las líneas
- 24. Espacios en blanco en Java
- 25. prevención de espacios en blanco al usar las abreviaturas vim
- 26. Ack buscar con espacios en blanco
- 27. Solr consulta comodín con espacios en blanco
- 28. ¿Por qué es malo usar locales de subprocesos en Django?
- 29. ¿Cómo eliminar elementos con espacios en blanco?
- 30. Espacios en blanco en CLASSPATH
¿quiere decir "check-in" como en control de código fuente? –
Sí, parece que alguien ya ha editado el título para aclararlo. :) También agregué la edición al cuerpo. Gracias – Readonly