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

    +78

    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, 03 Марта 2011

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

    • ava SadKo 03.03.2011 13:23 # +1

      Ну, не такой уж и говнокод, если передать правильный аргумент в Collections.sort.
      Ответить

      • ava gegMOPO4 03.03.2011 14:28 # +1

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

        • ava feodal 03.03.2011 15:36 # −1

          откройте глаза. что правильно?

          или Вы хотите сказать, что метод в результате вернет отсортированный список???
          Ответить

          • ava gegMOPO4 03.03.2011 16:11 # +1

            Э-э-э... так просто опечатка же. Опечатка -- не говнокод.
            Ответить

      • ava feodal 03.03.2011 15:37 # −1

        ну так в этом то и дело. сортирует одно, возвращает другое. хочет, чтобы входящий список оставался не измененным, а именно его и изменяет.

        настоящий говнокод
        Ответить

        • ava SadKo 03.03.2011 15:41 # +3

          Это не говнокод, это банальная ошибка.
          Говнокод, имхо, это когда без поллитры не разберёшься в том, что аффтар понаписал.
          Ответить

    • ava Анонимус 03.03.2011 14:36 # +1

      надесь автор реально хотел НЕ трогать старый лист
      тогда и правда -- не говнокод
      Ответить

      • ava feodal 03.03.2011 15:33 # −1

        старый лист трогать не хотел, но он его сортирует)) по моему настоящий гавнокодище))
        Ответить

        • ava tr00_gr1m_doomster 03.03.2011 15:47 # −1

          переданый список сортирует а возвращяет не отсортированый, судя по именам задумывалось не так, так что однозначно говнокод
          Ответить

    • ava Lure Of Chaos 03.03.2011 17:47 # 0

      отсортировать, но вернуть старый. Какая странная логика.
      Ответить

    • ava stas 06.03.2011 02:53 # −1

      Этот кусок кода выдран из контекста: http://community.livejournal.com/ru_java/1017065.html
      Не нужно придираться к опечатке.
      В реальном примере метод sort рисует дерево из списка, учитывая, что у объектов списка есть getParent и т.п.
      Ответить

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