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

    • ну у ифа обратной логики не существует конечноже ... боян, а не гавнокод, так делают все новички кто не слышал про "!" ну и "!=="
      Ответить
      • Как помнится, этот код брал из примера и переделал под себя не вдаваясь в подробности,сейчас прочитал, ужаснулся=)
        Ответить
    • Внимание вопрос: зачем тут фильтровать входные данные?
      Ответить
    • У Вас ошибка:
      >> {О_о я в шоке!!
      Забыли поставить закрывающую фигурную скобку "}"
      Ответить
      • В комменте только кусочек кода, не обращал тут внимание на скобки, а в коде все на месте=)))
        Ответить
    • "colerror"? ранее не встречал такого стиля наименования )
      Ответить
      • что-бы понятнее было- количество ошибок =)
        Ответить

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