Java — Говнокод.ру

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

−3

1

reader = new CSVReader(new BufferedReader(new InputStreamReader(new FileInputStream(csvFile), "UTF-8")), Config.getCSVDelimiter());

Хорошо что файл ещё не зазипован...

Запостил: tinynick, 29 Июля 2010

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

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

+4

1

s.replaceAll("(\\/\\.\\.\\/)+", "/").replaceFirst("^((\\.)+\\/)+", "");

вот такие вот суровые паттерны
как я это увидел - мне сразу вид двух друзей переханных трактором вспомнился

String s="../../../a./../../files/file1.rar"; // надо убрать строки вида "../"

Запостил: 3.14159265, 28 Июля 2010

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

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

+5

1
2
3
4
5
6
7

public class SomeClass
{
	@SuppressWarnings("unused")
	private Validator validator;

	... 
}

Эта переменная реально не используется. Удалить бы - ан нет, автор почему-то считает, что лучше пометить её как неиспользуемую, чтобы компилятор и Эклипс не ругались.
Чем только народ думает...

Запостил: asolntsev, 21 Июля 2010

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

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

+9

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

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

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

+7

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

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

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

0

01
02
03
04
05
06
07
08
09
10
11
12

public static String getClassName(final HWND hWnd) {
	final char[] cname_b = new char[1024];
	User32.INSTANCE.GetClassName(hWnd, cname_b, 1000);

	final char[] cname_b_truncated = ArrayUtils.toPrimitive(Iterables
		.toArray(Functional.takeWhile(Arrays.asList(ArrayUtils
			.toObject(cname_b)), Predicates.not(Predicates
			.equalTo(Character.valueOf((char) 0)))),
			Character.class));

	return String.valueOf(cname_b_truncated);
    }

Только что выдавил)

Интерес представляют строки с 5 по 9. Тут должно было быть всего-лишь
takeWhile (/= '\0') cname_b
но из-за боксинга/анбоксинга (примитивные типы в Java не могут быть параметрами Generic), и из-за того, что массивы в Java не Iterable - получилось ЭТО.

Запостил: burdakovd, 17 Июля 2010

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

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

+6

1
2
3
4
5
6

float lan = getLan();
       float lon = getLan();
       if(lan == lan || lon == lon)
       {
           throw new RuntimeException("Bad value");
       }

Из моего реалного проекта. Этот код реально делает полезную вещь

Запостил: xaoc, 16 Июля 2010

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

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

+5

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

for (JButton button : buttons) {
    if(buttons.length>0&&button==buttons[0]){
        if (canCreate()||button.getName().equals(BTN_EDIT_CONTRACT)) {
            UiUtils.setEnabled(button, true, canCreate()|| button.getName().equals(BTN_EDIT_CONTRACT));
        } else {
            UiUtils.setEnabled(button, false, canCreate()|| button.getName().equals(BTN_EDIT_CONTRACT));
        }
    }
    if(buttons.length>1&&button==buttons[1]){
        if (canEdit()||canView()||button.getName().equals(BTN_EDIT_CONTRACT)) {
            UiUtils.setEnabled(button, true, canEdit()||canView()|| button.getName().equals(BTN_EDIT_CONTRACT));
        } else {
            UiUtils.setEnabled(button, false, canEdit()||canView() || button.getName().equals(BTN_EDIT_CONTRACT));
        }
    }
    if(buttons.length>2&&button==buttons[2]){
        if (canDelete()||button.getName().equals(BTN_EDIT_CONTRACT)) {
            UiUtils.setEnabled(button, true, canDelete()|| button.getName().equals(BTN_EDIT_CONTRACT));
        } else {
            UiUtils.setEnabled(button, false, canDelete() || button.getName().equals(BTN_EDIT_CONTRACT));
        }
    }
    if(buttons.length>3&&button==buttons[3]){
        if (canEdit()||canView()||button.getName().equals(BTN_EDIT_CONTRACT)) {
            UiUtils.setEnabled(button, true, canEdit()||canView()|| button.getName().equals(BTN_EDIT_CONTRACT));
        } else {
            UiUtils.setEnabled(button, false, canEdit()||canView() || button.getName().equals(BTN_EDIT_CONTRACT));
        }
    }
    if(buttons.length>4&&button==buttons[4]){
        if (canEdit()||canView()||button.getName().equals(BTN_EDIT_CONTRACT)) {
            UiUtils.setEnabled(button, true, canEdit()||canView()|| button.getName().equals(BTN_EDIT_CONTRACT));
        } else {
            UiUtils.setEnabled(button, false, canEdit()||canView() || button.getName().equals(BTN_EDIT_CONTRACT));
        }
    }
}

Запостил: borka, 15 Июля 2010

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

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

+11

1

void setEnabled(boolean disable)