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

    +155.3

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    function gp($gp){ 
  $gp_res=$_GET[$gp]; 
  if ($gp_res == ''){
    $gp_res=$_POST[$gp];
  } 
  return $gp_res;
}

    а поидее можно было юзать $_REQUEST[$gp]

    Запостил: popstas, 21 Октября 2009

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

    • ava guest 22.10.2009 00:12 # −2

      Ты дебыл если $_REQUEST юзаешь.
      Так тоже ниправильно
      правильно через isset
      Ответить

      • ava guest 22.10.2009 00:33 # 0

        пиздец ты щас хуйню сморозил (на код не смотрю, чисто камент). связал таки иссет с суперглобальным массивом. Ты бы ещё килограммы с метрами посравнивал.
        Ответить

        • ava guest 22.10.2009 00:50 # 0

          Кури http://php.net/manual/en/function.isset.php

          Хули что он суперглобальный ?
          Ответить

      • ava popstas 22.10.2009 00:38 # 0

        > правильно через isset
        Ты об этом: $gp_res == '' ? Не об этом разговор, а о том, что всю эту функцию заменяет $_REQUEST
        Ответить

    • ava striker 22.10.2009 00:43 # 0

      REQUEST содержит еще и cookies, не забываем об этом!
      Ответить

      • ava DeAle 22.10.2009 09:03 # 0

        Голактеко от этого в опастности не станет) Принял, обработал, выполнил) А с печеньки или с формы данные прийшли уже не важно)
        Ответить

    • ava guest 22.10.2009 00:48 # +3

      Реквест для ебланов которые не могут проследить откуда у них переменные беруться, и от таких же ебланов в основном SQL-injection уязвимости происходят.
      Ответить

      • ava guest 22.10.2009 07:28 # 0

        Откуда такая категоричность?
        Ответить

        • ava interested 22.10.2009 08:06 # +1

          guest пришёл с работы не в духе, вот и решил выместить зло на окружающих, при этом не аргументированно, очень резко, это свойственно детям. У взрослых это называется "регрессией". Бывает у всех после тяжёлого рабочего дня.
          Ответить

      • ava DeAle 22.10.2009 11:55 # 0

        Каким боком $_REQUEST приводит к sql inj?)
        Ответить

    • ava nolka4 22.10.2009 04:26 # −1

      нормальная обертка, човы :)

      мне гораздо удобнее будет написать gp('login'), чем $_REQUEST['login']
      Ответить

      • ava guest 22.10.2009 07:43 # 0

        +1
        Ответить

      • ava striker 22.10.2009 13:49 # 0

        чем удобнее? а как проверить, что login был передан в REQUEST?
        Ответить

        • ava nolka4 23.10.2009 07:04 # −1

          а что мешает сделать 2 фии - gp($key) и pp($key) для гет и пост соответственно?
          Ответить

      • ava Yozheg 23.10.2009 01:02 # +1

        1. Что будет, если переменная таки передана ГЕТом, но является пустой строкой?
        2. Что будет, если такая переменная есть и там и там, причем в ПОСТе - нужная, а ГЕТом ты случайно передал что-то левое под тем же именем?
        3. И наконец! Что будет, если еррорлевел = E_ALL ??? :)))
        Ответить

        • ava nolka4 23.10.2009 07:05 # 0

          1. ничего страшного
          2. а что мешает сделать 2 фии - gp($key) и pp($key) для гет и пост соответственно?
          3. ну гоняй дальше свои сайты с еррорлевелом E_ALL если ты такой герой, удачи!
          Ответить

          • ava striker 23.10.2009 11:15 # +1

            3. ололо! говнокодер детектед! Ошибки надо не скрывать, а исправлять!
            Ответить

            • ava interested 23.10.2009 17:17 # 0

              Не буду выдавать имён и названий... Но... Видал даже выключенные предупреждения! Как только ошибки включаешь, так вместо сайта видишь одни "варнинги". Три экрана ошибок, три строчки сайта.
              Ответить

              • ava guest 27.10.2009 10:31 # 0

                быдлокодеры только так и могут, без нотисов писать очень даже можно, и даже нужно.
                Другое дело на продакшн, надо скрывать их от глаз пользователя, но складывать в лог, для дальнейшего анализа и фиксов.
                Ответить

            • ava nolka4 23.10.2009 17:52 # 0

              это понятно, но в продакшне использовать E_ALL имхо моветон
              Ответить

          • ava Yozheg 23.10.2009 16:44 # 0

            striker +1
            Ответить

    • ava guest8 11.07.2020 03:35 # −999

      показать все, что скрытоvanished
      Ответить

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