Jawa / Говнокод #23240 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
Объявление метода в интерфейсе:

@NotNull
List<ServicePojo> getAvailableServices(boolean withoutOffer);

реализация:

@NotNull
@Override
public List<ServicePojo> getAvailableServices(boolean withOfferService) {
...
}

Запостил: moonson moonson, (Updated )

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

    • ava bormand # Ссылка на оригинал 0 ⇈

      Вот именно поэтому енумы в параметрах читаются гораздо лучше бульонов.
      Ответить

      • ava Dummy00001 # Ссылка на оригинал 0 ⇈

        энумы на все булы заводить весьма напряжно. в прошлом лично пробовал. через какой-то время начинаешь в них путаться.

        к слову, в каком то линте давеча видел птичку для проверки имён аргументов в наследуемых классах. раньше не знал зачем - но вот после этого, начинаю догадыватся.
        Ответить

        • ava roman-kashitsyn # Ссылка на оригинал 0 ⇈

          > через какой-то время начинаешь в них путаться

          Если енумы разных типов, компилятор не даст запутаться.
          Ответить

          • ava Dummy00001 # Ссылка на оригинал 0 ⇈

            "ни компайла без ошибки." большое счастье.

            к слову, еще одна альтернатипа это делать алиасы для true/false. недавний пример из жизни: unknown/known. енум не получается, потому что кодогенератор не умеет - но я больше вводил потому что там логика наоборот (тру == неизвестно, фолс == известно).
            Ответить

            • ava roman-kashitsyn # Ссылка на оригинал 0 ⇈

              > это делать алиасы для true/false
              Это никак не помогает в коде подобном тому, что в топике.
              А вот енумы помогают.
              public enum OfferStatus { WithOffer, WithoutOffer }

List<ServicePojo> getAvailableServices(OfferStatus offerStatus) {
    if (offerStatus == OfferStatus.WithOffer) // нужно постараться, чтобы ошибиться
}
              Ответить

            • ava bormand # Ссылка на оригинал 0 ⇈

              Лучше уж компайл с ошибкой, чем запуск с ошибкой.

              Просто столько раз видел и перепутанную местами пару бульонов и попутанные негатив/позитив... И сам так залетал...
              Ответить

    • ava inho # Ссылка на оригинал 0 ⇈

      Я правда не понимаю что не так. Параметр назван по другому, ну и что? Смысл то не противоположный.
      Ответить

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

Помни, guest, за тобой могут следить!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓