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

    +151

    1. 1
    2. 2
    $g_source= SF(qGet("source")); // $_GET['source'] со всякой фильтрацией
if ($g_source=="") { $g_source=''; }

    Запостил: ferry-very-good, 11 Октября 2010

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

    • ava Lure Of Chaos 11.10.2010 18:08 # −2

      опять лисапедики. qGet("source") == $_GET['source']? щикарно ваще....
      Ответить

      • ava ferry-very-good 11.10.2010 21:24 # 0

        там всякие проверки на xss и пр., а SF это для занесения в БД, всякие pg_quot и str_replace'ы c char'ами )))
        Ответить

        • ava Lure Of Chaos 11.10.2010 21:31 # −2

          случай, когда краткость не сестра таланта.
          Ответить

          • ava ferry-very-good 11.10.2010 21:42 # +2

            Тут говно кодом является 2-я строка, а краткость в web-безопасности(и не только) огромный минус........
            Ответить

            • ava Lure Of Chaos 11.10.2010 22:24 # −2

              2ая вообще тут нахлебница.
              А краткость минус в понимании
              Ответить

            • ava Spider 18.10.2010 19:21 # 0

              Заменяют двойные кавычки на одинарные, что непонятно то? )))
              Ответить

    • ava istem 12.10.2010 00:43 # 0

      К.О. - без него никак...
      Или автор предполагал разницу между "" и '' ?..
      Ответить

    • ava ferry-very-good 12.10.2010 00:58 # 0

      там дальше есть:
      if ($var=="") { $var=0; }
      значит предполагал :D
      Ответить

      • ava Crazyzubr 18.10.2010 15:50 # 0

        Скорее всего автор предполагал, что если вдруг в $g_source окажется 0, то $g_source должен быть пустой строкой ))
        $var это подтверждает. Там, наоборот, если $var окажется пустой строкой, то должен быть нулём)
        Ответить

        • ava Spider 18.10.2010 19:23 # 0

          Скорее всего так и есть, но всё равно ГК, ибо empty()
          Ответить

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