Скрипт проверки Cookies

[Костик]

Скрипт проверяет включины у юзера cookies в браузере и если отключены выводит сообщение

Код: Выделить всё • Развернуть

<?php
if($_GET['cookie'] == 'test')
    {
        if(!isset($_COOKIE['cookietest']))
            {
            $no_cookies = "<font size=\"7\" color=\"#ff0000\">Cookies OFF</font>\n";
            }else{
            header("Location: $PHP_SELF?");
            }
    }else{

    if(!isset($_COOKIE['cookietest']))
        {
        setcookie("cookietest","blablabla");
        header("Location: $PHP_SELF?cookie=test");
        }
    }

//Выводим сообщение - Cookies ON или Cookies OFF
if(!isset($_COOKIE['cookietest']))
            {
            echo $no_cookies;
            }else{
            echo "<font size=\"7\" color=\"#336600\">Cookies ON</font>\n";
            }
?>

Вот можете скачать готовую страницу с рабочим скриптом

cookies.zip

(905 байт) Скачиваний: 423

[Lundil]

вроде правильный, рас ошибок интерпретатора php нет.

------
правки появились наконец.
ну что - это дело не меняет, остается вопрос - "а че не работает-то?" и "А че должно быть в итоге?"
жду еще одной правки!

------
как показал опрос виновника торжества, здесь не ошибку искать нужно, а это сделано для

вот сморите какую я хреновину написал!!! Восхищайтесь мной!!!

[Костик]

все теперь как бы финальная версия, надеюсь больше исправлять ничего не придется от забывчивости )

[Lundil]

все теперь как бы финальная версия, надеюсь больше исправлять ничего не придется от забывчивости )

значит так!
если это все сделано для показухи - вот критика - А че так замудренно-то?
нельзя например сделать так? (не проверял)

Код: Выделить всё • Развернуть

 if(!isset($_COOKIE['cookietest']))
      {
      setcookie("cookietest","blablabla");
sleep(1); //может и не нужен...
 if(!isset($_COOKIE['cookietest']))
      {
       echo 'Куков нет!';
       };
      };

[Lundil]

вот еще критика

Код: Выделить всё • Развернуть

setcookie("cookietest","blablabla");
      header("Location: $PHP_SELF?cookie=test");

1. после этих строк желательно сразу ставить die('');
2. header лучше перенаправлять с ошибкой header("Location: $PHP_SELF?cookie=test", TRUE,301);

[Lundil]

еще критика

Код: Выделить всё • Развернуть

header("Location: $PHP_SELF?");

нахера вопросительный знак в конце?

[Lundil]

кароче вердикт такой
1. твой скрипт сложно поддается разуму (от кого кодируешься? или чтоб враги не о чем не догадались?)
2. мелкие недочеты по стандартам HTTP

кароче нормально, но можно лучше!

[Костик]

хз) можно и по твоему сделать, но мне на тот момент задумалось делать сложным путём)) ыыы
хотя не на много он сложный, просто расписан по подробнее плюс функциональнее наверно

в общем пашет и ладно )