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

0

  1. 1
  2. 2
  3. 3
if(empty($_SESSION['login'])) {
    die("<script>top.location.href='index.php';</script>");
}

Запостил: dicos dicos, (Updated )

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

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

    Забыл добавить, на странице фреймов нет.
    Ответить

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

    Ну и??? Если это на страничке типа admin.php - чем не нравится??
    Ответить

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

        мб человек делал инклюды и у него не получалось воспользоваться header =))
        Ответить

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

          Такие инклюды надо стараться ставить после принятия критических решений.
          Такая реализация снижает эффективность работы.
          Ответить

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

    ИМХО, нифига не говнокод!

    Бывает проверки надо делать не перед отправкой хидеров, а позже.
    Джаваскрипт может быть отключен, а прекратить работу скрипта критично!
    По моему даже эллегантно! 😉

    Хотя если в начале страницы, то туповато...
    Ответить

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

      > Бывает проверки надо делать не перед отправкой хидеров, а позже.
      Надо использовать нормальный фреймворк, а не конструировать велосипед с треугольными колёсами.
      Ответить

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

      Дык если Джаваскрипт отключен, то никакого редиректа не будет.
      А от Хедера никуда не деться.
      Ответить

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

        От хейдера никуда не деться?)
        Ещё как можно...
        А вот включён/не включён у юзера ява скрипт это полностью его проблема, die() прекратил сценарий и нас уже больше ничего не волнует.
        Ответить

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

      Бывает проверки надо делать не перед отправкой хидеров, а позже.
      А для этого кагбэ есть ob_* - не?
      Ответить

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

    еще нашел:
    print "<script>location.href='main.php?act=non e'</script>"; die();
    Ответить

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

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

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

    ↑
    8
    ↓