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

    +144.8

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    <?php
    if(!@isset($_POST[submit]))
    {
    ...
    ?>

    Запостил: DmitryDick, 09 Апреля 2010

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

    • и зачем тут сцобако

      и где константа submit объявлена ?

      а так не особенно говнисто
      Ответить
    • да не, это не константа. к элементам массива так тоже можно обращаться
      Ответить
      • Notice вылезет
        Ответить
        • Срать на нотисы
          PHPшные движухи в плане "как надо" на самом деле абсолютно неуместны. Либо надо было сразу делать "как надо" либо уже молчать в тряпочку.
          Особенно меня убило ВНЕЗАПНОЕ отключение magic_quotes_gpc в рекомендуемой версии php.ini для production environment... (Кто в курсе тот поймет)...
          Ответить
    • Вот при виде такого хочется автора сунуть мордой в несвежее говно и держать там пока не захлебнется. Тут для правильного кода даже не надо ничего знать или искать, достаточно прочитать две строчки NOTICE'а. Но нет, это ведь охуеть как сложно
      Ответить
      • Быдло-книги советуют отключать "эти ненужные советы".
        Ответить
        • *плачет кровавыми слезами* А быдло-руководители ещё и заставляют писать говнокод по образу и подобию своего... Хоть увольняйся, ёпт.
          Ответить
          • Дак пиши без кавычек, раз просят, проблем-то.
            Лично я быдло-руководитель и я РЕКОМЕНДОВАЛ писать строковые ключи в кавычках, но некоторые так и пишут без кавычек, мне похуй. Главное что работает...
            Ответить
            • Меня, слава Богу, без кавычек не просят. Меня просят использовать register_globals вместо суперглобальных массивов и чихать на нотисы и варнинги вплоть до заявлений типа "вот тут нафига у тебя лишняя проверка? всё равно, раз переменная не объявлена, это null, вот и убери!". ООП объявлено вселенским злом, глобальный неймспейс засран под завязку. Уважаемый быдло-руководитель©, когда Вы поймёте, что борцы за чистоту кода не просто так глотки разрывают, а говорят вообще-то дельные и нужные вещи, может быть слишком поздно. До моего руководителя это пока не доходит, но я, по крайней мере, точно знаю тот момент, когда это по нему ударит, и я окажусь прав.
              Ответить
              • Эм...
                А в чем проблема-то?
                Это всё ведет только к тому, чтобы вас быстренько взломали, причем не хакеры, а дети-со-скриптами.
                Я не думаю что ты будешь крайним в этом случае, раз ты предупреждал...
                Или ещё лучше (для тебя), раздобудь любой (похуй какой, я думаю там любой что надо покажет) сканер уязвимостей, проскань и приди сутречка к директору с отчетом...
                При грамотном подходе премия обеспечена...
                Ответить
      • Ну у нас на серверах собственно error_reporting отключен, а в error_log разумеется смотрят только когда проблема... Так что всякими там ворнингами и прочими нотисами он засирается по дохуя мегабайт в день, при том что всё работает.
        Ответить
    • это писал несмыслящий человек, не надо его ругать, гг :)
      Ответить
    • Интересно, кто-нибудь расскажет мне, глупому зачем все пишут isset ???
      разве нельзя просто написать if ($_POST[submit]) { ... } ???
      Ответить
      • Результат разный
        В твоей конструкции условие не сработает если, например, в $_POST[submit] будет 0 (или "0", не важно)
        Ответить
        • В данном контексте это не имеет значение, т.к. в submit'e должно быть что-то но не "0"
          Ответить
          • Ну соглашусь, с учетом контекста, да. Если только там кнопка submit не называется "0" :) (а что, и такое возможно, чисто теоретически)
            Но всё же я бы не советовал лишний раз делать подобные допущения. А то мало ли что... Вдруг очередному заказчику захочется кнопку сабмита подписать как "0"...
            Хотя грешен, у самого такое есть в коде :(
            Ответить
            • Это к тому: накой хер делать через жопу, если всё можно сделать гораздо проще...
              Хотя лично я всегда использую hidden поля типа <input type='hidden' name='op' value='send' />
              if ($op) { ... }
              Ответить
      • Опять же вылезет нотис, если такого ключа вообще нету.

        Конечно, давайте говорите, что нотисы - это херня, продолжайте юзать строки без кавычек, а так же инкременьте необъявленные переменные и т.д.
        Да и потом не забудьте поругать PHP за то, что он, сука такая, медленный очень.

        PHP - не быдло-язык, просто это язык, на котором говнокод писать оочень легко.
        Ответить
      • низя))
        Ответить
    • как и хороший код при его знании :)
      Ответить

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