Jawa / Говнокод #2896 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 des-1008d, (Updated )

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

  • ava guest # Ссылка на оригинал 0

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

  • ava guest # Ссылка на оригинал 0

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

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

Переведи на "PHP", guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓