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

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 / Говнокод #2692

+81.1

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

public int random() {
        long info =  (long) (System.currentTimeMillis() + Runtime.getRuntime().freeMemory() + System.nanoTime());
        long info2 = (long) (System.currentTimeMillis() + Runtime.getRuntime().hashCode() + System.nanoTime());

        this.rnd1.setSeed(info);
        this.rnd2.setSeed(info2);

        int a = this.rnd1.nextInt();
        int b = this.rnd2.nextInt();

        return (int) (a <<= b);
}

функция для получения настоящего рандомного числа в какомто студенческом говнокоде

danilissimus, 28 Февраля 2010

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

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

+72.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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57

public class CountUnique {
    //here objects will be stored
    private Object[] variants;
    //temporaly array to store copying variants
    private Object[] temparr;

    private int total = 0;

    public CountUnique() {}

    public boolean test(Object obj) {
        total++;
        boolean hasSame = false;
        if(variants == null) {
            variants = new Object[1];
            variants[0] = obj;
            hasSame = false;
        } else {
            for(int i = 0; i < variants.length; i++) {
                if(variants[i] == null) {} else {
                    if(variants[i].equals(obj)) {
                        hasSame = true;
                        break;
                    }
                }
            }

            if(hasSame == false) {
                temparr = variants;
                variants = new Object[temparr.length+1];
                for(int i = 0; i < temparr.length; i++) {
                    variants[i] = temparr[i];
                }

                variants[temparr.length] = obj;
                temparr = null;
            }
        }

        return hasSame;
    }

    public int unique() {
        if(variants == null) {
            return 0;
        } else return variants.length;
    }

    public int total() {
        return total;
    }

    public void free() {
        variants = null;
        temparr = null;
    }
}

некий класс для подсчета уникальных обьектов.
особенно умиляет функция test()

danilissimus, 25 Февраля 2010

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

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