1. Java / Говнокод #2896

    +74

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    lstCmsObject = OpenCms.getOrgUnitManager().getGroups(obj, "/", true);
                            
                            for (int i = 0; i < lstCmsObject.size(); i++)
                            {
                            	lstUnits.add(new Units(lstCmsObject.get(i).getDisplayName(obj, Locale.US).replaceAll("Users from ","").replaceAll("[a-zA-Z]*","").replaceAll("[(/)]+",""),lstCmsObject.get(i).getOuFqn()));
                            }
                            
                            for (int i = 0; i < lstCmsObject.size(); i++) {
		                for (int j = 0; j < lstCmsObject.size(); j++) {
		                    if (lstUnits.get(i).GetName().compareTo(lstUnits.get(j).GetName())<0) {
		                        Collections.swap(lstUnits, i, j);
		                    }
		                }
		            }

    des-1008d, 30 Марта 2010

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

  2. Java / Говнокод #2890

    +70.8

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    if (tp != null) {
		DocElement d = (DocElement)tp.getLastPathComponent();
		if (d.getElementType() == Constants.TYPE_FOLDER) {
			context.setCursor(DragSource.DefaultMoveDrop);
		if (d.getElementType()==Constants.TYPE_FOLDER) {
			tree.setSelectionPath(tp);
			tree.expandPath(tp);
		}
		}else {
			context.setCursor(DragSource.DefaultMoveNoDrop);
		}
	}

    моё, так сказать носом ткнули)
    условие if (d.getElementType()==Constants.TYPE_FOLD ER)

    maxt, 29 Марта 2010

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

  3. Java / Говнокод #2870

    +76.8

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    public void isChangenabledExc(){
        if(isNew){
            return;
        }
        if(true)return;//TODO
        if(!isChangenabled()){
            throw new ChangeEIsNotLockedException();
        }
    }

    В этой функции находится единственный коментарий из всего файла.
    Видимо в планах эта функция должна была делать нечто хорошее, но
    планы так и не были осуществелны.
    В результате получилась функция, которая ничего не делает, а только проверяет какие то условия.
    поиском обнаружил 5 вызовов этой функции.

    maxt, 26 Марта 2010

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

  4. Java / Говнокод #2866

    +87.2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    else {
                                             if(codeName == SummaryConstants.TYPE_COST) {
                                                return new Cost(rtOBJ);
                                             }
                                             else {
                                                if(codeName == SummaryConstants.TYPE_EQUIPMENT) {
                                                   return new Equipment(rtOBJ);
                                                }
                                             }
                                          }
                                       }
                                    }
                                 }
                              }
                           }
                        }
                     }
                  }
               }
            }
         }

    не стал утомлять еще и километром вложенных if ов..
    а говнокод в том, что здесь можно было использовать switch

    maxt, 25 Марта 2010

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

  5. Java / Говнокод #2845

    +82.8

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    for (int i = 0; i < list.size(); i++) {
        SpecialItem item = list.get(i);
        if (item != null) {
          item = null;
        }
}
list.clear();

    Неиндийский способ очистить java.util.List от вместимого.

    xrobak, 23 Марта 2010

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

  6. Java / Говнокод #2819

    +71.2

    1. 1
    public class ObjectList<T> extends ErrorStatus

    Morbo, 18 Марта 2010

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

  7. Java / Говнокод #2817

    +75

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    private static byte[] getByteArrayFromInt(int number) {
	ByteArrayOutputStream out = new ByteArrayOutputStream();
	DataOutputStream o = new DataOutputStream(out);
	try {
		o.writeInt(number);
	} catch (IOException e) {
		e.printStackTrace();
	}
	return out.toByteArray();
}

    Интересный способ переводить Int в массив байт )

    Logarithm, 18 Марта 2010

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

  8. Java / Говнокод #2815

    +81.4

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if(condition) {
    function(false);
} else {
    function(true);
}

    danilissimus, 17 Марта 2010

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

  9. Java / Говнокод #2814

    +86.4

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    //#ifdef Slowpoke
    private static final int MILLIS = 1100;
    //#else
    private static final int MILLIS = 1000;
    //#endif

    Сколько миллисекунд в секунде?

    thePooh, 17 Марта 2010

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

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

    +144.6

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 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)