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

    Всего: 15

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

    +74

    1. 1
    for (Integer i = 0; i<_pwiList.size(); i++)

    Вот к чему приводит бездумное использование классов-обёрток "для унификации". Конечно, оптимизатор может здесь сообразить, но не факт. А если данный фрагмент трактовать буквально, то i++ приведёт к unboxing, увеличению и последующему boxing.

    konsoletyper, 30 Июля 2010

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

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

    +75

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    PriceWithInterval pwi = new PriceWithInterval();
            pwi.setPrice(p);
            pwi.setStart(dateFormat.parse(dateFormat.format(p.getStartDate())).getTime());
            Long endTm = dateFormat.parse(dateFormat.format(p.getEndDate())).getTime();
            //Больше 50 лет - техническая бесконечность
            if (((endTm-pwi.getStart())/1000/3600/24/365)>50)
            {
                endTm = dateFormat.parse("31.12.9999").getTime();
            }

            // Где dateFormat объявлен как
            new SimpleDateFormat("dd.MM.yyyy");

    Вот такое вот масло масляное. Думается, что этот трюк применён здесь для того, чтобы получить начало дня. Чем не устраивает Calendar, неясно. Ну в крайнем случае, можно было целочисленно поделить и умножить на 1000 * 3600 * 24. Ну и вычисление в общем-то константной "технической бесконечности" радует неимоверно.

    konsoletyper, 30 Июля 2010

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

  4. Java / Говнокод #3688

    +79

    1. 1
    long sourceBaseId = Long.parseLong(_depId+"00000000000000");

    konsoletyper, 12 Июля 2010

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

  5. Java / Говнокод #2940

    +69.2

    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
    14. 14
    15. 15
    16. 16
    List<Measure> meaList = q.getResultList();
        try
        {
            if (meaList.size() == 0)
            {
                throw new Exception("Measure not found: "+_wsPrice.getMeasureIso());
            }
            else
            {
                p.setMeasure(meaList.get(0));
            }
        }
        finally
        {
            meaList.clear();
        }

    А вот ещё порция паранойи от моего коллеги

    konsoletyper, 06 Апреля 2010

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

  6. Java / Говнокод #2937

    +77.6

    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
    14. 14
    15. 15
    16. 16
    public void run()
    {
        try
        {            
            startExport();
        }
        catch (Throwable _t)
        {
            sendEvent(""+_t.getMessage(), true);
            logger.log(Level.SEVERE, charset, _t);
        }
        finally
        {
            System.gc();
        }
    }

    А ещё этот человек дёргает в одном сервисе gc каждые 4 секунды. Любые попытки ударить по рукам нарываются на отлуп: "так работает же!". Это не студент! Это реальный production код!

    konsoletyper, 06 Апреля 2010

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