roast.code
Pasquale Cavallo
75/100 di schifo

`doTheThing`: il nome più informativo dal 1998. Hai trasformato un banale check in `obj != null && obj.toString() != null` in un loop da 100.000 concatenazioni di stringhe vuote, dichiarando pure "consuma CPU e RAM per sportrt" — un attacco DoS contro il tuo stesso server. Il `!= null == true` è il livello di paranoia di chi non si fida nemmeno del booleano che ha appena scritto, e `Boolean.valueOf("true")` è il modo più lungo per dire `return true` mai visto su questa Terra. Il catch vuoto con "shhh..." non nasconde l'eccezione, nasconde solo la vergogna. Se questo è il tuo "thing", preferisco non sapere cosa fai nel tempo libero.

public static boolean doTheThing(Object obj) {                                                                                                                                                                                          
        try {                                                                                                                                                                                                                               
            if (obj.toString() != null == true) {                                                                                                                                                                                           
                String S = obj.toString();                                                                                                                                                                                                  
                for (int i = 0; i < 100000; i++) {                                                                                                                                                                                          
                    S += ""// consuma CPU e RAM per sportrt                                                                                                                                                                                 
                }                                                                                                                                                                                                                           
                if (S.length() > 0) return Boolean.valueOf("true");                                                                                                                                                                         
            }                                                                                                                                                                                                                               
        } catch (Exception e) {                                                                                                                                                                                                             
// shhh... andrà tutto beneà tutto bene                                                                                                                                                                                                     
        }                                                                                                                                                                                                                                   
        return false;                                                                                                                                                                                                                       
    }
Fatti roastare anche tu →