Dovrei usare ReSharper per riordinare il codice di altri popoli?

Question

Io uso ReSharper al lavoro. Alcuni dei miei colleghi non fanno.

Quando apro un codice che è stato scritto qualcuno che non lo fa, si è immediatamente evidente dalla quantità di arancio sul mio schermo.

Quello che non sono sicuro di è fino a che punto dovrei sentirmi libero di riordinare i pasticci l'hanno inconsapevolmente lasciato. Con la maggior parte di quello che sto guardando, è sciatto ma innocuo, e non sarebbe davvero saltare fuori di me se non avessi mai usato ReSharper.

Credo che vedo le mie opzioni in linea di massima come

1) La storia delle modifiche al codice sorgente è essenziale per la manutenzione. Cambiare il meno possibile, o il prossimo ragazzo solito hanno una speranza di capire cosa è cambiato. Chi se ne frega del codice irraggiungibile, uso non necessario di .ToString () ecc in ogni caso.

2) Modificare la roba senza senso come comprende, correggere commenti di documentazione di metodo e cose del genere. Il ragazzo che ha scritto che gli piace il suo codice a guardare come questo, in modo da lasciarlo in uno stato in cui lui abituato lamentano, ma sbarazzarsi di alcuni dei arancio inutili

3) Orange è solo rosso ma più leggero. F12 poi Alt + Invio fino verde.

4) Dimenticate l'arancia, guarda che la funzione linea monster 700. Che cosa è questo 1997? È ora di darsi da fare ... e se avete il tempo, introdurre il suo collega al nostro buon amico e mentore il signor Fowler.

Io tendo a volteggiare tra le opzioni a seconda di quanto tempo ho avuto, fino a che punto io sono ora responsabile per il codice, e quanto sia complesso il codice sguardi (che può farmi andare per 1 o 4 in genere).

Sembra che una delle 4 opzioni dovrebbe essere quello che sto lottando per se, ma non ho idea di quale

Answer 1

"Leave the campsite cleaner than you found it."

E 'il principio boyscout. Se si tratta di "loro" codice e mantengono poi l'introduzione di piccoli di pulizia-cambiamenti non li dovrebbe offendere, ma andando troppo lontano potrebbe sembrare maleducato o si potrebbe essere presa in modo efficace la proprietà del codice.

Answer 2

La tua squadra dovrebbe accordarsi su uno standard. Se qualcun altro usa un altro strumento, si può trovare in una modifica di guerra non intenzionale.

Ma se è possibile tutti d'accordo, allora sì. Ripulire il codice, come si va.

Answer 3

faccio un "riformattare" il check-in prima di cambiare qualsiasi della logica attuale, in questo modo è possibile vedere cosa è cambiato.

Answer 4

refactoring Inutile è solo che - inutile. E 'ingombra i registri storia repository e si può introdurre bug.

Se si suppone che la roba "senza senso" (documentazione, commenti, ecc) per essere formattato in un certo modo, e non è fino ai vostri standard di sviluppo, quindi vorrei fare tutto in una volta nel minor numero di archiviazioni possibili.

Quando si sta effettivamente lavorando sui pezzi di codice e avete la possibilità di testare le modifiche, fare il refactoring poi. ReSharper sarà sempre a disposizione per mostrare la strada in quel punto.

Answer 5

Se gli strumenti di sviluppo nel vostro team non sono la stessa cosa alla fine ci sarà ancora più problemi. Seguire la "riformattare" check-in suggerito sopra e standardizzare il set di strumenti con i tuoi colleghi, o cadere ReSharper o dare a tutti la bacchetta magica formattazione.

Answer 6

Ogni volta che si cambia nulla, a prescindere dalla volontà, si rischia di rompere qualcosa involontariamente. Su un lato personale, non cambierei il codice di un'altra persona senza parlare con loro per primi.

Answer 7

Mi consiglia di cambiare le cose solo quando è necessario. Non v'è, naturalmente, qualche definizione che varia su ciò che "bisogno" si intende. Ad esempio, se si scrive un metodo che chiama un altro metodo, e questo metodo si sta chiamando succede ad avere un po 'di duplicazione del codice? Mi piacerebbe che refactoring, e già che ci sono lo rimuovo dichiarazioni using in eccesso, ecc mi piacerebbe provare a evitare proprio una massiccia refactoring di tutto il codice di base "solo perché" se.

Answer 8

Se è già registrato, fare attenzione. Alcune persone ottenere terribilmente permaloso ed è considerato maleducato se qualcuno della tua squadra sta ancora utilizzando strumenti da una decina di anni fa, che non può disattivare il rilevamento spazio bianco durante un diff.

In generale, io ripulire il codice styling per portarlo in linea con gli obiettivi di stile organizzazione ha dichiarato quando sto toccando il codice per un altro motivo. Cercate di ricordare che lo stile è lo stile, però, e come tale non v'è "un vero modo." Non fare i nemici, perché non ti piace il fatto che un collega utilizza meno (o più) spazi di te.

Answer 9

Il mio suggerimento sarebbe che se si fa una riformattazione che sia fatto separatamente da qualsiasi modifiche di codice. La ragione di questo è che è possibile contrassegnare come "riformattare solo" nel repository e un cambiamento di codice legittimo non saranno sepolti nel bel mezzo di un mucchio di cambiamenti di spaziatura rendendo impossibile per voi o la persona accanto a capire che cosa è rotto .

Answer 10

Come la maggior parte delle persone hanno già detto, sì, meglio il refactoring e lasciare più ordinato.

refactoring aiuta tutti ottenere una migliore, e tutti dovrebbero avere accesso agli strumenti di refactoring (CodeRush per me).

Tuttavia, se i vostri colleghi non condividono l'amore refactoring, allora è una buona opportunità per voi per illuminarli:)