Domande con tag 'code-reviews'

5
risposte

Controlla se la variabile non è nulla prima di impostare su null

A un nostro team piace scrivere codice come questo (C #): if (_someVar != null) { _someVar = null; } Piuttosto che semplicemente: _someVar = null; Non ho mai usato il primo dato che ho sempre pensato che non fosse necessario, spe...
posta 04.08.2017 - 16:45
4
risposte

È accettabile assegnare una variabile di classe all'interno di un metodo?

Ho appena fatto una revisione del codice in cui mi sono imbattuto in un metodo del genere: private void FindMinAndMax(double[] values) { // Some algorithm that results in two class vars being assigned: _min = ... _max = ... } (D...
posta 07.06.2017 - 17:06
5
risposte

Rivedi prima i test

Ho intenzione di adottare TDD nel nostro team e una delle idee che ho è di rivedere i test prima. Quindi si dovrebbero prima scrivere interfacce, mock e test, inviarli per una revisione del codice e una volta approvate interfacce e test (specifi...
posta 14.12.2012 - 03:09
1
risposta

Word per bandiere rosse / segnali di pericolo?

Durante la lettura delle migliori pratiche e recensioni di codice, ho trovato una parola / frase che le persone nei forum hanno usato per descrivere il codice che dovrebbe essere un segnale di avvertimento che il programmatore potrebbe aver pers...
posta 16.01.2012 - 01:11
3
risposte

Come faccio a indirizzare efficacemente il feedback di revisione del codice sui nomi di classi e metodi senza un suggerimento per un nome migliore o alternativo?

Parte del feedback sulla revisione del codice sta suggerendo nomi migliori per metodi e classi. Come posso indirizzare il feedback sui nomi di classi e metodi che non suggeriscono un'alternativa? Ad esempio, potrei creare la classe public c...
posta 10.02.2014 - 21:07
4
risposte

Come gestisci la scoperta del codice errato e falso nel tuo team? [duplicare]

Ogni anno a gennaio elaboriamo un grosso compito con il nostro sistema. Mentre le prestazioni durante l'attività erano superiori alla media, il follow-up di manutenzione ha attualmente molti problemi con i lavori eseguiti troppo a lungo e fuor...
posta 07.02.2013 - 10:35
2
risposte

L'analisi statica dovrebbe essere integrata con la revisione del codice? [chiuso]

Voglio integrare vari strumenti di analisi statica e quindi aggiungere i risultati come commenti su un file all'interno degli strumenti di revisione del codice, come la scheda Stash o Review. Sto esplorando la possibilità di scrivere un tale...
posta 05.05.2015 - 06:23
5
risposte

Qual è il tuo processo di revisione e approvazione del codice e quali sono i suoi vantaggi / svantaggi? [chiuso]

Il tuo team si basa sulle revisioni del codice per l'approvazione del lavoro? È semplicemente il QA che firma le tue funzionalità e le correzioni? Avete un programmatore capo (o architetto) che rivede la progettazione e l'implementazione del cod...
posta 06.01.2011 - 23:48
4
risposte

Codice pulito riutilizzabile VS: dov'è il bilanciamento?

Diciamo che ho un modello di dati per i post di un blog e ho due casi d'uso di quel modello - ottenendo tutti i post di blog e ricevendo solo post di blog che sono stati scritti da un autore specifico. Ci sono fondamentalmente due modi in cui...
posta 11.10.2012 - 10:06
1
risposta

Come eseguire la revisione del codice del codice visivo?

Ultimamente, ho scritto codice in SAS Enterprise Guide , che produce "codice" che assomiglia a questo (screenshot dalla pagina collegata): Lamiadomandaèduplice:Pensichesianecessarioeseguirelarevisionedelcodicesuquestotipodicodice?Ilmioistint...
posta 18.04.2012 - 16:26