Bêtisier 2010

Article publié par Xebia France le 31 décembre 2010.

Catégorie(s) : Divers

 

5 commentaires »

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

5 réponses à “Bêtisier 2010”

  1. Fred dit :

    Moi j’ai vu cette année :

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

    Le meilleur de cette année :

    public void update() {
        if(null != this) {
            refresh();
        }
    }
    
  3. Mike dit :

    Ca fait peur …
    Surtout ceux qui, au final, fonctionnent …

  4. Olivier dit :

    Ma perle de l’année :

    public BigDecimal getBooleanValue() { …

  5. Sven dit :

    Vu récemment :

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

Laisser un commentaire

 

Page optimized by WP Minify WordPress Plugin