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

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

+80

1
2
3
4
5

public static java.sql.Date currentSQLDate() {
        java.sql.Date result = null;
        Date date = new Date();
        return result;
    }

ох, ёптеть...

grobotron, 12 Марта 2013

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

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

+76

1

public static final String RANDOM_VALUE_QUERY = "select to_number(to_char(dbms_random.value(100,999),'999'),'999') from dual";

Прогрессивный способ генерации случайных чисел=.

nafania217518, 11 Марта 2013

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

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

+75

1
2
3
4
5

public Double toDouble(String str){
	if ((str==null)||(str.equals(""))) str = "0.0";
	if ((str.contains(","))&&(!str.contains("."))) str = str.replace(",", ".");
	return new Double(str);
}

Лишняя защита никогда не бывает лишней

nafania217518, 06 Марта 2013

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

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

+114

1
2
3
4
5
6
7
8
9

private List<PalletePath> killDupes(List<PalletePath> pathesNew) {
	List<PalletePath> noDupes = new ArrayList<PalletePath>();
	for (PalletePath tp : pathesNew) {
		if (!noDupes.contains(tp)) {
			noDupes.add(tp);
		}
	}
	return noDupes;
}

Set? Не, не слышали.

someone, 05 Марта 2013

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

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

+74

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18

private boolean userInOneRegistrationNode() throws DfException {

        String uname = OrganizationStaffStructureHelper.getCurrentUserName();

        int i = 0;
        IDfCollection NodesCol = DQLHelper.getCollection(DQL_GET_REGISTRATOR_DIV, new String[]{uname});
        while (NodesCol.next()) {
            if (!(NodesCol == null)) {
                String group_name = NodesCol.getString(GROUP_NAME);
                i = i + 1;
            }
        }

        if (i == 1) {
            return true;
        }
        return false;
}

Заменяется 2мя строками один - select count(*), вторая - полученный результат Integer.ValueOf(...).

Landing, 04 Марта 2013

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

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

+71

1
2
3
4
5
6
7

String currentDate = new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").format(new Date());
 accountNumber.append(String.valueOf(1900 + new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getYear()));
 accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getMonth()));
 accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getDate()));
 accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getHours()));
 accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getMinutes()));
 accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getSeconds()));

ну а чо, зато внушительно выглядит

nafania217518, 01 Марта 2013

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

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

+81

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

public String isThisScreenWithHdpiDensity() {
    DisplayMetrics metrics = new DisplayMetrics();
    getWindowManager().getDefaultDisplay().getMetrics(metrics);
    switch (metrics.densityDpi) {
        case DisplayMetrics.DENSITY_LOW:
            return "ldpi";
        case DisplayMetrics.DENSITY_MEDIUM:
            return "mdpi";
        case DisplayMetrics.DENSITY_HIGH:
            return "hdpi";
    }
    return "xhdpi";
}

...
if (isThisScreenWithHdpiDensity().equals("hdpi")) {
    lpTabPane.height = 65;
}
if (isThisScreenWithHdpiDensity().equals("mdpi")) {
    lpUpperPane.height = 70;
}
if (isThisScreenWithHdpiDensity().equals("ldpi")) {
    lpTabPane.height = 32;
}
if (isThisScreenWithHdpiDensity().equals("xhdpi")) {

}

Перегоним константу в стрингу, а потом проверим ее equals. Отлично!

danikula, 28 Февраля 2013

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

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

+142

1
2

if(!true) // это - не говнокод. это - заглушка.
                return false;

Оправдывается ещё )

pitch, 21 Февраля 2013

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

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

+67

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

public final class SomeActivity extends Activity {
       @Override
	protected Dialog onCreateDialog(int id) {
		Dialog dialog = null;
		if (id == DialogGenerator.SETTINGS_DIALOG) {
			dialog = mDialogGenerator.createSettingsDialog();
		}  {
			dialog = super.onCreateDialog(id);
		}
		return dialog;
	}

	@Override
	protected void onPrepareDialog(int id, Dialog dialog) {
		super.onPrepareDialog(id, dialog);
		if (id == DialogGenerator.SETTINGS_DIALOG) {
			mDialogGenerator.prepareSettingsDialog((AlertDialog) dialog, someBoolValue, someObjectValue);
		} 
	}
}

public final class DialogGenerator {
	public Dialog createSettingsDialog() {
		int dialogId = SETTINGS_DIALOG;
		int titleId = R.string.settingsTitle;
		String[] itemsArray = getStringArray(R.array.settings);
		ThreeTypeOptionsAdapter adapter = new ThreeTypeOptionsAdapter(mControllerAsActivity, itemsArray,
				mCheckableOptions, mTwoTextOptions);
		AlertDialogCallback dialogCallback = new AlertDialogCallback(mControllerAsDialogHost, dialogId);
		AlertDialog.Builder dialogBuilder = new AlertDialog.Builder(mControllerAsActivity);
		dialogBuilder.setTitle(titleId);
		// костыль
		dialogBuilder.setSingleChoiceItems(itemsArray, 0, null);
		// конец костыля
		dialogBuilder.setAdapter(adapter, dialogCallback);
		AlertDialog dialog = dialogBuilder.create();
		setMainListenersOnDialog(dialog);
		dialog.getListView().setChoiceMode(ListView.CHOICE_MODE_MULTIPLE);
		return dialog;
	}
	
	public void prepareSettingsDialog(AlertDialog dialog, boolean someBoolValue, String someStringValue) {
		//костыль
		ListView dialogList = dialog.getListView();
		dialogList.clearChoices();
		dialogList.setItemChecked(POSITION_FOR_BOOL, someBoolValue);
		//конец костыля
		ThreeTypeOptionsAdapter adapter = (ThreeTypeOptionsAdapter) dialogList.getAdapter();
		adapter.setAdditionalValue(POSITION_FOR_STRING, someStringValue);	
	}
}

Задача: показать диалог, внутри которого есть 3 типа ячеек: выделяемые с чекбоксом справа, невыделяемые с дополнительной надписью справа, обычные невыделяемые.
Примерно так:
Use GPS checkbox
Selected country Russia
Launch some activity

В комментах к методам я описал костыль:
Здравствуйте, дорогие друзья. Сегодня мы с вами поговорим об уникальном виде животных, которых открыли только в конце XX века. Это, дорогие друзья, Ява-обезьяны. Давайте дружно откроем сырцы Андроид-СДК, а именно - класс com.android.internal.app.AlertController .AlertParams - и метод createListView(AlertController). Видите, одна обезьяна решила, что нам будет удобней, если диалог автоматом задисмиссится, когда мы щёлкнем по элементу списка в CHOICE_MODE_NONE, и OnClickListener будет не null? А теперь давайте перейдём в android.app.AlertDialog.Builder. Видите, там другая обезьяна решила, что для multi-choice списка нам не потребуется хитроумный адаптер? Поэтому, дорогие друзья, когда мы поставили кастомный адаптер (setAdapter()) и сопроводили его слушателем, то какой бы мы choice mode не ставили после AlertDialog.Builder.create(), диалог дисмиссился. Поэтому, дорогие друзья, пришлось пойти на хитрость и вначале установить single choice (чтобы флаг mIsSingleChoice стал true), а потом уже ставить адаптер.

QuickNick, 20 Февраля 2013