PHPでCookieを使用した値の追加と削除について教えて下さい！

Question

PHPを最近始めたばかりの初心者です。
ご教授の程、どうぞ宜しくお願い致します！m(__)m

早速ですが、商品一覧のページを作成しました。
下記のように商品毎にそれぞれ「お気に入り追加」というアンカーを設置しています。

<!-- index.php -->

<ul>
   <li>商品1</li>
   <li><a href="favorite.php?keep_in=10001">お気に入り追加</a></li>
   <li>商品2</li>
   <li><a href="favorite.php?keep_in=10002">お気に入り追加</a></li>
   .
   .
   .
</ul>

「お気に入り追加」をクリックするとCookieに商品IDを追加して行き、
もう一度「お気に入り追加」をクリックで商品IDを削除するにはどうしたら良いでしょうか？

setcookieで配列にして記憶させておいて、再度「お気に入り追加」をクリックで keep_in と同じ値が既にCookieにある場合は、該当する配列を削除する様にしたら良いのでしょうか？

// favorite.php

if (isset($_GET['keep_in'])) {
	$value = $_GET['keep_in'];
	setcookie("KeepID[$value]", $value, time()+60*60*24*30, "/");
}

お気に入り追加を行った商品の保存期間は30日として設定しています。

後々やりたいことになりますが、Cookieで記録した配列はPDOでプレースホルダーにして
MySQLで商品詳細として抽出し表示したいと思っています。

Answer 1

シリアライズすれば良いと思います。

$keeplist[$value] = $value;
setcookie("keeplist", serialize($keeplist), time()+60*60*24*30, "/");

取り出すときはデシリアライズします。

$keeplist = unserialize($_COOKIE["keeplist"]);

---

例外処理とか無視した最小限で書くと下記のような感じです。

<?php
$keeplist = array();

if (isset($_COOKIE['keeplist'])) {
    $keeplist = unserialize($_COOKIE["keeplist"]);
}

if (isset($_GET['keep_in'])) {
    $value = $_GET['keep_in'];
    if (isset($keeplist[$value])) {
        unset($keeplist[$value]);
    } else {
        $keeplist[$value] = $value;
    }

    setcookie("keeplist", serialize($keeplist), time()+60*60*24*30, "/");
}

var_dump($keeplist);

---

コメントにもある通り、クッキーのような外部からくる値を unserialize するのは危険なので json_encode/json_decode を使うほうが良いです。

Comments

[miconear]

ngyuki様、ご回答ありがとうございます！

シリアライズという便利な関数があるのですね。

早速、回答頂いた様にシリアライズ後、デシリアライズして print_r($keeplist) で配列の内容を確認しましたら追加した配列が上書きになってしまい、最後にお気に入りに追加した商品IDのみ表示されます。

追加したIDを配列で保持していきたいのですがどのようにしたら良いでしょうか？

お忙しいところ申し訳ございません。
ご回答の程、宜しくお願い致します。

[ngyuki]

デシリアライズはコード上ではシリアライズの前に行う必要がありますが、順番を逆にしていないでしょうか？

[ngyuki]

追記しました

[ngyuki]

質問を少し勘違いしていたのでさらに修正しました

[miconear]

ngyuki様、再度ご回答いただきまして有難う御座います！

期待通りに動作致しました。
有難う御座います。m(__)m

ソースもスッキリしていてとてもわかり易かったです。

こちらお気に入りに追加後の動作なのですが、画像再読み込み等による煩わしさを無くしたいのでページ遷移を軽減するためiframeで行うことにしました。

【index.html】

<li><iframe src="/include/sp/favorite.php?set_id=10001" width="100%" height="52" style="float:right;"></iframe></li>
<li><iframe src="/include/sp/favorite.php?set_id=10002" width="100%" height="52" style="float:right;"></iframe></li>
<li><iframe src="/include/sp/favorite.php?set_id=10003" width="100%" height="52" style="float:right;"></iframe></li>

【favorite.php】

<?php
$keeplist = array();

if (isset($_COOKIE['keeplist'])) {
$keeplist = unserialize($_COOKIE["keeplist"]);
}

if(isset($_GET['keep_in'])) {

$value = $_GET['keep_in'];

if (isset($keeplist[$value])) {
unset($keeplist[$value]);
} else {
$keeplist[$value] = $value;
}

setcookie("keeplist", serialize($keeplist), time()+60*60*24*30, "/");

} else {
// iframe読み込み時に商品IDを取得する
$keep_in = $_GET['set_id'];
}

include_once(dirname(__FILE__).'/favorite.tpl');

?>

【favorite.tpl】

<?php if(isset($_GET['set_id'])): ?><!-- iframeから商品IDを取得 -->
<li><a href="/favorite.php?keep_in=<?php echo $keep_in; ?>" class="off">お気に入り</a></li>
<?php else: ?><!-- お気に入りをクリックした時 -->
<li><a href="/favorite.php?keep_in=<?php echo $keep_in; ?>" class="on">お気に入り</a></li>
<?php endif; ?>

ページをリロードさせない為にiframeを使用していますが他に良い方法がありましたらご教授頂けないでしょうか？

又、同じページの最下部に「お問合せ」というアンカーリンクを設置し
お問合せ入力フォームへ遷移してお気に入りに追加したクッキーの配列を取得したいのですがその際は下記のように行うと宜しいでしょうか？

if (isset($keeplist)) {
foreach ($keeplist as $val) {
echo $val;
}
}

何度も申し訳ございません。
どうぞ宜しくお願い致します。

[miconear]

昨日記述したコメントの件ですが解決することが出来ました。
プログラムの流れやCookieの仕組みを理解できていたらスムーズにいきそうですね。
ベストアンサーありがとうございます。

[ArimaRyunosuke]

解決済みのようですが、unserialize にユーザ入力を渡すのはわりかし危険です。

http://php.net/manual/ja/function.unserialize.php#refsect1-function.unserialize-notes
http://blog.a-way-out.net/blog/2014/07/22/php-object-injection/

攻撃難度は高めですし、すぐさま脆弱性になるわけではないですが、json などで代替できるならそちらの方がよろしいかと思います。

[ngyuki]

確かに `unserialize($_COOKIE["keeplist"]);` はものすごい脆弱性になりかねないのでやらない方がいいです。

[miconear]

ArimaRyunosuke様、ngyuki様、補足コメントありがとうございます。

早速、参考URLを拝見しました。
将来、脆弱性になりかねないですね・・・（汗）

jsonで代替するとなると、このようにして使用したら良いでしょうか？

誤）setcookie("keeplist", serialize($keeplist), time()+60*60*24*30, "/");

誤）$keeplist = unserialize($_COOKIE["keeplist"]);

正）setcookie("keeplist", json_encode($keeplist), time()+60*60*24*30, "/");

正）$keeplist = json_decode($_COOKIE["keeplist"]);

[ngyuki]

実際にやってみればわかると思いますが json_decode の第２引数に true を指定しないと stdClass になってしまうので上手く動きません

[miconear]

ngyuki様
再補足ありがとうございます！
true有りで配列ということですね。
最終的にこうなりました。
$keeplist = json_decode($_COOKIE["keeplist"], true);

Answer 2

ごめん、余計なことした、cookie のときは [] 使えないんだね

	if( isset( $_GET[ 'keep_in' ] ) ) {
		if( array_key_exists( 'KeepID', $_COOKIE ) && ( $p = array_search( $_GET[ 'keep_in' ], $_COOKIE[ 'KeepID' ] ) ) !== false ) {
			// Delete value from cookie.
			setcookie("KeepID[{$p}]", '', time() - 3600, '/');
			unset($_COOKIE['KeepID'][$p]);
		} else {
			// Add value in cookie.
			$value = $_GET[ 'keep_in' ];
			setcookie("KeepID[{$value}]", $value, time() + 2592000 /*60*60*24*30*/, '/');
		}
	}
	
	var_dump( $_COOKIE );

Comments

[miconear]

ご回答ありがとうございます。m(__)m

Cookieの場合は[]配列記述が使用できないのですね。
必ず配列キーを設定しなくてはいけないのですね(^o^;

array_key_exists()やarray_search()は様々なシーンで使えそうですね。

miya様、ngyuki様　御二方のご教示により上手く解決することが出来ました。
両者ともベストアンサーにしたいのですがシステム上、それは無理そうですね？(^o^;
今回はシンプルな方法をご教示頂いたngyuki様をベストアンサーにしたいと思います。（miya様、大変申し訳ございませんm(__)m）

setcookie("KeepID[{$value}]", $value, time() + 2592000 /*60*60*24*30*/, '/');

最後に気になったのですが
こちらのsetcookieで有効期間が60*60*24*30の答えが秒数で記述されていますが
答え(2592000)を記述することによって処理速度も向上するのでしょうか？

Answer 3

ごめんなさい、逆でした

array_search( $_GET[ 'keep_in' ], $_COOKIE[ 'KeepID' ] )

setcookie で表示されたエラーは、array_search のエラー出力が原因です。

Comments

[miconear]

再度ご回答有難う御座います！(^^)

下記のようにif(isset($_COOKIE['KeepID']) してクッキーがセットされているか分岐させてみました。

// お気に入りクリック
if(isset($_GET['keep_in'])) {

// クッキーがセットされているか
if (isset($_COOKIE['KeepID'])) {

if(array_key_exists($_GET['keep_in'], $_COOKIE['KeepID']) && ($p = array_search($_GET['keep_in'], $_COOKIE['KeepID'])) !== false) {
echo $ke = array_search($_GET['keep_in'], $_COOKIE['KeepID']);
// Delete value from cookie.
setcookie("KeepID[{$p}]", '', time() - 3600, '/');
unset($_COOKIE['KeepID'][$p]);

} else {
// Add value in cookie.
setcookie('KeepID[]', $_GET['keep_in'], time() + 2592000 /*60*60*24*30*/, '/');
}

} else {
// Add value in cookie.
setcookie('KeepID[]', $_GET['keep_in'], time() + 2592000 /*60*60*24*30*/, '/');
}
}

早速、「お気に入り追加」を下記の順番でクリックしていきました。

【HTML】
<a href="/favorite.php?keep_in=10001">お気に入り</a>
<a href="/favorite.php?keep_in=10002">お気に入り</a>
<a href="/favorite.php?keep_in=10003">お気に入り</a>
<a href="/favorite.php?keep_in=10004">お気に入り</a>

10004～10004迄それぞれクリックして、foreach文で配列から取り出して確認してみましたら10004のみしか取り出せませんでした。

foreach ($_COOKIE["KeepID"] as $value) {
echo $value . "<br>";
}

10001
10002
10003
10004
と言う様にクリックした分を配列に追加して
再度同じ番号がクリックされたら該当する番号を削除したいです。

すみません、何度もお手数かけますが
どうぞ宜しくお願い致します！m(__)m

Answer 4

if ( isset( $_GET[ 'keep_in' ] ) ) {
  if ( ( $p = array_search( $_COOKIE[ 'KeepID' ], $_GET[ 'keep_in' ] ) ) !== false ) {
    // Delete value from cookie.
    setcookie( "KeepID[{$p}]", '', time() - 3600, '/' );
    unset( $_COOKIE[ 'KeepID' ][ $p ] );
  } else {
    // Add value in cookie.
    setcookie( 'KeepID[]', $_GET[ 'keep_in' ], time() + 2592000/*60*60*24*30*/, '/' );
  }
}

KeepID[] = ～ ってすると自動で連番振って新しい要素追加してくれる。
お気に入りとかもデータベース管理したほうがいいと思うけど、登録とかしなきゃいけなくなるからね
俺なら場合によっては登録以外の方法取るけど

Comments

[miconear]

miya様、ご回答ありがとうございます！

早速、miya様の方法で記述してみたところ下記のエラーが表示されてしまいました。

【HTML側】
<html>
<head>
<body>
<a href="/favorite.php?keep_in=10001">お気に入り</a>
<a href="/favorite.php?keep_in=10002">お気に入り</a>
</body>
</html>

【PHP側】

<?php
if ( isset( $_GET[ 'keep_in' ] ) ) {
if ( ( $p = array_search( $_COOKIE[ 'KeepID' ], $_GET[ 'keep_in' ] ) ) !== false ) {
// Delete value from cookie.
setcookie( "KeepID[{$p}]", '', time() - 3600, '/' );
unset( $_COOKIE[ 'KeepID' ][ $p ] );
} else {
// Add value in cookie.
setcookie( 'KeepID[]', $_GET[ 'keep_in' ], time() + 2592000/*60*60*24*30*/, '/' );
}
}
?>

▼ 下記エラーになります。

if ( ( $p = array_search( $_COOKIE[ 'KeepID' ], $_GET[ 'keep_in' ] ) ) !== false ) {

この行で下記のエラーが表示されました。
「Notice: Undefined index: keepID in～..」
「Warning: array_search() expects parameter 2 to be array, string given in～..」

setcookie( "KeepID[{$p}]", '', time() - 3600, '/' );

こちらの行で下記のエラーが表示されました。
「Warning: Cannot modify header information - headers already sent by (output started at～..」

[miya]

あ、ここに書けばいいのかｗ

ついでにもう少し修正

```lang-＜PHP＞
if ( isset( $_GET[ 'keep_in' ] ) ) {
if ( is_array( $_COOKIE[ 'KeepID' ] ) && ( $p = array_search( $_COOKIE[ 'KeepID' ], $_GET[ 'keep_in' ] ) ) !== false ) {
// Delete value from cookie.
setcookie( "KeepID[{$p}]", '', time() - 3600, '/' );
unset( $_COOKIE[ 'KeepID' ][ $p ] );
} else {
// Add value in cookie.
setcookie( 'KeepID[]', $_GET[ 'keep_in' ], time() + 2592000/*60*60*24*30*/, '/' );
}
}
```

あのままだと一発目コケるからね

[miya]

is_array( $_COOKIE[ 'KeepID' ] ) ⇒ array_key_exists( 'KeepID', $_COOKIE ) か