Bêtisier 2010

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:

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

Commentaire

5 réponses pour " Bêtisier 2010 "

Published by Fred , Il y a 12 ans

Moi j’ai vu cette année :

public static void maMethode(String monParam) {
   if (monParam.equals(null)) {
      throw new IllegalArgumentException();
   }

   ...
}

Published by NSV , Il y a 12 ans

Le meilleur de cette année :

public void update() {
    if(null != this) {
        refresh();
    }
}

Published by Mike , Il y a 12 ans

Ca fait peur …
Surtout ceux qui, au final, fonctionnent …
Published by Olivier , Il y a 12 ans

Ma perle de l’année :

public BigDecimal getBooleanValue() { …
Published by Sven , Il y a 12 ans

Vu récemment :

if (id <= 0) {
return 0;
}
String idStr = String.valueOf(id);
return Integer.parseInt(idStr.substring(idStr.length() – 1));