modificato di nuovo dall'autore: no, non si tratta di 2 domande. Questa è una domanda sulle domande di revisione del codice che contengono due punti separati. Per favore non modificare la mia domanda.
Per le variabili di denominazione, i due lati riguardano l'uso degli articoli (a, an, the) nel nome della variabile (ad esempio, il libro vs. il libro - oppure - parseTheString vs. parseString).
Per le stringhe hard-coding, i tre lati sono se mettere tutte le stringhe in costanti (finale statico pubblico) in un file separato (classe o interfaccia), metterle tutte nello stesso file (finale statico privato) o don usare le costanti per le stringhe. (Ad esempio, nuovo pulsante ("Click Me") rispetto al nuovo pulsante (BUTTON_LABEL_CLICK)). Discutiamo anche un po 'il problema della notazione ungherese, ma ho visto quello che ho discusso qui in altre domande.
Per molto tempo è stato lo standard defacto di non usare articoli nei nomi delle variabili (aggiunge disordine visivo senza aggiungere valore) e di non avere stringhe codificate sparse attraverso il codice sorgente (un potenziale incubo di manutenzione e molto scarso per l'internazionalizzazione). Ora stiamo iniziando a vedere le violazioni di questi e gli sviluppatori sembrano davvero sorpresi quando qualcuno li contrassegna come difetti o almeno che richiedono attenzione. Il manager ha detto che non si preoccupa in un modo o nell'altro e questi punti non sono negli standard di codifica della nostra azienda.