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

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

+74

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

private static java.util.Hashtable<String, Session> sessions =
    new java.util.Hashtable<String, Session>();
...
public static SessionOperations[] getSessions() {
    int cnt = sessions.size();
    SessionOperations[] res = new SessionOperations[cnt];
    java.util.Enumeration<String> e = sessions.keys();
    for( int i = 0; i < res.length; i++ ) {
      String sid = e.nextElement();
      res[i] = sessions.get( sid ).getOperations( sid );
    }
    return res;
}

Удаление из sessions между получением размера и набора ключей приводит к весьма печальным последствиям...

P.S. В результате в течение трех суток в production не удалялись оставленные сессии (без Logout), sessions отожрал 600М и всё свалилось с OOM.

Skipy, 23 Июня 2011

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

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

+82

1
2
3

/*
        todo it seems that code below will be right, when client will be changed
*/

Вот такой емкий комментарий перед 200 закоментированными строками кода

galak, 20 Июня 2011

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

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

+76

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 class ValueList< T > implements Cloneable
{
    public ValueList() { }

    public void addValue( final T value ) {
        list.add( value );
    }

    @Override
    @SuppressWarnings( { "unchecked", "unchecked" } )
    public Object clone() throws CloneNotSupportedException {
        super.clone();
        ValueList< T > object = new ValueList< T >();
        List< T > objectList = object.getList();
        objectList = ( List< T > )list.clone();
        for( int index = 0; index < list.size() ; ++index ) {
            objectList.set( index, ( Object )list.get( index ).clone() );
        }
        return object;
    }

    public List< T > getList() {
        return list;
    }

    private ArrayList< T > list = new ArrayList< T >();
}

Человек из соседнего отдела порадовал...

SadKo, 20 Июня 2011

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

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

+77

1
2
3
4

Ingeter[] i = new Integer[1];
for (i = 0; i<n; i++) {
//что-то происходит
}

здесь все по-серьезному

skim1776, 18 Июня 2011

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

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

+87

1

Boolean.valueOf("1".equals(object[1].toString())?"true":"false"))

madhead, 18 Июня 2011

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

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

+83

1
2
3
4
5
6

public void fillFilialNames(ArrayList<String> filialNames) {
		for (String filialDTO : filialNames) {
			if (!filialNames.isEmpty())
			filialNames.add(filialDTO);
		}
	}

Бесконечный цикл :)

stepanPlus7, 17 Июня 2011

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

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

+146

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

File tarFile = new File(UUID.randomUUID() + ".tar");
    tarFile.createNewFile();
    FileOutputStream tarFileOutputStream = new FileOutputStream(tarFile.getAbsolutePath());
    TarArchiveOutputStream   archiveOutputStream = new TarArchiveOutputStream(tarFileOutputStream);
     
    String entryName = "пример.txt";
     
    File fileToPack = new File("C:\\пример.txt");
    InputStream in = new FileInputStream(fileToPack);
     
    //создаем запись в архиве с русским названием
    TarArchiveEntry entry = new TarArchiveEntry(entryName);
    entry.setSize(in.available());
    archiveOutputStream.putArchiveEntry(entry);
     
    while (in.available() > 10 * 1024 * 1024) {
                byte b[] = new byte[10 * 1024 * 1024];
                in.read(b);
                archiveOutputStream.write(b, 0, b.length);
        }
    byte b[] = new byte[in.available()];
    in.read(b);
    archiveOutputStream.write(b, 0, b.length);
    in.close();
     
    archiveOutputStream.flush();
    archiveOutputStream.closeArchiveEntry();
    archiveOutputStream.flush();
    archiveOutputStream.close();

найдено в сети.

Lure Of Chaos, 16 Июня 2011

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

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

+147

1
2
3

if(entry.getName().equals("back.jpg"));{

}

просто смешная (но с последствиями) опечатка

Lure Of Chaos, 16 Июня 2011

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

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

+78

01
02
03
04
05
06
07
08
09
10
11

if (searchFB.getOrderNo()!=null && searchFB.getOrderNo().length()==6){
    int drawingStatus = searchFB.getDrawingStatus();
    int criticalStatus = searchFB.getCriticalStatus();
    int blockingStatus = searchFB.getBlockingStatus();
    int suspensionStatus = searchFB.getSuspensionStatus();

    searchFB.setDrawingStatus(drawingStatus);
    searchFB.setCriticalStatus(criticalStatus);
    searchFB.setBlockingStatus(blockingStatus);
    searchFB.setSuspensionStatus(suspensionStatus);
}

И это мне поддерживать...

TC1, 16 Июня 2011

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

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