Помогите с решением.

[shabalin-roman]

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

Скрипт проверяет авторизован ли посетитель на сайте. Это код для защищенных страциц сайта.

<?php
 mysql_connect ("XXXXX.mysql.XXXXX.ru", "XXXXX", "XXXXX");
 mysql_select_db ("XXXXX");
 mysql_query ("set names utf8");
?>
<?php
 if ($_COOKIE ["Identificator"])
  {
   $Identificator = $_COOKIE ["Identificator"];
   $Request_Select = mysql_query ("SELECT * FROM Table WHERE Identificator = '$Identificator'");
   $Result = mysql_fetch_array ($Request_Select);
   if ($_COOKIE ["Identificator"] == $Result ["Identificator"])
    {
     echo ("Страница для авторизованного");
     exit;
    }
   else
    {
     echo ("Страница входа");
     exit;
    }
   exit;
  }
 else
  {
   echo ("Страница для не авторизованного");
   exit;
  }
?>

[stars]

смущает только вот эта строчка

if ($_COOKIE ["Identificator"] == $Result ["Identificator"])

логично было бы

if ($Result ["Identificator"]!==false)

[LunatiK]

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

Скрипт проверяет авторизован ли посетитель на сайте. Это код для защищенных страциц сайта.

<?php
 mysql_connect ("XXXXX.mysql.XXXXX.ru", "XXXXX", "XXXXX");
 mysql_select_db ("XXXXX");
 mysql_query ("set names utf8");
?>
<?php
 if ($_COOKIE ["Identificator"])
  {
   $Identificator = $_COOKIE ["Identificator"];

Какой смыс из пустово в порожний переносить переменную тем более если нигде нет проверки на правильность её содержимого?

   $Request_Select = mysql_query ("SELECT * FROM Table WHERE Identificator = '$Identificator'");

Если вам нужно всего навсего единственное поле то для чего используете * ?

   $Result = mysql_fetch_array ($Request_Select);

ну тут наверно нужно добавить параметр MYSQL_ASSOC, ибо нумерованную часть вы не используете.

   if ($_COOKIE ["Identificator"] == $Result ["Identificator"])
    {
     echo ("Страница для авторизованного");
     exit;
    }
   else
    {
     echo ("Страница входа");
     exit;
    }
   exit;
  }
 else
  {
   echo ("Страница для не авторизованного");
   exit;
  }
?>

Не стоит так exit'ами разбрасываться, у вас же может дальше что-нибуть выводиться.

Ну и stars сделал верное замечание: Если вы в БД нашли свой идентификатор значит он совпадает с куками уже, пересравнивать их нет смысла

[pkorobeinikov]

 $Identificator = $_COOKIE ["Identificator"];

Словите sql-инъекцию.

[CalvinKlein]

$Request_Select = mysql_query ("SELECT * FROM Table WHERE Identificator = '$Identificator'");

я бы прежде чем делать запрос с переменной $Identificator проверил бы её на безопасность, ведь она берётся из куков, а там может что угодно быть.

Edited August 11, 2011 by CalvinKlein

[shabalin-roman]

Вот так сделал.

<?php
 ini_set ("display_errors", "1");
 ini_set ("error_reporting", E_ALL);
?>
<?php
 mysql_connect ("123.mysql.123.ru", "123", "123");
 mysql_select_db ("123");
 mysql_set_charset ("utf8");
?>
<?php
 if (isset ($_COOKIE ["Identificator"]))
  {
   $Cookie_Identificator = mysql_real_escape_string ($_COOKIE ["Identificator"]);
   $Request_Cookie_Identificator = mysql_query ("SELECT * FROM Info WHERE Identificator = '$Cookie_Identificator'");
   $Result_Cookie_Identificator = mysql_fetch_array ($Request_Cookie_Identificator, MYSQL_BOTH);
   if ($Result_Cookie_Identificator ["Identificator"] == TRUE)
    {
     include ("info-user.php");//Страница для авторизованного.
     exit;
    }
   else
    {
     header ("Location: http://www.123.ru/login.php");//Страница для входа.
     exit;
    }
   exit;
  }
 else
  {
   include ("info-guest.php");//Страница для гостей.
   exit;
  }
?>