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

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

+144.6

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

public class Main {
    public static void main(String[] args) {
    	f0();//this is call of f0() from line 4
    	f0();//this is call of f0() from line 4
    }

    public static void f0(){
    	System.out.println("hi, I am f0");
    	Exception e = new Exception();
    	if(e.getStackTrace()[1].getLineNumber()==3){
    		f1();
    	} else if(e.getStackTrace()[1].getLineNumber()==4){
			f2();
    	} 
    }
    
	public static void f1(){
		System.out.println("hi, I am f1");
	}

	public static void f2(){
		System.out.println("hi, I am f2");
	}
}

вот кусочек, который выполняется по разному в зависимости от того с какой строчки в исходном коде он вызван. Сам сочинил.

pcherkas, 17 Марта 2010

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

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

+81.6

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

public void calculate(){
        jTextField1.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner1.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner13.getValue()))));
        jTextField2.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner2.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner14.getValue()))));
        jTextField3.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner3.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner15.getValue()))));
        jTextField4.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner4.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner16.getValue()))));
        jTextField5.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner5.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner17.getValue()))));
        jTextField6.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner6.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner18.getValue()))));
        jTextField7.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner7.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner19.getValue()))));
        jTextField8.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner8.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner20.getValue()))));
        jTextField9.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner9.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner21.getValue()))));
        jTextField10.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner10.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner22.getValue()))));
        jTextField11.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner11.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner23.getValue()))));
        jTextField12.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner12.getValue()))
                *Integer.valueOf(String.valueOf(jSpinner24.getValue()))));
    }

x_X

ChakuZa, 14 Марта 2010

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

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

+79.1

1
2
3
4
5
6
7
8

if (at == null) {
	try {
	throw new NullPointerException();
	}catch (Exception e) {
	e.printStackTrace();
	throw new NullPointerException();
	}
}

проверка корректности полученного аргумента в функции.
хорошо придумано, что бы не вызывать
throw new IllegalArgumentException();

maxt, 13 Марта 2010

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

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

+72.6

1
2
3
4

Object[] o = new Object[16];
for (int i = 0; i < 20; i++) {
    o[i] = "";
}

Korwin, 11 Марта 2010

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

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

+71

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

String ID = httpServletRequest.getParameter("id");
        ChannelData data = new ChannelData();

        int pointPos = ID.indexOf(';');

        while(pointPos >= 0) {
            String CurrentID = ID.substring(0, pointPos);
            data.setId(new Integer(CurrentID));

           ChannelData channel = (ChannelData) channelDao.Get(Integer.valueOf(CurrentID));

            channelService.delete(data);
            ID = ID.substring(pointPos + 1, ID.length());
            pointPos = ID.indexOf(';');
        }
        if(ID.length() > 0) {
            ChannelData channel = (ChannelData) channelDao.Get(Integer.valueOf(ID));
            data.setId(new Integer(ID));
            channelService.delete(data);
        }

prop, 11 Марта 2010

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

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

+73.4

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

public class SaveBlankElementException extends RuntimeException{
    public SaveBlankElementException(Throwable cause){
        super(cause);
    }
    public SaveBlankElementException(String message,Throwable cause){
        super(message,cause);
    }
    public SaveBlankElementException(String message){
        super(message);
    }
    public SaveBlankElementException(){

    }
    @Override
    public String getMessage(){
        return "Попытка сохранения пустого элемента\nНе вызван prepareCreate()";
    }
  
}

Зачем, спрашивается было создавать этот класс, когда можно было бы
вызвать исключение
throw new UnsupportedOperationException("Попытка сохранения пустого элемента\nНе вызван prepareCreate()");

maxt, 10 Марта 2010

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

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

+73.8

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

public class Date extends java.util.Date{
    @Override
    public String toString(){
        return toLocaleString();
    }
    public Date(String s){
        super(s);
    }
    public Date(int year,int month,int date,int hrs,int min,int sec){
        super(year,month,date,hrs,min,sec);
    }
    public Date(int year,int month,int date,int hrs,int min){
        super(year,month,date,hrs,min);
    }
    public Date(int year,int month,int date){
        super(year,month,date);
    }
    public Date(long date){
        super(date);
    }
    public Date(){
    }
    public java.sql.Date getSQLDate(){
        return new java.sql.Date(getTime());
    }
}

из за переопределения и еще какого, а главное как:
@Override
public String toString(){
return toLocaleString();
}
писать целый класс, это не каждый сможет.

maxt, 09 Марта 2010

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

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

+83.5

1
2
3
4

public void connect(boolean numberOfTransactions)
{
...
}

Че туда передавать???

Nike83, 04 Марта 2010

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

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

+75.4

1
2
3
4

if (!Float.valueOf("0.0").equals(price.getActualPrice()))
{
	return true;
}

Сравнение чисел (float) нездоровым способом.

asolntsev, 04 Марта 2010

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

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

+73.8

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

List<SomeObj> list = getTodayObjects();
.....
int i = list.size() - 1;
for (; i >= 0; i--) {
    SomeObj oldState = list.get(i);
    if (currentObj.getTimestamp() - oldState.getTimestamp() >= 5 * MINUTE) {
        value5MinAgo = oldState.getValue();
        break;
    }
}
for (; i >= 0; i--) {
    SomeObj oldState = list.get(i);
    if (currentObj.getTimestamp() - oldState.getTimestamp() >= 15 * MINUTE) {
        value15MinAgo = oldState.getValue();
        break;
    }
}
for (; i >= 0; i--) {
    SomeObj oldState = list.get(i);
    if (currentObj.getTimestamp() - oldState.getTimestamp() >= 30 * MINUTE) {
        value30MinAgo = oldState.getValue();
        break;
    }
}
for (; i >= 0; i--) {
    SomeObj oldState = list.get(i);
    if (currentObj.getTimestamp() - oldState.getTimestamp() >= 60 * MINUTE) {
        value60MinAgo = oldState.getValue();
        break;
    }
}
.....

фанданго, не иначе

Anonimous, 04 Марта 2010

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

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

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

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

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

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

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

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

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

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

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

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