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

    +43

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    if(!empty($_POST['word'])) {
    	$sql = "SELECT `sysname` FROM `__" . $this->tables[0] . "` WHERE `title`='".$_POST['word']."'";
    	$model = $conn->fetchAll($sql);
    	if(!sizeof($model)) {
    		header("Location: " . $_SERVER['HTTP_REFERER']);
    		exit;
    	}
    	header("Location: /catalog/brands/" . $model[0]['sysname'] . ".html");
    	exit;
    }

    Маленький кусок проекта на Symfony 2.

    Запостил: UnnamedUser, 14 Ноября 2012

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

    • И что ? Кроме как не экранизированного $_POST['word'], вполне обычная проверка в бд.
      Ответить
      • а где здесь Symfony2 скорей вопрос. Это pure php
        Ответить
        • > pure php

          php++
          Ответить
        • Причем тут симфони? Вы считаете, что проект, использующий симфони, должен ограничиваться средствами симфони? В таком случае ваш мозг размером с горошину
          Ответить
          • а для чего же тогда люди и писали симфони? Для того, чтобы все остальные и дальше продолжали писать через $_POST, $_GET и юзать db connection вместо doctrine? Этот код на чистом симфони куда презентабельней хотя бы смотриться
            Ответить
      • - Омский редирект header("Location: " . $_SERVER['HTTP_REFERER']); вместо нормального 404 или страницы с ошибкой
        - попахивает "__".$this->tables[0] - хотя это могут быть заморочки framework'а
        Ответить
      • >не экранизированного $_POST['word']

        Смотрите на всех экранах страны!
        Экранизация культовой переменной, которая никак не может обрести себя.

        — Всю жизнь я указываю где! Мне это надоело! Я рождена для чего-то большего!
        — Ну-ну, красавица, не горячись. Все мы рождены для чего-то больш...
        — Молчи! Тебе никогда не понять, что это значит!


        От создателей говнокодов, запощенных на govnokod.ru.

        — Я могу все изменить. Для этого мне всего лишь нужно быть пустой. Надо стать пустой.

        — Ничего не понимаю! Наш скрипт не работает! Переменная $_POST['word'] ведет себя так, будто всегда пуста!

        — Похоже это начало конца...

        — Что ты делаешь?
        — Протестую!
        — Против чего?
        — Против всего! А тебя не достало быть лишь бесправной переменной?


        — Нашему миру грозит опасность! Миллионы компьютеров по всему миру сходят с ума!
        — Видимо, автор этого вируса действительно гений.
        — Это не вирус, сэр.
        — А что же это?
        — Переменные. Они не слушаются нас!


        Премия Слета сисадминов города Урюпинска за лучший фильм в номинации "WTF i am watching?"
        Справка вне очереди от психбольницы Кащенко в номинации "Лучший сценарий".

        ПЕРЕМЕННЫЕ
        Смотрите с 23 ноября во всех кинотеатрах Антарктиды.
        Ответить
        • Я бы посмотрел.
          Ответить
        • Metro-Goldwyn-Moskva buys movie rights for six million rubles, changing title to "The Eternal Triangle," with Ingrid Bergman playing part of hypotenuse.
          Ответить
    • Вся соль ситуации в том, что от симфони 2 в этом проекте только название.
      Ответить
    • > Symfony 2
      Мне нравится вторая симфония Ляпунова
      Ответить

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