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

    +2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    public static int booleanToInt(Boolean value) {

        int res;

        if (value == null) {

            res = 0;

        } else {
            if (value) {
                res = 1;
            } else {
                res = 0;
            }
        }

        return res;
    }

    Добро пожаловать к нам в индийскую школу программирования!

    Запостил: nekkiy, 28 Июня 2021

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

    • ava Mithun_Chakraborty 28.06.2021 18:45 # +4

      Всегда так пишу.
      Ответить

    • ava ASD_77 29.06.2021 16:31 # +4

      смотри на это как на чёрный ящик - если работает заначит все ОК
      Ответить

      • ava Fike 29.06.2021 17:04 # +2

        ...а если выебываться будешь, то он за привилегии спросит
        Ответить

      • ava gologub 29.06.2021 19:56 # 0

        два бака – два ширака!

        https://politikus.ru/uploads/posts/2020-11/1606692352_1.jpg
        Ответить

    • ava KoWe4Ka_l7porpaMMep 01.07.2021 10:38 # 0

      В чём здесь говнокод? В том, что на джаве Boolean – это объект, и жабисты СОСНУЛИ?
      Ответить

      • ava MAPTbIwKA 01.07.2021 15:04 # 0

        В том, что 
        int res = value != null && value ? 1 : 0;

        Ваш КО
        Ответить

        • ava KoWe4Ka_l7porpaMMep 01.07.2021 15:16 # 0

          Тогда уж

          return value != null && value ? 1 : 0;


          А вообще это плохой год. Трудночитаемый, с магическими числами, не интуитивно понятный.
          Ответить

          • ava bormand 01.07.2021 15:19 # +1

            Что-то в духе return true.equals(value) ? 1 : 0 не прокатит?

            Или return Boolean(true).equal(value) ? 1 : 0, если само не забоксится...
            Ответить

          • ava MAPTbIwKA 01.07.2021 15:22 # 0

            Единица может быть магическом числом, а может и не быть.
            Магические числа это те, которые не входят в область знаний читающего кода.

            Например "2^8" (имеется ввиду степень конечно) звучит как магическое число, но для байтоёба она совсем не магическое, и в лоу левел коде наверное нет смысла выносить его в константу BYTE_CARDINALITY

            А вот для кода магазина на PHP его нужно, разумеется, вынести
            Ответить

            • ava bormand 01.07.2021 15:24 # +1

              CHAR_BITS уже нагородили, так что почему бы и не поюзать.
              Ответить

              • ava MAPTbIwKA 01.07.2021 15:41 # +2

                хотел доебаца, что там кол-во бит в чаре и теоретически может быть не 8, но уже прописали, что это кол-во бит в байте
                Ответить

                • ava Soul_re@ver 01.07.2021 16:34 # 0

                  ТАк поддержку невосьмибитных байтов ещё не дропнули вроде.
                  Ответить

          • ava Fike 01.07.2021 15:26 # +3

            value ?? true

            именно поэтому я за сишарп*!

            *в шарпе и вовсе есть нормальная структура nullable, которую сахар еще и позволяет разворачивать не руками, а (в случае с bool?) использовать напрямую в control structures
            Ответить

            • ava MAPTbIwKA 01.07.2021 16:28 # 0

              Причем она не reference, а как std::any, что особенно хорошо. В жобе так нельзя

              Джава вообще отстает от C# на бесконечное число лет, что и говорить
              Ответить

    • ava avk17 14.08.2021 12:50 # 0

      А мне нравится, совершенно не обязательно помнить как там трактуется if (null), всегда надо писать так, что бы у читающего не возникало лишних вопросов. А что если null это признак ошибки и мы тут это явно обрабатывает? Код устойчивый к ошибкам.
      Ответить

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