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

    +70

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    protected SocketChannel getSockById(int id)
    {
              Iterator<SocketChannel> i = clientSock.iterator();
              while(i.hasNext())
              {
              	SocketChannel s = i.next();
              	if(s.hashCode() == id)
               		return s;
         
              }
              return null;
    }

    Инетесно, о чём думал автор в момент создания этого шедевра

    Запостил: roman-kashitsyn, 30 Октября 2012

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

    • > Инетесно
      Интересно
      selffix
      Ответить
    • Хеш-мапа!
      Ответить
    • > Инетесно, о чём думал автор в момент создания этого шедевра

      Он думал, что использует хеш.

      Помню, один новичок, впоследствии оказавшийся хелп-вампиром, делал поиск в списке. Линейный поиск. Потом спросил, как правильно. Я сказал, что нужен хеш. Посоветовал array [word] of array of Pair(key,value). Он сделал array of Pair и стал линейно искать нужный хеш.
      Ответить
      • > Я сказал, что нужен хеш. Посоветовал array [word] of array of Pair(key,value).
        На паскале хэш всегда реализуют самостоятельно и с нуля?
        Ответить
    • Hash code is not unique. Don't use for key.
      Ответить
      • https://i.chzbgr.com/completestore/12/8/30/wyAMmxbICkeHHS0T7M5bQw2.jpg
        Ответить
        • И куда Вы предлагаете мне это засунуть?
          Ответить
          • Господа, ни слова о ████ ██ ███████ ███
            Вы о чем нибудь другом думаете помимо мыслей о "засунуть"?
            Ответить
            • Конечно. CD можно использовать как:
              - Зеркало
              - Подставку для чашки
              - Элемент украшения
              - Деталь шаманского бубна
              ...
              А вы - только засунуть. У меня уж и совать-то некуда...
              Ответить
              • Недавно вот видел в тему:
                https://twitter.com/pepelsbey/status/262628642750885888/photo/1/large
                Ответить
    • Как бы while(i.hasNext()) уже выдает либо полного дурака, либо абсолютного новичка...
      Ответить
    • я бы создал мапу id -> socket
      а вообще, вайл по итератору вместо кошерного форыча - низкоуровневое извращение
      Ответить

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