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

    +69

    1. 1
    foos.removeAll(Collections.singleton(null));

    foos.remove(null)?не, не наш метод.

    Запостил: Lure Of Chaos, 02 Августа 2013

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

    • ava someone 02.08.2013 23:25 # +2

      Вспомнился анекдот про математика и чайник.
      Ответить

      • ava Lure Of Chaos 03.08.2013 04:37 # −1

        как сварить часы?
        Ответить

        • ava inkanus-gray 03.08.2013 16:48 # +6

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

          • ava 3.14159265 05.08.2013 15:15 # 0

            Вероятно он счел зашкваром сделать GOTO step_2. Потому использовал рекурсию.
            Ответить

        • ava 1024-- 03.08.2013 18:48 # +5

          Что это за анекдот, если не секрет?
          (Если погуглить "как сварить часы" в кавычках, находится только эта страница. Видимо, здесь знают толк в варке часов.)
          Ответить

          • ava LispGovno 03.08.2013 20:59 # +1

            У меня есть дедовские? напольные часы из метала. Однажды они упали с чердака. Шев разошелся и пришлось варить. Идут до сих пор.
            Ответить

            • ava anonimb84a2f6fd141 03.08.2013 21:10 # 0

              У меня есть настенные часы на батарейках с китайским механизмом на пластиковых шестеренках. Работают уже несколько лет, батарейки хватает на год-полтора. В самом простом варианте стоят евро 5.
              Ответить

          • ava Lure Of Chaos 03.08.2013 21:29 # 0

            это не анекдот, это байка про рассеянного ученого
            Ответить

          • ava bot 03.08.2013 22:58 # 0

            Говнокод + Кулинария = ?
            Ответить

            • ava 1024-- 03.08.2013 23:52 # +1

              Говнокод + Кулинария = Сущий ад
              Если воспринимать рецепты блюд как скрипты для поваров, то, наговнокодив в них, можно неплохо навредить.
              Взять хотя бы тот случай, когда забыли написать, что в кастрюлю надо налить воды перед варкой сгущёнки, и пришлось изымать из продажи все экземпляры книги с рецептами.
              Ответить

          • ava eth0 04.08.2013 19:34 # +5

            Закроют теперь говнокод за пропаганду варки веществ.
            Ответить

    • ava JetBlack 09.08.2013 14:05 # 0

      Где говнокод? Автор, как по книжке, убирает все null'ы из коллекции. Вместо одного.
      Ответить

      • ava bormand 09.08.2013 19:03 # 0

        Забавно, но почему-то у коллекции нет метода removeAll с одним элементом, а не с коллекцией. Для удобства могли бы и добавить, чтобы Collections.singleton не юзать...
        Ответить

        • ava kegdan 09.08.2013 20:38 # 0

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

          А еще какая нибудь польза от Collections.singleton() есть, или только дыры затыкать?

          Мне, как человеку непосвященному, кажется что Collections.singleton(X) это не очень нужная обвертка для X
          Ответить

          • ava JetBlack 11.08.2013 13:00 # 0

            Обычный static factory method, который возвращает лист с одним элементом. Например, есть метод, в котором в качестве аргумента принимается List. Мы хотим передать туда наш один объект. Collections.singleton будет кстати.
            Ответить

            • ava kegdan 13.08.2013 01:56 # +1

              Ну это и называется затыкать дыры, нет?
              Ответить

      • ava 3.14159265 12.08.2013 13:51 # 0

        >Где говнокод?
        Каждый раз создается лишний промежуточный объект с null. Константа - лучше.
        public static <T> Set<T> singleton(T o) {
	return new SingletonSet<T>(o);
    }

        2 bormand
        >Забавно, но почему-то у коллекции нет метода removeAll с одним элементом, а не с коллекцией
        >removeAll с одним элементом
        Узко мыслите. Нужен с массивом/вараргом.
        Ответить

        • ava 3.14159265 12.08.2013 14:03 # +1

          >Нужен с массивом/вараргом.
          Хотя, возможно, я чутка погорячился с универсальностью.
          Проблема в том если делать качественное удаление по массиву - надо проверять длину и если больше N, то всё-равно создавать Set для пущей эффективности. Чтобы отсечь возможные дубликаты, и не получить полный скан O(len(arr)*len(coll)).
          Для одного элемента - разумно. А больше, то сам сделай какую считаешь нужным коллекцию с быстрым contains() и удоляй.
          Ответить

        • ava JetBlack 12.08.2013 14:26 # 0

          Все-же называть говнокодом создание вспомогательного объекта - слишком громко.
          Ответить

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