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

    +171

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    if (isset($_POST['nick']))       {$nick = $_POST['nick'];}
if (isset($_POST['pass']))       {$pass = $_POST['pass'];}


if (isset($_GET['nick'])) {$nick = $_GET['nick'];}
if (isset($_GET['pass'])) {$pass = $_GET['pass'];}

    один из видеоуроков Е.Попова=)

    Запостил: Ice, 02 Мая 2010

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

    • ava guest 02.05.2010 13:25 # +4

      А дайте ссылку. Хочу также круто програмить на PHP.
      Ответить

    • ava guest 02.05.2010 13:36 # +3

      попова вообще в палату мер и весов как эталонного говнокодера.
      Ответить

    • ava guest 02.05.2010 13:40 # +2

      да у него все там в том же духе...
      Ответить

    • ava unfunk 02.05.2010 17:23 # 0

      видеоуроки по похапе? олололо
      Ответить

    • ava DanxilLs 02.05.2010 18:55 # +4

      Прикол видел в одном его уроке. Там короче проверка url на корректность. Там было короткое регулярное выражение. Он прокомментировал его примерно так:
      /***************************************
      Это выражение учить не обязательно. Я к примеру его записал в текстовый документ,и теперь когда оно мне нужно я его просто копирую
      /***************************************
      Гыыыыыыыы
      Ответить

      • ava guest 02.05.2010 18:58 # 0

        Будто-бы не верно говорит... Ты, видимо, каждый раз пишешь новое, изобретая колесо заново?
        Ответить

        • ava guest 02.05.2010 18:59 # 0

          Смотри, как-бы колесо не получилось квадратным.
          Ответить

        • ava DanxilLs 02.05.2010 19:03 # 0

          А не лучше ли запомнить выражение? Так на будущее?
          Ответить

          • ava guest 02.05.2010 20:52 # 0

            Да. Запомни на изусть. Если запоминать, то на диск, но понимать его всё равно нужно.
            Ответить

      • ava Ice 03.05.2010 03:22 # +1

        просрал попоф текстовый документ с регвыром и приплыл гыыыы))))
        Ответить

    • ava guest 02.05.2010 19:24 # 0

      а в чем прикол? то что REQUEST не юзает или как?
      или еще то что @ не использовал вместо иссета?
      Ответить

      • ava fih 02.05.2010 19:30 # −2

        кстати да, чего плохого в этом коде?
        Ответить

      • ava Ice 02.05.2010 20:01 # +1

        а прикол в том, что нет контроля над методом передачи данных. Ну и какбе етсамое... пароль гетом передавать - это верх маразма. Фильтрации входного так сказать потока, тоже как бе нету:) Вобщем на таком сайте я бы регаться не стал=)
        Ответить

        • ava guest 02.05.2010 21:06 # 0

          фильтрация может быть ниже..
          не обязаельно же $_POST проверять..
          можно и потом логин проверить..
          Ответить

          • ava Ice 02.05.2010 22:21 # 0

            =))))))) потом, это когда?)))) смотри не забудь)))
            Ответить

            • ava guest 03.05.2010 01:56 # 0

              $id = @$_REQUEST['id'];
              ...
              $id=intval($id);

              в одном месте получаем все переменные, ниже проверяем как надо каждую..
              чем плохо?
              Ответить

              • ava Ice 03.05.2010 03:24 # 0

                у него этого не было. Я не весь код просто выложил:) я чаще всего пользуюсь условиями на $_SERVER['REQUEST_METHOD']
                Ответить

      • ava guest 03.05.2010 14:11 # −2

        Например foreach($_GET as $key => $val) $$key = $val; можно и одной строкой записать, и да, пароль гетом это что-то новое:)
        Ответить

    • ava guest 02.05.2010 20:44 # +1

      резюмирую. код - говно.
      Ответить

    • ava DanxilLs 02.05.2010 21:52 # +6

      Сделал бы еще так!
      if (isset($_POST['nick']))       {$nick = $_POST['nick'];}
if (isset($_POST['pass']))       {$pass = $_POST['pass'];}


if (isset($_GET['nick'])) {$nick = $_GET['nick'];}
if (isset($_GET['pass'])) {$pass = $_GET['pass'];}

if (isset($_SESSION['nick'])) {$nick = $_SESSION['nick'];}
if (isset($_SESSION['pass'])) {$pass = $_SESSION['pass'];}

if (isset($_COOKIE['nick'])) {$nick = $_COOKIE['nick'];}
if (isset($_COOKIE['pass'])) {$pass = $_COOKIE['pass'];}
      Ответить

      • ava guest 03.05.2010 12:45 # 0

        Да, в уроках Евгения Попова много примеров как не нужно делать =)))
        Ответить

    • ava guest 03.05.2010 14:08 # 0

      Незнаю. Я проходил уроки его но такого не замечал... Хотя, я и изучал основы на его уроках, но замечал, что можно сделать и по другому)))
      Ответить

    • ava Sin3v 03.05.2010 20:57 # −6

      показать все, что скрытоПопов красавчик, благодаря ему я научился пыхать, а то что там указано специально для новичков чтобы осваивали.
      Ответить

    • ava guest 06.05.2010 01:54 # +3

      А для чего фигурные скобки после каждого if ? Блока-то как такового нет - по одной строке везде.
      Ответить

      • ava guest 06.05.2010 05:55 # +3

        Многие кодинг стандарты рекомендуют ставить фигурные скобки всегда. Когда я перешёл с ручного вбивания на code snippets, я к этому ненавязчиво привык. Читаемость кода при этом увеличивается, хотя размер тоже =) Но в данном случае афтару по-любому надо в анус паяльник, потому что кто так оформляет? Красивость фигурных скобок становится видна при нормальном форматировании.
        Ответить

    • ava guest10 25.08.2021 03:20 # 0

      - Со мной тоже что-то твориться, Лёка, я так скучал по тебе в Ростове. Я не знал как тебе сказать, что хотел бы с тобой... Но ведь это не правильно, Лёка - парни должны трахаться с телками, а не друг с другом, я что, педик, мы выходит педики? И я не могу справиться с этим - вот посмотри, что у меня твориться, когда я вижу тебя.
      Ответить

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