1. Java / Говнокод #3479

    +69

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    private String setArticleHTML() {
    // ...
}

...

mArticleView.loadDataWithBaseURL(mBaseUrl, setArticleHTML(), "text/html", "UTF-8", "");

    yvu, 15 Июня 2010

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

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

    +74

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    protected void setPhones(ArrayList<MBPhone> phones) {
    if(phones!=null) 
        this.phones=phones;
    else 
        phones.clear();
}

    Код из одного западного вэб-сервиса для профессионалов в сфере недвижимости.
    Мораль: не надо игнорировать подсказки IDE. Автор на самом деле хотел очистить this.phones. И Intelij Idea подсказывает, что в этом месте может быть брошен NullPointerException. Увидеть эту проблему можно было только используя аннотацию @SuppressWarnings. Потому что варнингов так много, что различить среди них опасные очень сложно.

    vladimir.loshchin, 10 Июня 2010

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

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

    +80

    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
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    private JMyForm frm;

public void view() 
{
    if (((JOurComponent) frm).getAction() == Actions.Action1 
    {
        viewList(Actions.Action1);
    } 
    else
    if (((JOurComponent) frm).getAction() == Actions.Action2
    {
        viewList(Actions.Action2);
    }
    ...
    else
    if (((JOurComponent) frm).getAction() == Actions.Action117
    {
        viewList(Actions.Action117);
    }
}

private void viewList(String action) {
    if (action == Actions.Action1)
    {
        //
    }
    ...
    if (action == Actions.Action117)
    {
        //
    }
}

    Что тут может радовать? ООП отменяется. И паттерны туда же.

    lotik, 10 Июня 2010

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

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

    +72

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (doSomething() == true) {
    if (doSomething2()) {
        //...
    }
}

    lotik, 10 Июня 2010

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

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

    +66

    1. 1
    2. 2
    3. 3
    if (xobj.getStringProp() == "N") {
    //do it
}

    Ну а что тут скажешь?

    lotik, 10 Июня 2010

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

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

    +75

    1. 1
    2. 2
    3. 3
    if ((getProductId() != null) && (getProductId().equals(new String("5")))) {
  //do it
}

    ProductId читается один раз при старте приложения, а таких проверок около 50-и.

    lotik, 10 Июня 2010

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

  7. Java / Говнокод #3432

    +83

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    try {
    //snipped
} catch(OmgExtremeCriticalEmergencyException ignored) {
    //not important
}

    gvsmirnov, 10 Июня 2010

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

  8. Java / Говнокод #3424

    +144

    1. 1
    textPane.setText(<span style="font-size: 20pt">Big</span>);

    http://www.dynamicdrive.com/forums/showthread.php?t=18899

    O_O

    я подозревал, что вас обрезают, но что бы так?

    Lure Of Chaos, 08 Июня 2010

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

  9. Java / Говнокод #3418

    +75

    1. 1
    setDouble((float)getAnotherDouble());

    Не могу понять: то ли ето какой-то реликтовый артефакт, то ли в етом есть глубокий смысл.
    Подскажите, а..

    OlegYch, 08 Июня 2010

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

  10. Java / Говнокод #3395

    +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
    14. 14
    15. 15
    16. 16
    private void StreamToBytes()throws Exception
    {
        int i;
        InputStream in=getStream();
        ArrayList<Integer> byt=new ArrayList<Integer>();
        while((i=in.read())!=-1)
        {
            byt.add(i);
        }
        in.close();
        b=new byte[byt.size()];
        for(int j=0;j<b.length;j++)
        {
            b[j]=byt.get(j).byteValue();
        }
    }

    хуита, 04 Июня 2010

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