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

    +168

    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
    function check() {
   $pattern = '|^[-\d]*$|i';
   if ($this->is_required) {
     # Проверяем поле value на максимальное и минимальное значение
     if ($this->min_value != $this->max_value) {
       if ($this->value < $this->min_value || $this->value > $this->max_value) {
         return "Поле \"".$this->caption."\"  должно быть больше ".$this->min_value." и меньше ".$this->max_value."";
       }
     }
     $pattern = "|^[-\d]+$|i";
   }
   # Проверяем, является ли введённое значениe целым числом
   if(!preg_match($pattern, $this->value)) {
     return "Поле \"".$this->caption."\" должно содержать лишь цифры";
   }
   return "";
}

    Целое число - это число, в котором может быть целая куча знаков минус, но самое главное - без учёта регистра! (Softtime.ru)

    Запостил: dwinner, 03 Декабря 2010

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

    • ava Oleg_quadro 06.12.2010 13:50 # 0

      Целое число — это число, которое может быть пустой строкой!
      Либо такое:
      -3-23-23-3--23- номер телефона.
      Ответить

      • ava dwinner 06.12.2010 14:03 # 0

        Номер телефона всё-таки подвержен определённому формату ввода и все web-приложения для подобного ввода так или иначе отражают этот факт. А здесь хотели проверить ввод целого числа со знаком, но вместо довольно тривиальной |^[-]?[\d]+$| сделали глупейшую ошибку.
        Ответить

        • ava Uchkuma 06.12.2010 16:17 # 0

          А is_numeric не судьба?
          Ответить

          • ava dwinner 06.12.2010 19:58 # −1

            Судьба-судьба... intval ещё, но я про регулярное выражение, ДЛЯ ТЕХ, КТО В ТАНКЕ!
            Ответить

          • ava andremacareno 06.12.2010 21:18 # 0

            а чего не (int) ?
            Ответить

            • ava Oleg_quadro 06.12.2010 22:17 # 0

              База может поддерживать гораздо большие числа чем PHP.
              <?php
              $var = (int) 9999999999999;
              print $var;//1316134911
              ?>
              Ответить

            • ava dwinner 07.12.2010 00:14 # 0

              Можно и (int), побыстрее будет... Сишник?! Уважаю! ;-)
              Ответить

              • ava Oleg_quadro 07.12.2010 01:52 # 0

                Труъсишник
                Ответить

                • ava Lure Of Chaos 07.12.2010 17:51 # +1

                  > Труъсишник
                  http://3.bp.blogspot.com/_4C1ZwEub0Po/Sri4VnBqqPI/AAAAAAAAAVI/GoPvTrDdNmE/s1600/benny_hill_clothesline.jpg
                  Ответить

                  • ava Анонимус 07.12.2010 19:58 # 0

                    в чем смысл игры? воровать женское нижнее белье?
                    Ответить

                    • ava Lure Of Chaos 07.12.2010 20:08 # 0

                      в 1ой главе белье, во второй у мужика из сада яблоки, что потом, не хватило терпения узнать = ) но главное не получить за это по чайнику = )
                      эх, спекки... классика бессмертна... = )
                      Ответить

                      • ava bugmenot 07.12.2010 20:09 # 0

                        есть ссылка на более полное описание?
                        Ответить

                      • ava Анонимус 07.12.2010 21:17 # 0

                        да у спекки все игры отличные
                        а сейчас мир пхп и говноигр, где кроме купленного 3Д движка ничего нету
                        Ответить

                        • ava Lure Of Chaos 07.12.2010 22:43 # 0

                          поэтому играем в классику. Из писюшных еще могу отметить Gish
                          Ответить

                          • ava Анонимус 08.12.2010 13:22 # 0

                            из писюшны: gobliins (1,2), the lost vikings (первые), prince of persia (первый, да и второй), another world, и еще парочка квестов от сиерры

                            это святое
                            Ответить

                            • ava Lure Of Chaos 08.12.2010 15:49 # 0

                              ну я даже молчу про это, потому что это общеизвестное
                              из менее известного:
                              за игровую атмосферу мне понравились Bloodrayne 2, Aliens vs Predator 2 (до сих пор иногда врубаю, повышает настрой)
                              был бы шедевром Vampire: the Masquerade - bloodlines, отличная задумка. Но движок настолько похабен, что играть невозможно из-за глюков (((
                              Ответить

            • ava Uchkuma 07.12.2010 08:51 # 0

              Причем тут (int) и intval() вообще? Как это применимо в контексте данной задачи? Это ведь не защита от sql-инъекции.
              Ответить

              • ava dwinner 07.12.2010 10:03 # +1

                Да мы собственно не про оптимальное решение задачи, а про корявую PCRE в данной задаче, кстати is_numeric('any float number') = true! ;-)
                Ответить

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