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) RSS

    • ava guest 30.03.2010 14:20 # −0.2

      Как я понимаю, говнокод в корявой реализации метода пузырька?
      Ну, конечно, удобнее бы было использовать Collections.sort и Comparator.
      И да, еще не очень красиво, что цикл идет по одной коллекции, а работа в цикле идет с другой коллекцией.
      Ответить

    • ava guest 01.04.2010 04:07 # −0.2

      Я так понимаю, что неиспользование регекспов тоже туда попадает.
      /[(/)]+/
      Это они вообще что имели в виду? Больше на псевдографику похоже, чем на регексп :) Или это была все-таки попытка вырезать слеши (если да, то неужелив Java их не нужно экранировать?)
      Ответить

    Добавить комментарий