STEVER Dec 20 2011 at 13:33

Используем $_COOKIE как $_SESSION

2 min

12K

PHP*

From sandbox

+47

Comments 37

UFO landed and left these words here

trueneutral Dec 20 2011 at 14:17

С одной стороны — хорошо, с другой — накладывает излишние ограничения (вроде невозможности переопределить время выставления кук). На мой взгляд, лучше использовать собственные врапперы вроде setcookie($name, $value), getcookie($name), не менее удобные, но дающие возможность указывать значения отличные от дефолтовых.

Skull Dec 20 2011 at 14:56

А помоему можно допилить этот класс, сделав _setCookie() доступной вне класса. Тогда и дефолтное expire можно изменить, и path.

mr_avi Dec 20 2011 at 15:55

И получится функция set_cookie:)

Nc_Soft Dec 20 2011 at 17:24

Skull Dec 20 2011 at 17:44

Да :)
Нужно в нее еще добавить сохранение в массив storage[] и будет действительно setcookie с без стандартного потери функционала.

resurection Dec 21 2011 at 12:30

Можно добавить публичный метод:
simpleSetCooke($name, $value, $expire=0x7FFFFFFF)

Остальные параметры (domain, secure, httponly) практически никогда не используются и идут в лес.

Потом можно допилить эту функцию, что бы третьим аргументом она могла принимать строку и прогонять её через strtotime(). Например:

$COOKIE->simpleSetCooke('foo', 'bar', '+2 week'); // разве не клёво?

В любом случае, топикстартер создал фундамент, а построить на нём можно очень много.

resurection Dec 21 2011 at 12:55

P.S. Переопределять глобальные переменные, это конечно же очень плохо.

mr_avi Dec 21 2011 at 13:12

Ну автор как раз хотел избавиться от функций (методов), а тема оберток стандартных функций (глобальных переменных) это уже другая тема. А так да, действительно классно, но слишком длинно ($COOKIE->simpleSetCookie...), лучше уж Cookie::set(...) какой-то.

romy4 Dec 20 2011 at 14:33

«Эс как доллар», прастите)

zim32 Dec 20 2011 at 14:35

У меня без всяких SPL и ArrayAccess die(print_r($_COOKIE)); выводит массив кук. Я что-то делаю не так?

zim32 Dec 20 2011 at 14:36

Простите. Вы имеете ввиду установку кук. Я подумал что получение.

zim32 Dec 20 2011 at 14:39

Вообще ArrayAccess очень удобная порой штука. Я на ней делал поиск переводов в шаблонах. Просто переопределяя обычный массив переводов объектом с ArrayAccess

MpaK999 Dec 20 2011 at 14:38

Сто лет не работал с $_COOKIE напрямую, всё через класс, свой или от CodeIgniter.

alagar Dec 20 2011 at 14:41

trigger_error('Cookie value was not set', E_USER_WARNING); может лучше Exception кидать?

the_ghost Dec 20 2011 at 15:35

Да ладно. Всегда ли приложение не может работать без куки?

alagar Dec 20 2011 at 16:57

не понял вас

Riateche Dec 20 2011 at 17:31

Exception нужно еще отлавливать, по умолчанию скрипт просто завершит работу при возникновении исключения. Обычный warning просто будет проигнорирован.

alagar Dec 20 2011 at 17:43

что мешает добавить try{} catch(e)?

KeepYourMind Dec 20 2011 at 18:22

Все еще игнорируете варнинги? Тогда мы идем к вам!

IgorMats Dec 20 2011 at 14:52

А я наоборот предпочитаю обертки, даже для $_SESSION или там $_POST.

Disturbed Dec 20 2011 at 15:17

Боюсь, что unset реализован неправильно — следуя этому интерфейсу, клиент будет ожидать, что кука удалится на клиенте после вызова unset.
Да, и кстати, — переписывать супеглобальные массивы — это плохо :)

STEVER Dec 20 2011 at 16:19

да, вы правы, unset стоит дополнить.

shagguboy Dec 20 2011 at 15:18

trigger_error('Cookie value was not set', E_USER_WARNING);
не делайте так… кидайте исключение. их придумали гораздо раньше SPL

ainu Dec 20 2011 at 15:51

Идея на пятёрку. Велосипед — это здорово.
Вообще, ИМХО велосипед = новый код, которого раньше не было, который реализует существующую идею или её часть.

mr_avi Dec 20 2011 at 15:54

А как задать домен, время жизни и другие параметры?

nick4fake Dec 20 2011 at 17:27

Забавно. Вас не смущает, что работа с cookies на php потому и происходит через setcookie, что иначе выплывает ряд проблем? Идея интересна в реализации, но абсолютно бесполезна

AstonMartin Dec 20 2011 at 18:34

Каких? Расскажите, если не затруднит.

nick4fake Dec 20 2011 at 18:44

Во-первых, самая очевидная проблема — заголовки могут быть уже отправлены. Сегодня это не так актуально, но сама функция setcookie добавляет наглядности.
Во-вторых, наличие отдельно read-only массива и функции установки символизирует саму схему работы с ними. Только после отправки заголовка и обновления страницы мы можем удостовериться, что cookie установлен. У вас же все в перемешку
Это то, что сходу в голову пришло. Может еще что-то придумаю (не считая сложности реализации, неустойчивости и т.д.: не будем консерваторами и вообще унылыми личностями, эти проблемы несущественны)

mr_avi Dec 20 2011 at 19:25

В куках, в отличии от сессии, не только ключ значение, а еще и ряд дополнительных параметров хранения.

VolCh Dec 20 2011 at 19:43

ООПничать так ООПничать — саму куку нужно сделать объектом. Обязательный параметр в конструкторе -значение куки, необязательные — домен, время и т. д. Плюс метод __toString(), возвращающий строковое значение. А методы ArrayAccess возвращают/принимают объект. Как-то так :)

winbackgo Dec 20 2011 at 20:15

Решение абсолютно бесполезно, хотя бы по следующим причинам:
1. Любой опытный программист знает что $_COOKIE['name'] = 'val' никуда ничего не сохраняет. И подсознательно это использовать не будет.
2. В любом месте программы можно допустить ошибку if ($_COOKIE['name'] = 'myval') и весь механизм летит к черту. Это как вариант). Проблема глобальных переменных.
3. Куки ставятся на заранее заданное кол-во времени и управлять этим никак нельзя, возвращаемся к setckookie. А если мне надо их поставить на год? А если на сессию?
Навскидку что пришло в голову, возможно есть еще нюансы.

Рискну предложить свою обертку которую я использую. Подрихтовал немного и выложил на гитхаб: github.com/behigh/Cookie

mr_avi Dec 21 2011 at 13:15

2. Во избежание всегда делаю в йода-стиле if ( 'val' == $var )

Обёртка хорошая, а для сессии подобный враппер не используете?

winbackgo Dec 21 2011 at 20:21

>> для сессии подобный враппер не используете?
Да, конечно использую. Но сам объект управления сессиями в моем случае делает намного больше чем просто управлением параметрами. Это аутентификация, а также предоставление данных о текущем пользователе. Иными словами, класс заточен под фрэймворк.

namespace Dec 21 2011 at 00:04

Спасибо. Это правда очень полезно.

wartur Dec 21 2011 at 04:23

у меня есть замечание по части кода.

> $domain = false, $secure = false, $httponly = false
лучше в таком случае пользоваться null типом.
а проверять if(isset($variable)){...}

в таком случае просто быстрее будет.

Big_Shark Dec 21 2011 at 07:48

Можно ещё добавить сохранения массивов

$_COOKIE['lang'] = array('ru', 'en');