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

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
static List sort(List unsorted, Comparator comparator) { 
    List sorted = new ArrayList(unsorted); 
    Collections.sort(unsorted, comparator);
    return sorted;
}

Запостил: feodal feodal, (Updated )

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

  • Ну, не такой уж и говнокод, если передать правильный аргумент в Collections.sort.
    Ответить
    • Да всё вроде правильно. Только генерализировать бы... Но это может быть старый код.
      Ответить
      • откройте глаза. что правильно?

        или Вы хотите сказать, что метод в результате вернет отсортированный список???
        Ответить
        • Э-э-э... так просто опечатка же. Опечатка -- не говнокод.
          Ответить
    • ну так в этом то и дело. сортирует одно, возвращает другое. хочет, чтобы входящий список оставался не измененным, а именно его и изменяет.

      настоящий говнокод
      Ответить
      • Это не говнокод, это банальная ошибка.
        Говнокод, имхо, это когда без поллитры не разберёшься в том, что аффтар понаписал.
        Ответить
  • надесь автор реально хотел НЕ трогать старый лист
    тогда и правда -- не говнокод
    Ответить
    • старый лист трогать не хотел, но он его сортирует)) по моему настоящий гавнокодище))
      Ответить
      • переданый список сортирует а возвращяет не отсортированый, судя по именам задумывалось не так, так что однозначно говнокод
        Ответить
  • Этот кусок кода выдран из контекста: http://community.livejournal.com/ru_java/1017065.html
    Не нужно придираться к опечатке.
    В реальном примере метод sort рисует дерево из списка, учитывая, что у объектов списка есть getParent и т.п.
    Ответить

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

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


    8