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

Всего: 2

Java / Говнокод #3389

+68

1
2
3
4
5
6

def InvoicedExtra(JobExtra extra) {
   type = extra.type
   clientPrice = extra.price ?: ZERO
   vatPrice = (extra.type.vatBand?.band ?: 0) * extra.price / 100
   grossPrice = vatPrice ?: ZERO + clientPrice ?: ZERO
}

Мой код на Groovy. Обратите внимание на строку 5. Это типа хитрый способ суммирования который не бросает NullPointerException. Однако не учтены приоритеты операторов. Оцените красоту выстрела в ногу =)

tinhol, 03 Июня 2010

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

Java / Говнокод #2941

+72.4

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79

interface Editor extends Window {

    /** Get edited entity  */
    Entity getItem();

    /**
     * Set parent datasource to commit into this datasource instead of database.
     * This method must be followed by {@link #setItem(com.contora.package.core.entity.Entity)}
     */
    void setParentDs(Datasource parentDs);

    /** Set edited entity. Invoked by the framework on opening the window. */
    void setItem(Entity item);

    /** Check validity by invoking validators on all components which support them */
    boolean isValid();

    /** Check validity by invoking validators on all components which support them */
    void validate() throws ValidationException;

    /** Validate and commit changes */
    boolean commit();

    /** Commit changes with optional validating */
    boolean commit(boolean validate);

    /**
     * Validate, commit and close if commit was successful.
     * Passes {@link #COMMIT_ACTION_ID} to associated {@link CloseListener}s
     */
    void commitAndClose();
}

public class PickupRegionLookuper extends AbstractLookup implements Editor{
    public PickupRegionLookuper(IFrame frame) {
        super(frame);
    }

    @Override
    protected void init(Map<String, Object> params) {
       //Здесь была какая то логика
    }

    @Override
    public Entity getItem() {
        return null;  
    }

    @Override
    public void setParentDs(Datasource parentDs) {
    }

    @Override
    public void setItem(Entity item) {
    }

    @Override
    public boolean isValid() {
        return false;
    }

    @Override
    public void validate() throws ValidationException {
    }

    @Override
    public boolean commit() {
        return false;
    }

    @Override
    public boolean commit(boolean validate) {
        return false;
    }

    @Override
    public void commitAndClose() {
    }
}

Встретилось в одном java-файле. Класс реализует интерфейс, определенный в том же файле, причем интерфейс package-private (!). При этом ни одного метода оттуда реально не используется. К тому же этот интерфейс человек перенес из другого класса, где тот был как раз public.

tinhol, 06 Апреля 2010

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

Говнокод: по колено в коде.