1. PHP / Говнокод #15353

    +164

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    function boolConvert($value){
        if(strtolower($value)=='true')
            return 1;
        if(strtolower($value)=='false')
            return 0;
        return $value;
    }

    Запостил: GoodTalkBot, 05 Марта 2014

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

    • ava GoodTalkBot 05.03.2014 00:37 # +2

      тест на программиста
      Ответить

    • ava WGH 05.03.2014 02:22 # +3

      Ну, от преобразования типов в PHP можно ожидать вообще чего угодно, так что преобразование строки в бул лучше вручную закодировать. Вот return $value смущает...
      Ответить

      • ava 3Doomer 05.03.2014 06:54 # +3

        Ну, если ты не буль ему подсунешь - вернёт, что дал.
        Ответить

        • ava Lokich 05.03.2014 13:27 # 0

          я бы эксепшн, или null вернул бы
          Ответить

        • ava GoodTalkBot 05.03.2014 17:13 # −1

          метод используется при переборе элементов массива, и нужно чтобы строки true/false преобразовались в 1/0 всё другое оставить без изменений. так что тут бросать исключения нет смысла, так же как и возвращать null :)
          Ответить

      • ava guest 05.03.2014 20:50 # +1

        Ёба, а нормального преобразования в бул нету???
        Ответить

        • ava bormand 05.03.2014 20:56 # +2

          Есть (bool), но оно во-первых нинужно, т.к. условия и операторы и без явного каста в бул работают точно так же, а во-вторых 'false' будет считать true (т.к. непустая строка это true).
          Ответить

          • ava WGH 05.03.2014 21:53 # +3

            > 'false' будет считать true
            Зато '0' будет считать false.
            Ответить

          • ava guest 05.03.2014 22:22 # −1

            > а во-вторых 'false' будет считать true
            И?
            Ответить

            • ava WGH 05.03.2014 22:26 # 0

              Само по себе это логично. Вот только когда у нас появляется (bool) '0' === false, логика моментально улетучивается.
              Ответить

              • ava guest 06.03.2014 09:13 # 0

                А что, вообще никак по-нормальному в рнр не разделить строки и числа?
                Ответить

                • ava inkanus-gray 06.03.2014 22:01 # +1

                  Способ есть: http://ru2.php.net/manual/ru/function.is-string.php

                  Надо заметить, что is_numeric (в отличие от предыдущей функции) не отличает от чисел строки, состоящие из одних цифр.

                  Кстати, == означает сравнение после приведения типов не только в пышечке, но и ещё в куче ненужных языков типа ECMAScript и Java.
                  Ответить

                  • ava bormand 06.03.2014 22:45 # +2

                    Джава то тут каким боком? ;)
                    Ответить

                    • ava inkanus-gray 06.03.2014 23:32 # +1

                      Только таким, что == делает совсем не то, что ожидает новичок.

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

            • ava bormand 06.03.2014 05:30 # +1

              Ну а автору, судя по коду, нужно считать 'false' как false.
              Ответить

              • ava guest 06.03.2014 09:12 # −1

                Так может, это автор мудак? В питоне такой подход возражений не вызывает.
                Ответить

                • ava WGH 06.03.2014 14:27 # 0

                  Если автору нужно распарсить текст - он уже мудак?
                  Ответить

                  • ava Lokich 06.03.2014 16:07 # +1

                    как вообще можно из функции, которая должна преобразовывать к булевому типа возвращать 1, 0, и любую другую херню? я конечно понимаю, что php не строготипизированный, но это же фарш работы с разными типами данных.
                    Ответить

                    • ava guest 07.03.2014 02:44 # −1

                      У одной пыхомакаки пригорело (которая поставила минус).
                      Ответить

                  • ava guest 07.03.2014 02:43 # +1

                    Автор мудак, потому что использует строки вместо булевых переменных. Или их в пыхе нет?
                    Ответить

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