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

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

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