"PHP" / Говнокод #2017 Ссылка на оригинал

0

  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 popstas, (Updated )

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

  • ava guest # Ссылка на оригинал 0

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

    • ava guest # Ссылка на оригинал 0 ⇈

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

    • ava popstas # Ссылка на оригинал 0 ⇈

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

  • ava striker # Ссылка на оригинал 0

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

    • ava DeAle # Ссылка на оригинал 0 ⇈

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

  • ava guest # Ссылка на оригинал 0

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

      • ava interested # Ссылка на оригинал 0 ⇈

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

  • ava nolka4 # Ссылка на оригинал 0

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

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

    • ava striker # Ссылка на оригинал 0 ⇈

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

      • ava nolka4 # Ссылка на оригинал 0 ⇈

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

    • ava Yozheg # Ссылка на оригинал 0 ⇈

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

      • ava nolka4 # Ссылка на оригинал 0 ⇈

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

        • ava striker # Ссылка на оригинал 0 ⇈

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

          • ava interested # Ссылка на оригинал 0 ⇈

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

            • ava guest # Ссылка на оригинал 0 ⇈

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

          • ava nolka4 # Ссылка на оригинал 0 ⇈

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

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

Где здесь C++, guest?!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓