PMD - Project Mess Creator

2009-08-24 14:46:55

Hab gerade PMD auf code von mir losgelassen:

public class foo { 
    public String toString() { 
        return this.bar + ":" this.baz + ":" + this.xyz;
    }
}

-> meckert er, dass ":" keine variable is, also mehrere literale verwendet. ok.

String sX = ":";

-> meckert er, das könnte final sein

final String sX = ":";

-> meckert er, dass lokale Variablen nicht final sein müssen -> ausserdem ist + böse, also StringBuffer benutzen

Bläht also insgesamt den Code von:

public class foo { 
    public String toString() { 
        return this.bar + ":" this.baz + ":" + this.xyz;
    }
}

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();
    }
}

auf.

Und dann nennt es sich Project Mess Detector statt Creator...

About

Life's a bitch, life's a whore. Nothing less, nothing more.

Read More