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

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

+29

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

if (searchBox.isFocused()) {
			addressBookPopupMenu.enableItem(0, false);//view
			addressBookPopupMenu.enableItem(1, true);//search
			addressBookPopupMenu.enableItem(2, true);//add contact
			addressBookPopupMenu.enableItem(3, false);//edit contact
			addressBookPopupMenu.enableItem(4, false);//delete contact
			addressBookPopupMenu.enableItem(5, false);            
		}
		else {
			addressBookPopupMenu.enableItem(0,true);
			addressBookPopupMenu.enableItem(1,false);
			addressBookPopupMenu.enableItem(2,true);
			addressBookPopupMenu.enableItem(3,true);
			addressBookPopupMenu.enableItem(4,true);
			addressBookPopupMenu.enableItem(5, true);           
		}

		if(isSocialNWportal()) {
			addressBookPopupMenu.enableItem(2, false);//add contact
			addressBookPopupMenu.enableItem(3, false);//edit contact
			addressBookPopupMenu.enableItem(4, false);//delete contact
		}

индусская логика vs Российская. Чуть ни упал пацтол когда соптимизячил и сравнил со своим кодом:

boolean allowFlag = !(searchBox.isFocused() || contactListBox.isEmpty());
boolean portalFlag = !isSocialNWportal();
addressBookPopupMenu.enableItem(0, allowFlag);
addressBookPopupMenu.enableItem(1, !allowFlag);
addressBookPopupMenu.enableItem(2, portalFlag);//add contact
addressBookPopupMenu.enableItem(3, allowFlag && portalFlag);
addressBookPopupMenu.enableItem(4, allowFlag && portalFlag);
addressBookPopupMenu.enableItem(5, !searchBox.isFocused());

guest, 14 Января 2009

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

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

−56.7

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

private List<WorkOrderWrapper> copy(List<WorkOrderWrapper> src) {
		List<WorkOrderWrapper> ret = Collections.emptyList();

		if (src != null) {
			ret = new ArrayList<WorkOrderWrapper>(src.size());

			for (int i = src.size() - 1; i >= 0; i--) {
				ret.add(src.get(i));
			}
		}

		return ret;
	}

Мы легких путей не ищем!

guest, 29 Декабря 2008

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

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

−45.2

001
002
003
004
005
006
007
008
009
010
011
012
013
014
015
016
017
018
019
020
021
022
023
024
025
026
027
028
029
030
031
032
033
034
035
036
037
038
039
040
041
042
043
044
045
046
047
048
049
050
051
052
053
054
055
056
057
058
059
060
061
062
063
064
065
066
067
068
069
070
071
072
073
074
075
076
077
078
079
080
081
082
083
084
085
086
087
088
089
090
091
092
093
094
095
096
097
098
099
100

public void updateTable() {

        /**
         * Удаляем лишние строки таблицы, оставшиейся от предыдущего списка
         */
        while (viewingData.getResult().getRowDataItems().size() + 1 < dataTable.getRowCount()) {
            dataTable.removeRow(dataTable.getRowCount()-1);
        }

        /**
         * Отображаем заголовки таблицы
         */
        int titleColumnIndex = 0;
        String[] columnIds = new String[viewingData.getViewingColumns().size()];
        for (String viewingColumnId : viewingData.getViewingColumns().keySet()) {
            columnIds[titleColumnIndex] = viewingColumnId;
            ViewingColumnData viewingColumn = viewingData.getViewingColumns().get(viewingColumnId);

            HorizontalPanel titlePanel = new HorizontalPanel();
            titlePanel.setVerticalAlignment(HasVerticalAlignment.ALIGN_MIDDLE);

            Widget columnTitleButton;
            if (viewingColumn.isSortable()) {
                columnTitleButton = new Button(viewingColumn.getTitle());
                ((Button)columnTitleButton).addClickListener(new SortDataClickListener(viewingColumnId));
            } else {
                columnTitleButton = new Label(viewingColumn.getTitle());
            }

            titlePanel.add(columnTitleButton);
            columnTitleButton.setStyleName(TABLE_STYLE);


            if (viewingData.getOrderColumnId() != null && viewingData.getOrderColumnId().equals( viewingColumnId) ) {
                Image orderDirectionImage;
                if (viewingData.getOrderDesc()) {
                    orderDirectionImage = new Image(GWT.getModuleBaseURL() + UP_ARROW_IMG);
                } else {
                    orderDirectionImage = new Image(GWT.getModuleBaseURL() + DOWN_ARROW_IMG);
                }
                titlePanel.add(orderDirectionImage);
            } else {
            }

            dataTable.setWidget(0, titleColumnIndex, titlePanel);

            FlexTable.FlexCellFormatter titlePanelFormatter = dataTable.getFlexCellFormatter();
            titlePanelFormatter.setStyleName(0, titleColumnIndex, "tableTitle");
            if (viewingColumn.getWidth() > 0) {
                titlePanelFormatter.setWidth(0, titleColumnIndex, viewingColumn.getWidth() + "%");
            }
            titlePanelFormatter.setHorizontalAlignment(0, titleColumnIndex, HasHorizontalAlignment.ALIGN_CENTER);

            titleColumnIndex++;
        }
        /**
         * Удаляем лишние столбцы в заголовке
         */
        int columnToRemove = dataTable.getCellCount(0) - viewingData.getViewingColumns().size();
        dataTable.removeCells(0, viewingData.getViewingColumns().size(), columnToRemove);

        /**
         * Добавляем чекбокс
         */
        addTitleCheckbox();

        /**
         * Отображаем значения раскрытых групп
         */
        if (viewingData.getGroupingFilterItems() != null) {
            for (int groupingIndex = 0; groupingIndex < viewingData.getGroupingFilterItems().size(); groupingIndex++ ) {
                String groupingFilterValue = viewingData.getGroupingFilterItems().get(groupingIndex);
                Label valueLabel = new Label(groupingFilterValue);
                Image img = new Image(LEFT_ARROW_IMG);
                HorizontalPanel valuePanel = new HorizontalPanel();
                valuePanel.add(img);
                valuePanel.add(valueLabel);
                valuePanel.setWidth("100%");
                valuePanel.setHorizontalAlignment(HasHorizontalAlignment.ALIGN_LEFT);

                ClickListener hideGroupClickListener = new HideGroupClickListener(groupingIndex);
                valueLabel.addClickListener(hideGroupClickListener);
                img.addClickListener(hideGroupClickListener);

                String cellStyle;
                if (groupingIndex % 2 == 0) {
                    cellStyle = "tableRow";
                } else {
                    cellStyle = "tableRowOdd";
                }

                for (int column = 0; column < viewingData.getViewingColumns().size(); column++) {
                    String text = null;

                    /**
                     *
                     */
                    if (groupingIndex == viewingData.getGroupingFilterItems().size()-1 && viewingData.getFunctionData().containsKey(columnIds[column])) {
                        text = "(" + viewingData.getFunctionData().get(columnIds[column]) + ")";
                    } else {

краткость сестра таланта

guest, 26 Декабря 2008

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

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

−42.6

1
2
3

if ((sum != -1 && sum > 0) ) {
    AccountFactory.editAccount(number, sum, bank_num);
}

Скоро проект сдавать, мозги едут и новый функционал иногда идёт в продакшн в таком вот виде

guest, 26 Декабря 2008

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

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

−15.8

1

StringBuffer url = new StringBuffer( HTTPS + module + TAGS + tag + SLESH );

guest, 23 Декабря 2008

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

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

−48

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

private static final String PROMPT = " >";
private static final int BUFFER_SIZE = 255;
private static final int NEW_LINE_CHAR = 13;
private static final int CARRIAGE_RETURN_CHAR = 10;
...
System.out.print( prompt + PROMPT );
StringBuffer sb = new StringBuffer();
byte[] buf = new byte[BUFFER_SIZE];
int read = System.in.read( buf );
if ( read != 0 && buf[0] != CARRIAGE_RETURN_CHAR && buf[0] != NEW_LINE_CHAR )
{
    sb.append( new String( buf, 0, read ).trim() );
}
return sb.toString();

Вам нужно вывести приглашение и прочитать введенную строку? Что может быть проще?

guest, 20 Декабря 2008

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

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

−53.2

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

private void setTims( String project )
{
    int endIndex = project.indexOf( "-" );
    boolean isNumber = false;
    if ( endIndex != -1 )
    {
        try
        {
            Integer.parseInt( project.substring( 0, endIndex ) );
            isNumber = true;
        }
        catch ( NumberFormatException e )
        {
            //ignore
        }
    }
    if ( isNumber )
    {
        runtimeContext.put( RuntimeContext.TIMS_NUMBER, project.substring( 0, endIndex ) );
    }
    else
    {
        runtimeContext.put( RuntimeContext.TIMS_NUMBER, project );
    }
}

guest, 20 Декабря 2008

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

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

−47.1

1
2
3
4
5
6
7
8
9

public AlarmKey(AlarmKey key)
    {
        oc = "";
        id = "";
        source = null;
        oc = key.oc;
        id = key.id;
        source = key.source;
    }

DoubleCheck :)

guest, 18 Декабря 2008

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

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

−43

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

if(isRegKeyRequired) {
            keyItems = new XItem [4];
            valueItems = new XItem [4][];
            totalItems = 4;

            keyItems[0] = new XItem();
            keyItems[0].setLabel(TextUtils.getText(280),XItem.LABEL_CONSTRAINT_CENTER);
            values = getValueItems(countryBox_rp);
            totalItems += setValues(valueItems, values,0);
            keyItems[1] = new XItem();
            keyItems[1].setLabel(TextUtils.getText(281),XItem.LABEL_CONSTRAINT_CENTER);
            values = getValueItems(phoneNoBox_rp);
            totalItems += setValues(valueItems, values,1);
            keyItems[2] = new XItem();
            keyItems[2].setLabel(TextUtils.getText(282),XItem.LABEL_CONSTRAINT_CENTER);
            values = getValueItems(regKeyBox_rp);
            totalItems += setValues(valueItems, values,2);
            keyItems[3] = new XItem();
            keyItems[3].setLabel(TextUtils.getText(283),XItem.LABEL_CONSTRAINT_CENTER);
            values = getValueItems(emailBox_rp);
            totalItems += setValues(valueItems, values,3);
        }
        else {
            keyItems = new XItem [3];
            valueItems = new XItem [3][];
            totalItems = 3;

            keyItems[0] = new XItem();
            keyItems[0].setLabel(TextUtils.getText(280),XItem.LABEL_CONSTRAINT_CENTER);
            values = getValueItems(countryBox_rp);
            totalItems += setValues(valueItems, values,0);
            keyItems[1] = new XItem();
            keyItems[1].setLabel(TextUtils.getText(281),XItem.LABEL_CONSTRAINT_CENTER);
            values = getValueItems(phoneNoBox_rp);
            totalItems += setValues(valueItems, values,1);
            keyItems[2] = new XItem();
            keyItems[2].setLabel(TextUtils.getText(283),XItem.LABEL_CONSTRAINT_CENTER);
            values = getValueItems(emailBox_rp);
            totalItems += setValues(valueItems, values,2);
        }

очень трудолюбивые эти индусы. Такой код - это просто пипец особенно если учесть что приложение разрабатывается для J2ME

guest, 16 Декабря 2008

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

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

−37.2

1

isRegKeyRequired = distKey == null ? false : distKey.equals("yes") ? true : false;

Уматный ХавнАкоД коллеги из Индии. Ну просто няйсь. Смотрится куда зло*буче чем желаемый вариант (isRegKeyRequired = "yes".equals(distKey));

guest, 16 Декабря 2008

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

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

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

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

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

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

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

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

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

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

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

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