Hab gerade PMD auf code von mir losgelassen:
-> meckert er, dass ":" keine variable is, also mehrere literale verwendet. ok.
-> meckert er, das könnte final sein
-> meckert er, dass lokale Variablen nicht final sein müssen
-> ausserdem ist + böse, also StringBuffer benutzen
Bläht also insgesamt den Code von:
auf
auf.
Und dann nennt es sich Project Mess Detector statt Creator...
-> meckert er, dass ":" keine variable is, also mehrere literale verwendet. ok.
-> meckert er, das könnte final sein
-> meckert er, dass lokale Variablen nicht final sein müssen
-> ausserdem ist + böse, also StringBuffer benutzen
Bläht also insgesamt den Code von:
auf
public class foo {
public String toString() {
String sS;
sS = ":";
StringBuffer sbSB = new StringBuffer();
sb.append(this.bar);
sb.append(sS);
sb.append(this.baz);
sb.append(tsS);
sb.append(this.xyz);
return sb.toString();
}
}
public String toString() {
String sS;
sS = ":";
StringBuffer sbSB = new StringBuffer();
sb.append(this.bar);
sb.append(sS);
sb.append(this.baz);
sb.append(tsS);
sb.append(this.xyz);
return sb.toString();
}
}
auf.
Und dann nennt es sich Project Mess Detector statt Creator...
Trackbacks
Trackback specific URI for this entry
No Trackbacks
Comments
Display comments as
(Linear | Threaded)
public class Foo {
private static final String COLON = ":";
public String toString() {
return (new StringBuilder()).append(this.bar).append(COLON).append(this.baz).append(COLON).append(this.xyz).toString();
}
}
StringBuffer ist böse, weil synchronized, ab 1.5 nutzt man StringBuilder, sofern er nicht über Threads geshared wird (wozu mir kein sinnvolles Beispiel einfällt)
About
Life's a bitch, life's a whore. Nothing less, nothing more.
Read More
CC by-nc-sa 2.5
