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

    +68

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    int docTypeId1 = csResult.getDocType().getId();
int docTypeId2 = attachment.getDocument().getId();
if (docTypeId1 == docTypeId2) {		// здесь сравниваем типы этих документов
....
}

    Запостил: asmirnov, 08 Декабря 2014

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

    • ava asmirnov 09.12.2014 08:47 # 0

      Объясните мне плиз, где тут логика?? может я чего не понимаю.
      Ответить

    • ava bakagaijin 09.12.2014 10:56 # 0

      Единственное, к чему бы я доебался, это какого хера getId для документа возвращает его доктайпайди. Надо бы тогда уже getDocumentType().getId() или getDocTypeId(). Больше вроде ничего не пахнет. Ну ещё, да, можно доебаться к var1/var2 вместо осмысленных названий, но тут всё очевидно.
      Ответить

      • ava asmirnov 09.12.2014 11:01 # 0

        а разве не должно быть csResult.getDocType().getId().equals(att achment.getDocument().getId()) или вообще:
        csResult.getDocType().equals(attachment. getDocument()) ???
        Ответить

        • ava bakagaijin 09.12.2014 11:05 # 0

          int же. Я не настоящий джавист, но мне казалось, что equals - это для строк?
          Второй вариант - надо уточнить, что конкретно хотим узнать. Лежит ли в csResult и attachment ссылка на один и тот же объект (?), или то, что у двух потенциально разных документов одинаковый доктайп.
          Ответить

          • ava asmirnov 09.12.2014 11:10 # 0

            equals для объектов.
            Ответить

            • ava bakagaijin 09.12.2014 11:16 # 0

              Ну, я в контексте сравнения каких-то entity id (которые от перепоя архтиектора могут быть и строками, но в данном случае - инты).
              Ответить

              • ava asmirnov 09.12.2014 11:19 # 0

                в данном случае не int а именно объектный Integer. Стоило сразу об этом сказать.
                Ответить

    • ava guest 09.12.2014 13:36 # 0

      Да уж, вот это говнокод ... кошмар.
      Ответить

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