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

    • ava nobody 15.11.2012 00:32 # 0

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

      • ava daemon_master 15.11.2012 10:18 # +1

        а где здесь Symfony2 скорей вопрос. Это pure php
        Ответить

        • ava TarasB 15.11.2012 11:29 # −1

          > pure php

          php++
          Ответить

        • ava guest 23.11.2012 20:17 # −2

          Причем тут симфони? Вы считаете, что проект, использующий симфони, должен ограничиваться средствами симфони? В таком случае ваш мозг размером с горошину
          Ответить

          • ava daemon_master 24.11.2012 08:59 # +1

            а для чего же тогда люди и писали симфони? Для того, чтобы все остальные и дальше продолжали писать через $_POST, $_GET и юзать db connection вместо doctrine? Этот код на чистом симфони куда презентабельней хотя бы смотриться
            Ответить

      • ava niggerzilla 15.11.2012 14:42 # 0

        - Омский редирект header("Location: " . $_SERVER['HTTP_REFERER']); вместо нормального 404 или страницы с ошибкой
        - попахивает "__".$this->tables[0] - хотя это могут быть заморочки framework'а
        Ответить

      • ava 7ion 20.11.2012 22:43 # +6

        >не экранизированного $_POST['word']

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

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

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

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

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

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

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

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

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

        ПЕРЕМЕННЫЕ
        Смотрите с 23 ноября во всех кинотеатрах Антарктиды.
        Ответить

        • ava Fai 20.11.2012 22:48 # +2

          Я бы посмотрел.
          Ответить

        • ava PascalGovno 21.11.2012 08:20 # 0

          Metro-Goldwyn-Moskva buys movie rights for six million rubles, changing title to "The Eternal Triangle," with Ingrid Bergman playing part of hypotenuse.
          Ответить

    • ava UnnamedUser 15.11.2012 15:55 # +1

      Вся соль ситуации в том, что от симфони 2 в этом проекте только название.
      Ответить

    • ava roman-kashitsyn 15.11.2012 15:56 # +3

      > Symfony 2
      Мне нравится вторая симфония Ляпунова
      Ответить

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