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

    0

    1. 1
    2. 2
    if (Boolean.TRUE.equals(planItem.isIgnoredPurchase()))
        {

    Запостил: vistefan, 21 Марта 2020

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

    • ava 3.14159265 22.03.2020 00:24 # +2

      Нормальный, идиоматичный код.
      Ответить

      • ava gost 22.03.2020 00:26 # 0

        Сначала удивился комментарию, потом посмотрел на раздел кода. Действительно, правильно и идиоматично.[‍/color]
        Ответить

        • ava 3.14159265 22.03.2020 00:26 # +3

          Если isIgnoredPurchase возвращает Boolean, то в нём возможен null.

          И обычный if (planItem.isIgnoredPurchase()) кинет NPE. Именно поэтому Boolean.TRUE стоит спереди.

          Edit: ну всяко же лучше Optional<Boolean>
          Ответить

          • ava gost 22.03.2020 00:29 # +2

            Погуглил. Действительно, ТРОИЧНАЯ ЛОГИКА!

            Пиздец, конечно.
            Ответить

          • ava gostinho 22.03.2020 00:34 # 0

            За возврат Boolean, а не boolean нужно же бить по рукам?
            Ответить

            • ava 3.14159265 22.03.2020 00:42 # +1

              Нужно. И тем не менее я повидал ОЧЕНЬ много такого кода.

              Бывают всякие dto, маппинги на xml/json где флаг например необязательно будет.
              Можно конечно и там не писать Boolean. Но это же жавашки. Они просто ебошат код, не особо задумываясь.
              Ответить

          • ava Janycz 22.03.2020 10:08 # 0

            Поэтому я за C++ (ну и немного за C#).
            Вот в нашем C# все гораздо лучше чем в этой питушиной Java.
            System.Nullable<System.Boolean>, или более коротко bool? из C# лучше этой джавовской питушни.
            Правда если planItem.isIgnoredPurchase() это bool?, то придется писать planItem.isIgnoredPurchase() == true или planItem.isIgnoredPurchase() ?? true.
            Да, и System.Boolean у нас это синоним bool.
            Ответить

            • ava eukaryote 22.03.2020 12:52 # 0

              Наверное, всё-таки не «planItem.isIgnoredPurchase() ?? true», а «planItem.isIgnoredPurchase() ?? false».
              Можно ещё так: 
              planItem.isIgnoredPurchase().GetValueOrDefault()
              Ответить

    • ava phpBidlokoder2 22.03.2020 00:40 # 0

      В жабо помойке null это Boolean ? Именно поэтому я за PHP
      Ответить

      • ava vistefan 22.03.2020 00:42 # 0

        null — это null.
        Boolean — это обёрточный типа над boolean, который нужен для философии everything is an object.
        Ответить

        • ava gost 22.03.2020 00:46 # 0

          > который нужен для философии everything is an object.
          Именно поэтому я за «Python».
          Ответить

          • ava KOPOHABuPYC 25.03.2020 12:52 # 0

            Не нужно быть философом-пацифистом, чтобы тихо скончаться от пневмонии, где-нибудь в клинике в Мытищах. Твоими пёрлами будет какое-то время подтираться медперсонал, так как всю бумагу уже давно украли. Но скоро и они иссяктнут.
            Ответить

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