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

    +164

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (isset($_SESSION['captcha_keystring']) && $_SESSION['captcha_keystring'] ==  $_POST['keystring']) 
{} 
else {
$colerror=$colerror+1;
echo "<div class='error'> Неверный код </div>";}

    Еще нашел в своих зарослях, кроме того, что входные данные не фильтровались, так и еще {} else { 0_о я в шоке!!

    Запостил: BOOMik, 23 Ноября 2010

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

    • ava nobody 23.11.2010 01:00 # 0

      ну у ифа обратной логики не существует конечноже ... боян, а не гавнокод, так делают все новички кто не слышал про "!" ну и "!=="
      Ответить

      • ava BOOMik 23.11.2010 01:02 # 0

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

    • ava sectus 23.11.2010 05:46 # +2

      Внимание вопрос: зачем тут фильтровать входные данные?
      Ответить

    • ava andremacareno 23.11.2010 09:24 # +4

      У Вас ошибка:
      >> {О_о я в шоке!!
      Забыли поставить закрывающую фигурную скобку "}"
      Ответить

      • ava BOOMik 23.11.2010 16:27 # 0

        В комменте только кусочек кода, не обращал тут внимание на скобки, а в коде все на месте=)))
        Ответить

    • ava banderror 24.11.2010 21:20 # 0

      "colerror"? ранее не встречал такого стиля наименования )
      Ответить

      • ava BOOMik 24.11.2010 21:21 # 0

        что-бы понятнее было- количество ошибок =)
        Ответить

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