Форум PHP программистов

[ Поиск ] - [ Пользователи ] - [ Календарь ]

shabalin-roman

14.08.2011 - 18:44

Скрипт проверяет авторизован ли посетитель на странице. Всё работает интересно знать мнение других, ошибки.


<?php
 mysql_connect ("88888.mysql.88888.ru", "88888", "88888");
 mysql_select_db ("88888");
 mysql_set_charset ("88888");
?>
<?php
 if ($_COOKIE ["Identificator"])
  {
   $Identificator = mysql_real_escape_string ($_COOKIE ["Identificator"]);
   $Request = mysql_query ("SELECT Identificator FROM Table WHERE Identificator = '$Identificator'");
   $Result = mysql_fetch_array ($Request, MYSQL_ASSOC);
   if ($Result ["Identificator"] == TRUE)
    {
     include ("Страница для авторизованного посетителя");
     exit;
    }
   else
    {
     header ("Location: http://www.88888.ru/login.php");
     exit;
    }
   exit;
  }
 else
  {
   include ("Страница дла не авторизованного посетителя");
   exit;
  }
?>

Спасибо!!!

Спустя 19 минут, 52 секунды (14.08.2011 - 18:04) Invis1ble написал(а):

Мнение об ошибках? Их есть у меня!
начнем с того, что неплохо было бы написать error_reporting(E_ALL); в начале скрипта, тогда сам увидишь бОльшую часть ляпов

Спустя 5 минут, 31 секунда (14.08.2011 - 18:09) Invis1ble написал(а):

Продолжим.
1. Вместо mysql_fetch_array() уместней в данном случае mysql_result()
2. куча ненужных exit'ов, оставить можно (а нужно ли?) только после header()
Вобщем-то, это все, на мой взгляд. Особых недочетов больше не вижу

Спустя 14 минут, 21 секунда (14.08.2011 - 18:23) imbalance_hero написал(а):

shabalin-roman
Лишняя переменная Identificator , exit - это зло!

Спустя 2 минуты, 15 секунд (14.08.2011 - 18:26) Invis1ble написал(а):

оффтоп

Свернутый текст

Товарисчь одменестратар када вы проверете астальные работы?

Спустя 11 минут, 3 секунды (14.08.2011 - 18:37) imbalance_hero написал(а):

Invis1ble
Ну я уже 10-ую работу поломал, сегодня 100% оценю её

Остальные по возможности

Быстрый ответ:

Здесь расположена полная версия этой страницы.