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

    +150

    1. 1
    2. 2
    3. 3
    if (id != object.id) return false;
    
    return true;

    IntelliJ IDEA 8 :) equals по интовому полю.
    Хорошо хоть сама тут же предлагает упростить.. :)

    Запостил: guest, 24 Апреля 2009

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

    • Ja!:
      и что тут гавняного?
      просто кусок кода, непонятно откуда вход в функцию.
      Ответить
    • Kitsch:
      [quote=Ja!]и что тут гавняного?[/quote]

      Э... я не большой знаток явы, но чем не угодило:

      return (id == object.id)


      ?
      Ответить
    • Там Code Snippet видимо так генерит.

      что-то типа

      if (field1 != object.field1) return false;
      if (field2 != object.field2) return false;
      if (field3 != object.field3) return false;

      и т.д.

      return true;

      и в случае одного поля получилось то, что получилось.
      Ответить
    • почему бы ему сразу не генерить
      return
       (field1 == object.field1) &&
       (field2 == object.field2) &&
       ...

      ?
      Ответить
    • Капитан Очевидность:
      Данный кусок говнокода, говнокодом не является. За то говнокодом является тот говнокод который генерирует данный говнокод.
      Ответить

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