я щас уписаюсь
if (!$_COOKIE[PHPSESSID]) define (COOKIE, false);
else define (COOKIE, true);

на кой

так нельзя ?

define (COOKIE, !$_COOKIE[PHPSESSID]?false:true);

это вообще порно
if (COOKIE && $_COOKIE['MENU'] && md5($_COOKIE['MENU']) == $_SESSION['MENU_CRC'])

Basili4 Спасибо за совет, ща поправлю, я просто такого способа не знал, дай ссылочку на тему, буду благодарен

Basili4

в общем это вот переписать
f (COOKIE && $_COOKIE['MENU'] && md5($_COOKIE['MENU']) == $_SESSION['MENU_CRC'])
$menu = $_COOKIE['MENU'];
else
{
$engine = new hq_framework;
if(!$engine->exists_table('menu')) $engine->new_table('menu','name,link');
$count = $engine->get_info('menu','name');

остольное пойдет но это все моё ИМХО можешь забить

KaFe
Очень плохо в статеечка могут и соврать а книгах обычно все проверяют

Basili4 Ну я же на опытах все проверяю, да и у меня мозги есть. Да и вообще php.net, php.su - я вполне доверяю. Ладно мы ушли от темы

Ты сказал заменить тут

if (COOKIE && $_COOKIE['MENU'] && md5($_COOKIE['MENU']) == $_SESSION['MENU_CRC'])
$menu = $_COOKIE['MENU'];
else
{
$engine = new hq_framework;
if(!$engine->exists_table('menu')) $engine->new_table('menu','name,link');
$count = $engine->get_info('menu','name');

что именно тебя не устраивает

спасибо twin как всегда выручаешь

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

if (COOKIE && md5($_COOKIE['MENU']) == $_SESSION['MENU_CRC'])

и вообще думаю md5 поменять на crc32

if (COOKIE && md5($_COOKIE['MENU']) == $_SESSION['MENU_CRC']) ох мне уж эти детки

if (COOKIE && (md5($_COOKIE['MENU']) == $_SESSION['MENU_CRC'])) вот как надо.
как у тебя COOKIE && md5($_COOKIE['MENU']) вообще не понятно что даст

Basili4 раньше я ставил скобки сейчас нет, но мне кажется оно и так работает безупречно.

KaFe
вот пример

выведи и погляди что получится
difine ('COOKIE',true);
$Keks="12213131231";
echo COOKIE && md5($Keks)."\n";
echo md5($Keks)."\n";

Basili4 я тебя понимаю, но сделай в условии. Вот например статья от софтайма где они с данной темой опростоволосились
У них

   $flag1 = true;
   $flag2 = true;
   $flag3 = false;
   if($flag1 && ($flag2 || $flag3))
   {
      echo "<p>Условие истинно<p>";
   }
   else
   {
      echo "<p>Условие ложно<p>";
   }

у них этот код Возвращает "Условие ложно", а у меня наоборот.

Я поставлю скобочки, надо же логику разделать

Это код возвращает условие истинно я проверил.

KaFe
для тебя нужно вот что
$flag1 = true;
$flag2 = 5;
$flag3 = 5;
if($flag1 && ($flag2 == $flag3))
{
echo "<p>Условие истинно<p>";
}
else
{
echo "<p>Условие ложно<p>";
}

Basili4 ладно забили я уже раставил скобочки, да и буду ставить впредь.

теперь с этм

{
$menu = $_COOKIE['MENU'];

$engine->new_table('menu','name,link');
}
else
{
$engine = new hq_framework;

$count = $engine->get_info('menu','name');

тут если я тебя правильно понял должно быть так

Те не правильно понял

Эту строчка создает таблицу в файловой структуре, если ее нет

if(!$engine->exists_table('menu')) $engine->new_table('menu','name,link');

KaFe
ну тогда все.
Мне не очень нравтся твой стиль как то все скомкано сжато а сам код вроде ниче. Я с ООП ни работал еще про него тебе ничего не скажу

по стилю мне не нравится вот это

{
				$link = $engine->get_record('menu','link',$x);
				$menu.='<li><a href="'.$link.'">'.$name.'</a><li>';
				}

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

      { 
				     $link = $engine->get_record('menu','link',$x);
				     $menu.='<li><a href="'.$link.'">'.$name.'</a><li>';
				}

Но это твое дело

Про продуманности не знаю там везде ООП.
Насчет безопасности я дырок не вижу.

Кстати, До использования это кода было, точнее до того пока я не начал хранить меню в куках

0.0022349357605

а щас

0.00167798995972

Результаты радуют

KaFe
ты 10000 раз запускал скрипт может в этот момент ты музычку на компе включил и потому такая разница. при однократном запуске нельзя судить о скорости

Basili4 я на двух хостингах проверял и на своих двух серверах, на денвере прирост производительности, на смалле тоже, на хостинге от 1gb прирост, но вот на другом хостинге без этого метода скрипт работал быстрее

я по 10 раз запускаю один скрипт