Gli edifici hanno il loro posto, anche in Java. Dovresti usarli solo se le seguenti due cose sono vere:
- Hai solo bisogno di aggregare dati che non hanno alcun comportamento, ad es. passare come parametro
- Non importa un po 'quale tipo di valori hanno dati aggregati
Se questo è il caso, allora dovresti rendere pubblici i campi e saltare i getter / setter. Getter e setter sono comunque goffi, e Java è stupido per non avere proprietà come un linguaggio utile. Dal momento che il tuo oggetto struct-like non dovrebbe avere comunque alcun metodo, i campi pubblici hanno più senso.
Tuttavia, se uno di questi non si applica, hai a che fare con una vera classe. Ciò significa che tutti i campi dovrebbero essere privati. (Se hai assolutamente bisogno di un campo in un ambito più accessibile, usa un getter / setter.)
Per verificare se la tua presunta struttura ha un comportamento, guarda quando i campi sono usati. Se sembra violare dire, non chiedere , allora devi spostare quel comportamento nella tua classe.
Se alcuni dei tuoi dati non dovessero cambiare, allora devi rendere definitivi tutti quei campi. Potresti considerare di rendere la tua classe immutabile . Se è necessario convalidare i dati, fornire la convalida ai setter e ai costruttori. (Un trucco utile è definire un setter privato e modificare il campo all'interno della classe usando solo quel setter.)
Probabilmente il tuo esempio di Bottiglia fallirà entrambi i test. Potresti avere un codice (forzato) che assomiglia a questo:
public double calculateVolumeAsCylinder(Bottle bottle) {
return bottle.height * (bottle.diameter / 2.0) * Math.PI);
}
Invece dovrebbe essere
double volume = bottle.calculateVolumeAsCylinder();
Se hai cambiato altezza e diametro, sarebbe la stessa bottiglia? Probabilmente no. Quelli dovrebbero essere definitivi È un valore negativo ok per il diametro? La tua bottiglia deve essere più alta di quanto sia larga? Il Cap può essere nullo? No? Come stai convalidando questo? Supponiamo che il cliente sia o stupido o cattivo. ( È impossibile capire la differenza. ) Devi controllare questi valori.
Ecco come potrebbe apparire la tua nuova classe Bottle:
public class Bottle {
private final int height, diameter;
private Cap capType;
public Bottle(final int height, final int diameter, final Cap capType) {
if (diameter < 1) throw new IllegalArgumentException("diameter must be positive");
if (height < diameter) throw new IllegalArgumentException("bottle must be taller than its diameter");
setCapType(capType);
this.height = height;
this.diameter = diameter;
}
public double getVolumeAsCylinder() {
return height * (diameter / 2.0) * Math.PI;
}
public void setCapType(final Cap capType) {
if (capType == null) throw new NullPointerException("capType cannot be null");
this.capType = capType;
}
// potentially more methods...
}