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

    +158.2

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

    Запостил: dicos, 03 Декабря 2009

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

    • ava dicos 03.12.2009 05:04 # 0

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

    • ava guest 03.12.2009 08:53 # −1

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

      • ava guest 03.12.2009 10:07 # +6

        есть более стандартный способ, через header('Location: http://адрес');
        Ответить

        • ava VermiVermi 03.12.2009 11:23 # +1

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

          • ava mrbig66 03.12.2009 13:13 # +1

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

    • ava guest 03.12.2009 15:24 # −4

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

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

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

      • ava guest 03.12.2009 16:08 # +2

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

      • ava mrbig66 03.12.2009 17:45 # 0

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

        • ava guest 03.12.2009 23:06 # +1

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

      • ava guest 04.12.2009 01:44 # 0

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

    • ava dicos 08.12.2009 13:46 # 0

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

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