setter und getter: Parameter Überprüfung

[Guest]

Hallo,
mich würde mal interessieren wie ein solches Schema zu bewerten ist. Ist das generell sinnvoll, ist das guter Stil, schlechter Stil, überflüssig etc. ? Klar kommt es wohl immer auf den Kontext an, aber wie macht ihr das ? Danke schonmal.

In einer Klasse wird in einer setter oder getter Methode immer der Parameter überprüft und in Abhängigkeit dessen, dann eine Zuweisung gemacht.

class AClass {
     private String name;
     private String age;

...

public void setName(String name){
     if(name == null | name.equal("")){
       this.name = name;
     }
}

etc..

}

 

[Guest]

kleine Korrektur: Die Beispielmethode sollte nat. so aussehen:

public void setName(String name){
       this.name = (name == null | name.equal("") ? "Undefinied" : name;
}
}

 

[Zed]

Wenn du einen default Parameter bei falscher Angabe haben willst wieso nicht. Ich habe gern eine NullPointerException somit weiß ich das irgendwas nicht richtig läuft.

 

[Landei]

Dir ist schon klar, dass

name == null | name.equals("")

eine NullPointerException wirft, wenn name null ist? Weswegen man normalerweise auch

name == null || name.equals("")

schreibt.

 

[Guest]

Oha das war mir tatsächlich nicht bewußt. hatte gewohnheitsmäßig immer && und || geschrieben meinte aber irgendwann mal, dass es das gleiche ergebnis bringt.

 

[Marco13]

Das kannst du machen, wie du es für richtig hältst. Das wichtige in diesem Fall:

/* 
 * Set the name of this thing. If the given name 
 * is null or the empty String, then the name 
 * "Undefined" will be assigned to this thing.
 *
 * @param name The new name.
 */
void setName(String name) { .... }

oder

/* 
 * Set the name of this thing. 
 *
 * @param name The new name. May not be null.
 */
void setName(String name) { .... }

 

[didjitalist]

sorry, aber die methode ist grosser bockmist. wenn null kein erlaubter parameter ist, dann MUSS der setter eine exception schmeissen, sofern der kommentar (im beispiel komplett fehlend) nichts gegenteiliges behauptet. ansonstent hat ein calller keine chance, fehler irgendwie zu erkennen.