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

    +74.4

    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
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    private String getNullString(int kol){
            String nullstr="";
            for(int i=0;i<kol;i++){
                nullstr+="0";
            }
            return nullstr;
        }
    
    public String getNumber(int idd){
            String regid="";
            String idds=(idd+"");
            if(idds.length()<getNumberLenth()){
                regid+=getNullString(getNumberLenth()-idds.length());
                regid+=idds;
            }
            String regnum="";
            regnum+=getNumberPrefix()+regid;
            if(getNumberPeriod()!=NUM_PERIOD_NOPERIOD){
                GregorianCalendar c=new GregorianCalendar();
                switch(getNumberPeriod()){
                    case NUM_PERIOD_MONTH:
                        int m=c.get(Calendar.MONTH)+1;
                        regnum+="-"+m;
                        regnum+=".";
                        regnum+=c.get(Calendar.YEAR);
                        break;
                    case NUM_PERIOD_YEAR:
                        regnum+="-"+c.get(Calendar.YEAR);
                }
            }
            return regnum;
        }

    Ну незнал я про String.format (((..

    maxt, 01 Апреля 2010

    Комментарии (4)
  2. 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)
  3. 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)
  4. 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)
  5. 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)
  6. 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)
  7. Java / Говнокод #2819

    +71.2

    1. 1
    public class ObjectList<T> extends ErrorStatus

    Morbo, 18 Марта 2010

    Комментарии (0)
  8. 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)
  9. 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)
  10. 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)