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

    • ava guest 09.04.2010 22:55 # −1.6

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

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

      а так не особенно говнисто
      Ответить

    • ava DmitryDick 09.04.2010 23:09 # −2.8

      да не, это не константа. к элементам массива так тоже можно обращаться
      Ответить

      • ava HyperGeek 09.04.2010 23:26 # +0.4

        Notice вылезет
        Ответить

        • ava larrygingras2 10.04.2010 19:00 # −1

          Срать на нотисы
          PHPшные движухи в плане "как надо" на самом деле абсолютно неуместны. Либо надо было сразу делать "как надо" либо уже молчать в тряпочку.
          Особенно меня убило ВНЕЗАПНОЕ отключение magic_quotes_gpc в рекомендуемой версии php.ini для production environment... (Кто в курсе тот поймет)...
          Ответить

    • ava guest 09.04.2010 23:44 # +0.4

      Вот при виде такого хочется автора сунуть мордой в несвежее говно и держать там пока не захлебнется. Тут для правильного кода даже не надо ничего знать или искать, достаточно прочитать две строчки NOTICE'а. Но нет, это ведь охуеть как сложно
      Ответить

      • ava pee 10.04.2010 00:17 # +3

        Быдло-книги советуют отключать "эти ненужные советы".
        Ответить

        • ava guest 10.04.2010 06:05 # +0.8

          *плачет кровавыми слезами* А быдло-руководители ещё и заставляют писать говнокод по образу и подобию своего... Хоть увольняйся, ёпт.
          Ответить

          • ava larrygingras2 10.04.2010 19:09 # 0

            Дак пиши без кавычек, раз просят, проблем-то.
            Лично я быдло-руководитель и я РЕКОМЕНДОВАЛ писать строковые ключи в кавычках, но некоторые так и пишут без кавычек, мне похуй. Главное что работает...
            Ответить

            • ava guest 10.04.2010 23:30 # +0.2

              Меня, слава Богу, без кавычек не просят. Меня просят использовать register_globals вместо суперглобальных массивов и чихать на нотисы и варнинги вплоть до заявлений типа "вот тут нафига у тебя лишняя проверка? всё равно, раз переменная не объявлена, это null, вот и убери!". ООП объявлено вселенским злом, глобальный неймспейс засран под завязку. Уважаемый быдло-руководитель©, когда Вы поймёте, что борцы за чистоту кода не просто так глотки разрывают, а говорят вообще-то дельные и нужные вещи, может быть слишком поздно. До моего руководителя это пока не доходит, но я, по крайней мере, точно знаю тот момент, когда это по нему ударит, и я окажусь прав.
              Ответить

              • ava larrygingras2 10.04.2010 23:38 # 0

                Эм...
                А в чем проблема-то?
                Это всё ведет только к тому, чтобы вас быстренько взломали, причем не хакеры, а дети-со-скриптами.
                Я не думаю что ты будешь крайним в этом случае, раз ты предупреждал...
                Или ещё лучше (для тебя), раздобудь любой (похуй какой, я думаю там любой что надо покажет) сканер уязвимостей, проскань и приди сутречка к директору с отчетом...
                При грамотном подходе премия обеспечена...
                Ответить

      • ava larrygingras2 10.04.2010 19:04 # −0.2

        Ну у нас на серверах собственно error_reporting отключен, а в error_log разумеется смотрят только когда проблема... Так что всякими там ворнингами и прочими нотисами он засирается по дохуя мегабайт в день, при том что всё работает.
        Ответить

    • ava DmitryDick 10.04.2010 11:10 # +1.2

      это писал несмыслящий человек, не надо его ругать, гг :)
      Ответить

    • ava Death 10.04.2010 18:34 # −1.4

      Интересно, кто-нибудь расскажет мне, глупому зачем все пишут isset ???
      разве нельзя просто написать if ($_POST[submit]) { ... } ???
      Ответить

      • ava larrygingras2 10.04.2010 19:06 # −1

        Результат разный
        В твоей конструкции условие не сработает если, например, в $_POST[submit] будет 0 (или "0", не важно)
        Ответить

        • ava Death 10.04.2010 19:26 # 0

          В данном контексте это не имеет значение, т.к. в submit'e должно быть что-то но не "0"
          Ответить

          • ava larrygingras2 10.04.2010 19:31 # 0

            Ну соглашусь, с учетом контекста, да. Если только там кнопка submit не называется "0" :) (а что, и такое возможно, чисто теоретически)
            Но всё же я бы не советовал лишний раз делать подобные допущения. А то мало ли что... Вдруг очередному заказчику захочется кнопку сабмита подписать как "0"...
            Хотя грешен, у самого такое есть в коде :(
            Ответить

            • ava Death 10.04.2010 20:58 # 0

              Это к тому: накой хер делать через жопу, если всё можно сделать гораздо проще...
              Хотя лично я всегда использую hidden поля типа <input type='hidden' name='op' value='send' />
              if ($op) { ... }
              Ответить

      • ava Dreyk 10.04.2010 23:45 # +1.2

        Опять же вылезет нотис, если такого ключа вообще нету.

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

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

      • ava DmitryDick 10.04.2010 23:53 # 0

        низя))
        Ответить

    • ava DmitryDick 10.04.2010 23:49 # 0

      как и хороший код при его знании :)
      Ответить

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