Почему первый id записывается в куку сколько угодно раз?

Question

[Dreaded]

Суть задачи записывать в куку просмотренные лоты на сайте, но только в том случае, если кука еще не содержит в себе id лота. То есть все id в куке должны быть уникальными.
У меня, почему-то уникальность соблюдается только для второго и последующих записей в куку. То есть если я с чистыми куками захожу на страничку с id=1, и нажимаю f5, то все обновления записываются в куку. Т.е. кука будет в себе содержать 1,1,1,1,1 и т.д. Но начиная со второго элемента условие уникальности уже соблюдается. Т.е. если после того как кука уже содержит 1,1,1,1,1 зайти на страничку c другим id , то он уже запишется в куку только один раз.

if(isset($_COOKIE['lot_history'])) {
        $lot_history = json_decode($_COOKIE['lot_history']);
        if (!(array_search($_GET['id']-1, $lot_history))){
            $lot_history[] = $_GET['id']-1;
            $lot_history = json_encode($lot_history);
            setcookie("lot_history", $lot_history, strtotime("+1 day"), "/");
        }
    }
    else {
        $lot_history = json_encode(array($_GET['id']-1));
        setcookie("lot_history", $lot_history, strtotime("+1 day"), "/");
    }

Answer 1

[Максим Федоров]

array_search может вернуть как false так и 0 (в качестве индекса элемента). В Вашей ситуации как раз 0 и возвращается, который в нестрогой проверке интерпретируется как false.
Просто сделайте строгую проверку ...

if (array_search($_GET['id']-1, $lot_history) === false){
 ...
}

P.S. и от такого ужаса как $_GET['id']-1 по всему коду нужно избавляться..

Comments

[Dreaded]

Спасибо, действительно помогло.
А на счёт $_GET['id']-1 не совсем понятно. Во первых - почему это ужас? И как лучше от этого избавиться?
Поясню: id я присваиваю элементам начиная с 1, а индексация массива идёт с 0.

[Максим Федоров]

Dreaded, ужас потому что:
1. Зачем Вы в 3-х местах пишете одно и тоже $_GET['id']-1 ? Если нужно что-то посчитать - посчитали один раз и используйте, Arik в своем ответе как раз это показал
2. Какой смысл от ID отнимать 1? зачем?

[Dreaded]

Максим Федоров, первый пункт понятен, спасибо, согласен.
А по поводу второго: так id у меня нумеруются начиная с 1, а хранятся начиная с 0. То есть первому id соответствует нулевой элемент массива. Кстати вменяемого ответа, почему я нумерую id с нуля - я дать немогу. Просто подумал что ?id=0 в ссылке будет смотреться некрасиво. Я только учусь, и делаю сейчас учебный проект, поэтому буду рад если подскажете, допустимо ли использовать в ссылке нумерацию с нуля )

[Максим Федоров]

Dreaded, в идентификаторы обычно начинаются с 1 а не с нуля....

Answer 2

[Arman]

Согласен с Максимом, но еще добавлю что куки имеют ограничения и после скорее всего nginx будет кидать 500 ошибку т.к. не может прочитать заголовки.

Как-то проще нужно

<?php
$maxHistory = 20;
$addId = $_GET['id'] - 1;

$lot_history = isset($_COOKIE['lot_history']) ? (array)json_decode($_COOKIE['lot_history']) : [];
$lot_history[] = $addId;
$lot_history = array_slice(array_unique($lot_history), -$maxHistory);

setcookie("lot_history", $lot_history, strtotime("+1 day"), "/");

Answer 3

[Dreaded]

Спасибо, за ответы. Что бы не создавать новый вопрос, задам еще один тут, напрямую касающийся темы:
Если пользователь вводит в get запросе id несуществующего лота, то как лучше поступить : изначально не писать его в куку, или потом на странице отображения лотов просто не выводить не существующие?

Comments

[Arman]

404 страница?

[Dreaded]

Arik, не совсем понял что вы хотите этим сказать. Это предложение? Или вопрос ко мне, знаю ли я что это такое? ))
Я загружаю страничку со списком просмотренных лотов, так что 404 тут скорее всего не подходит. Нужно отсеять лоты которых нет в базе. Я вижу два варианта - либо изначально проверять id на наличие в базе, и если его нет, то не писать их в куку. Либо записывать всё в куку, но перед выводом проверять что я вывожу.

404 можно использовать при просмотре лотов , и если запрашивается неверный лот то загружать страницу 404(хотя, мне больше нравится вариант подгрузить просто основной лейаут странички без вывода данных, или с выводом ошибки). В принципе это и будет означать - не записывать изначально в куку.

Собственно в этом и был весь вопрос вопрос - каким путем пойти лучше.

[Arman]

Dreaded, предложение. 1. Вы записываете в куки где был человек, выходит он должен зайти на страницу, на этой странице и должны вроде записывать, выходит если человек не попал на страницу то и записывать ничего не должны. 2. Человек может завтра зайти, а лот в это время скрыли/удалил, выходит нужно и тут проверять есть такой лот на данный момент или нет.