Sebbene io sia uno di quelli che spesso sottolinea che i nomi con trattini bassi di sottolineatura possono essere riservati, raccomando vivamente di non farlo a meno che non si verifichino problemi causati dai nomi. Fare questo cambiamento globale renderà il tuo sistema di controllo della versione meno utile di quanto potrebbe altrimenti causando ogni tipo di diff spuri. Inoltre, c'è la possibilità di creare nomi duplicati.
Inoltre, ci sono molti nomi prefissi di sottolineatura che sono perfettamente validi. Si pensa immediatamente a __LINE__
e __FILE__
ecc. E a tutti i nomi di funzioni non standard che possono essere fornite da l'implementazione specifica. Filtrare quei nomi sarà tutt'altro che banale (direi quasi impossibile), certamente un semplice script Perl o sed non sarà sufficiente.
Quindi, invece, ti incoraggio a cambiare i nomi su file per file mentre apporti altre modifiche al codice. Cambia i tuoi standard di programmazione per evitare i principali nomi di sottolineatura e scrivi un nuovo codice in conformità con gli standard.
fonte
2009-07-22 11:37:53
Sono davvero d'accordo. Dove il trattino basso non è così importante. –
Se il rilevamento del problema, tuttavia, solleva un errore (a bassa priorità) sulla denominazione e quando si aggiorna un file si effettuano due check-in: uno per correggere i nomi e uno per apportare altre modifiche. I check-in come "fatto un cambio di linea, e anche modificato 200 righe per correggere i nomi" hanno diffs davvero inutili. –
Sono d'accordo anch'io. Non cercare di aggiustare qualcosa che non sia rotto. So che il codice a volte può sembrare terribile. I miei progetti sono pieni di cose del genere. Ma ti ci abituerai. – TalkingCode