1. Список говнокодов пользователя Skipy

    Всего: 2

  2. Java / Говнокод #7032

    +74

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    private static java.util.Hashtable<String, Session> sessions =
    new java.util.Hashtable<String, Session>();
...
public static SessionOperations[] getSessions() {
    int cnt = sessions.size();
    SessionOperations[] res = new SessionOperations[cnt];
    java.util.Enumeration<String> e = sessions.keys();
    for( int i = 0; i < res.length; i++ ) {
      String sid = e.nextElement();
      res[i] = sessions.get( sid ).getOperations( sid );
    }
    return res;
}

    Удаление из sessions между получением размера и набора ключей приводит к весьма печальным последствиям...

    P.S. В результате в течение трех суток в production не удалялись оставленные сессии (без Logout), sessions отожрал 600М и всё свалилось с OOM.

    Skipy, 23 Июня 2011

    Комментарии (4)

  3. Java / Говнокод #6683

    +79

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    Object sync;

try {
    synchronized (sync){
        sync.wait();
    }
} catch (NullPointerException npe){
    sync = new Object();
    synchronized (sync){
        sync.wait();
    }
}

    Вот так чел пытался избежать "ненужного" создания объекта-монитора. Память экономил...

    Skipy, 17 Мая 2011

    Комментарии (14)