L’époque s’y prête : pour finir l’année en beauté, et exploiter une formule qui fait les choux gras de TF1, voici une sélection de perles que nous avons rencontrées en 2010, sur des projets ou sur la toile.

Elle vient compléter nos sélections des années précédentes:

• Bêtisier 2007
• Bêtisier 2008
• Bêtisier 2009

N’hésitez pas à poster en commentaire vos propres rencontres du troisième type … A vous de jouer !

Après le break dance, le break coding

while (mamethode()) {
 break;
}

En entier …

public static int toInteger(String value) {
     try {
            return new Double(Double.parseDouble(value)).intValue();
     } catch (NumberFormatException e) {
           return 0;
     }
}

Un Long ?

private static final String DOT = ".";
...
public static boolean isLongNumber(String value) {
     try {
           Double.parseDouble(value);
           return !value.contains(DOT);
     } catch (NumberFormatException e) {
           return false;
     }
}

Cherchez l’erreur

public static ColumnsTableManagement getInstance() {
     synchronized (ColumnsTableManagement.class) {
            instance = (ColumnsTableManagement) FacesContext.getCurrentInstance().getExternalContext().getSessionMap().get("columnsTableManagement");
            if (instance == null) {
                instance = new ColumnsTableManagement();
                FacesContext.getCurrentInstance().getExternalContext().getSessionMap().put("columnsTableManagement", instance);
            }
            return instance;
     }
}

Une jolie trouvée dans un test unitaire

if (messagesList.size() == 3) {
    assertEquals(true, true);
} else {
    assertEquals(true, false);
}

Quelques nulls fraîchement instanciés

if (null == new Integer(srt.getValuationType())) {
    //...
}

GetTrace response = new GetTrace();
if (response != null)
    cache_getTrace.putValue(signature, response);
else
    cache_getTrace.failRelease(signature);

Au cas où une exception n’hériterait pas de Throwable

try {
    //...
} catch (Exception e) {
    if (e instanceof Throwable) {
        throw new SecurityManagerException(e);
    }
    if (code == 2) {
        accountsummary.setIsBlocked(accountBlocked);
    }
}