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

Всего: 15

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

+76

1
2
3
4
5

private Boolean active = false;
...
synchronized (active) {
...
}

Чудо синхронизации. Блокируется раз и навсегда.

borka, 25 Августа 2014

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

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

+79

1
2
3
4
5

public class LessOperator extends GreaterOperator {
    public Object evaluateNode(ITreeNode node, List<NodeValuePair> list) throws CalculatorException {
        return !(Boolean)super.evaluateNode(node, list);
    }
}

Видимо тут программист решил сократить себе работу и реализовал оператор сравнения "меньше" как "не больше", использовав готовую реализацию оператора "больше".
В результате вместо операции "меньше" получили операцию "меньше или равно".

borka, 30 Декабря 2010

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

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

+144

1
2

// указываем какие label делать подчеркнутыми
setMarked(new boolean[] {false, true, true, true, true, true, true, false, true, false, false, false, false, true, true, false});

borka, 16 Сентября 2010

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

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

+70

01
02
03
04
05
06
07
08
09
10

/**
 * Окно, которое реагирует на нажатие клавиши Esc
 * определенным образом (например, окно закрывается).
 * Для этого в потомках класса необходимо реализовать метод
 * doEscapeKeyAction(), который будет вызываться при нажатии
 * клавиши ESC.
 */
public abstract class JEscFrame extends ProfilableFrame {
...
}

borka, 22 Августа 2010

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

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

+72

1

public static final String DOCUMENT_NUMBER_FORMAT = "******************************";

borka, 18 Августа 2010

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

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

+76

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

public static void setSQLSafeFormat(JFormattedTextField ftf){
    DefaultFormatter sqlSafeFormatter = new DefaultFormatter(){
        @Override
        public Object stringToValue(String string) throws ParseException {
            string = string.replaceAll("\'", "");
            return super.stringToValue(string);
        }
        @Override
        public String valueToString(Object value) throws ParseException {
             String result = super.valueToString(value);
             return result.replaceFirst("\'", "");
        }
    };
    sqlSafeFormatter.setOverwriteMode(false);
    ftf.setFormatterFactory(new DefaultFormatterFactory(sqlSafeFormatter));
}
public static void setSQLSafeFilter(JTextField txt){
    DocumentFilter dc = new DocumentFilter(){
        @Override
        public void insertString(FilterBypass fb, int offset, String string, AttributeSet attr) throws BadLocationException {
            if(!string.contains("'"))
            super.insertString(fb, offset, string, attr);
        }
        @Override
        public void replace(FilterBypass fb, int offset, int length, String text, AttributeSet attrs) throws BadLocationException {
            if(!text.contains("'"))
            super.replace(fb, offset, length, text, attrs);
        }
    };
    AbstractDocument asb = (AbstractDocument)txt.getDocument();
    asb.setDocumentFilter(dc);
}

суровая борьба с sql injection

borka, 09 Августа 2010

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

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

+71

1
2
3
4
5
6
7

public void changeUI() {
    // Во все контролы ниже вместо .setEnabled(true) пишем
    // .setEnabled(_true), чтобы не рассерить
    // контролы при поднятии формы на просмотр
    boolean _true = getMainBehaviour().canEditForm() != ReadOnlyState.READ_ONLY;
    ...
}

borka, 06 Августа 2010

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

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

+74

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

public static int count(String s) {
    int res = 0;
    for (int i = 0; i < s.length(); i++) {
        if(s.charAt(i) == '1') res++;
    }
    return res;
}

public List<VDictLiability> get(String currency_id, String usedTrailer, Date actualDate, String contract_class_id, String product_id, String risk_object_type_id) {
    for(int i = 0;i<5;i++){
        int j = 0;
        while(j < 32) {
            if(count(Integer.toBinaryString(j)) == i) {
                String s = Integer.toBinaryString(j);
                while (s.length() != 5) {
                    s = "0" + s;
                }
                List<VDictLiability> result = get(currency_id, usedTrailer, actualDate, contract_class_id, product_id, risk_object_type_id, s);
                if(result.size() > 0) return result;
            }
            j++;
        }
    }
    return Collections.EMPTY_LIST;
}

public List<VDictLiability> get(String currency_id, String usedTrailer, Date actualDate, String contract_class_id, String product_id, String risk_object_type_id, String mask) {
    String territoryCode = App.config.getCurrentBrunchCode().substring(1, 3);
    
...
    
    HashMap<Double, VDictLiability> map = new HashMap<Double, VDictLiability>();
    for (VDictLiability liabilityRow : get()) {
        if(!territoryType.equals(liabilityRow.getterritory_type_id()) && !(mask.charAt(4) == '1' && liabilityRow.getterritory_type_id() == null)){
            continue;
        }
        if(actualDate != null && !CompareUtil.isInside(liabilityRow.getstart_date(), liabilityRow.getend_date(), actualDate)){
            continue;
        }
        if(((risk_object_type_id != null && !CompareUtil.isEquals(liabilityRow.getrisk_object_type_id(), risk_object_type_id)) && !(mask.charAt(0) == '1' && liabilityRow.getrisk_object_type_id() == null))){
            continue;
        }
        if(((product_id != null && !CompareUtil.isEquals(liabilityRow.getproduct_id(), product_id)) && !(mask.charAt(1) == '1' && liabilityRow.getproduct_id() == null))){
            continue;
        }
        if(((contract_class_id != null && !CompareUtil.isEquals(liabilityRow.getcontract_class_id(), contract_class_id)) && !(mask.charAt(2) == '1' && liabilityRow.getcontract_class_id() == null))){
            continue;
        }
        if((liabilityRow.getcurrency_id() != null & !CompareUtil.isEquals(liabilityRow.getcurrency_id(), currency_id)) && !(mask.charAt(3) == '1' && liabilityRow.getcurrency_id() == null)){
            continue;
        }
        if(liabilityRow.getis_used_trailer() != SQLUtils.NULL_INTEGER_VALUE & !CompareUtil.isEquals(""+liabilityRow.getis_used_trailer(), usedTrailer)){
            continue;
        }
        map.put(liabilityRow.getvalue(), liabilityRow);
    }
    
    ArrayList<VDictLiability> result = new ArrayList<VDictLiability>();
    result.addAll(map.values());
...
    return result;
}

public List<VDictLiability> get(String currency_id, boolean usedTrailer, Date actualDate, String contract_class_id, String product_id, String risk_object_type_id) {
    return get(currency_id, usedTrailer?"1":"0", actualDate, contract_class_id, product_id, risk_object_type_id);
}

Это ад

borka, 03 Августа 2010

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

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

+82

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

@ValidationExpression(value = "if (inn != null && !\"\".equals(inn)) {\n" +
                              "char[] st = inn.toCharArray();\n" +
                              "for(int i=0; i < st.length; i++) {\n" +
                              "if (!Character.isDigit(st[i])) { return false; }\n" +
                              "}\n" +
                              "}" +
                              "return true;", message = Messages.WRONG_INN_FORMAT)
@Expression(value = "if (inn != null && !\"\".equals(inn)) {\n" +
                    "char[] st = inn.toCharArray();\n" +
                    "for(int i=0; i < st.length; i++) {\n" +
                    "if (!Character.isDigit(st[i])) { return false; }\n" +
                    "}\n" +
                    "}" +
                    "return true;", messageKey = Messages.WRONG_INN_FORMAT)
public String getinn() {
	return getPerson().getinn();
}

@Required(messageKey = Messages.REQUIRED)
@DictEntityExists(dictId = MDictionaries.DICT_CURRENCY, message = Messages.NO_DICT_ENTITY)
@Expression(value = "SystemCounter.DICTCURRENCY__RUR_CURRENCY_TYPE.equals(agentStatementCurrencyId) ? " +
                    "   (" +
                    "       SystemCounter.DICTCURRENCY__RUR_CURRENCY_TYPE.equals(currency_id) ? " +
                    "           true : " +
                    "           SystemCounter.DICTCONTRACTCLASS__CONTRACT_CLASS_XXX.equals(contract_class_id) ? " +
                    "               false : " +
                    "               is_currency_equivalent == 1" +
                    "   ) : (" +
                    "       SystemCounter.DICTCONTRACTCLASS__CONTRACT_CLASS_GENERIC.equals(contract_class_id) &&" +
                    "       is_currency_equivalent == 0 &&" +
                    "       agentStatementCurrencyId.equals(currency_id)" +
                    "   )", messageKey = Messages.CURRENCY_NOT_EQUIVALENT_WITH_AGENT_STATEMENT)
public String getcurrency_id() {return getContract().getcurrency_id();}

нет слов

borka, 21 Июля 2010

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

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

+79

1
2
3
4
5
6
7

private void getCurrentBranch();
public TDictBranch getCurrentBrach();
public TDictBranch2 getCurrentBrach2();
public TDictBranch2 getCurrentBrach2(Date lookupDate);
public String getCurrentBrachId();
public String getCurrentBrunchCode();
public static String getCurrentBranchCharId();

некоторые методы из класса конфигурации приложения

borka, 20 Июля 2010

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

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