Java setter e "questo"

Question

ho notato in un sacco di persone che fanno in setter Java:

1)

public void setX(int x) { 
    this.x = x; 
} 

Personalmente non mi piace questo e penso che dovrebbe essere qualcosa di simile:

2)

public void setX(int newX) { 
    x = newX; 
} 

Esistono ragioni la prima sarebbe meglio?

Non è 1) più facile effettuare un errore con. Su un paio di occasioni ho rintracciato bug nel codice fino a persone che fanno:

x = x; 

per sbaglio, forse perché erano a digitare per digiunare e solo voluto ottenere i getter e setter fuori strada.

Answer 1

È una questione di stile. L'argomento a favore di

public void setX(int x) { this.x = x; } 

è che non c'è bisogno di pensare a un nome nuovo e significativo per l'argomento di input, dal momento che avete già uno.

Scegliere uno stile e utilizzarlo in modo coerente.

Answer 2

Non penso ci sia un "migliore" o "peggio" qui. È solo una questione di stile.

Answer 3

motivi per preferire this.x = x:

1. utilizzando this.x = x nei costruttori ed è un modello noto, che i programmatori Java riconoscono.
2. Il x in entrambi i campi e il parametro rappresentano lo stesso valore.
3. Con x = newX non c'è indicazione sintattica che uno sia un campo e l'altro un parametro.

Answer 4

Suppongo che i setter siano generati automaticamente dall'IDE. Quindi la mia preferenza è il modulo che viene utilizzato di default nell'IDE. Vale a dire, sono abbastanza neutrale e ritengo che non meriti di modificare le impostazioni IDE predefinite.

Answer 5

Preferisco this.x = x;. Non c'è davvero nessun modo giusto su questo. In un IDE, avrà un bell'aspetto per il colore in più su this.

Answer 6

È solo una convenzione di codifica. Sei libero di scegliere, ma usalo in modo coerente. Inoltre, un IDE medio può generare automaticamente tutti i getter/setter in pochi tasti/clic. In Eclipse, ad esempio, fare Alt +Maiusc +R e poi S e quindi selezionare le proprietà che desideri per generare getter/setter per e fare clic su OK. Inoltre, un IDE medio visualizzerà un avviso quando si esegue x = x anziché this.x = x.

Io stesso preferisco lo this.x = x.Anche Eclipse e molti strumenti di reverse engineering (come Dali e Hibernate Tools) lo generano in modo predefinito.

La convenzione x = newX (il Hungarian notation) è appena ereditata dal mondo della programmazione procedurale (vecchio stile) e semplicemente non si adatta perfettamente all'ideologia OO.

Answer 7

Se è il tuo codice o non fa parte di una base esistente, scegli quella che ti piace e segui. Se invece stai lavorando con codice esistente segui qualsiasi convenzione sia già in uso.

Answer 8

E 'del tutto soggettivo, ma il primo modulo fornisce un bel metodo prototype chiaro nel javadoc ecc

non mi sento non c'è molto spazio per l'errore utilizzando il modulo 'this.x = x'. Sicuramente nulla che non verrà catturato dai test unitari. Hai dei test unitari, giusto?