【PHP7】カート機能のカートに入れる、カートにデータを保持する、カート内の商品削除の処理

前提・実現したいこと

質問内容
PHP7にて、簡易的なショッピングサイトを作ろうとしています。
その中で、カート機能でつまづいています。
商品は全て１つずつしか売らないようにしているので、個数指定の設定は必要ないかなと考えております。
問題点は、商品の詳細画面(item1~3.php)から「カートに入れる」を選択後にカートページ(cart-items.php)に商品を追加できるようにすること、別の商品（２つ目以降の商品）を「カートに入れる」を選択してカートページに２つ目以降の商品が追加できるようにすること（最後にカートに入れた商品以外の商品がカートページから消えることなく）、あとカート内から商品を削除することです。

ページ遷移してもデータを保持できる$_SESSION(SESSION変数？)と、データを別ページに移せる＄＿POSTを使えばなんとかできるんじゃないかなって思っていたのですが、なかなかうまくいきませんでした。カートに入れる処理は、$_SESSIONを配列化して、そこに$_POSTで送られてきたデータを入れたらできるんじゃないかと思ったのですが、データが残りませんでした。

どなたか教えてくれると嬉しいです。

発生している問題・エラーメッセージ

エラーメッセージはないが、商品が１つしか出力されない。（戻るを押して別の商品をカートに入れても前に入れた商品が存在しない）。それ以降の商品を削除する処理は、データが保持されないのでできない。

エラーメッセージ

該当のソースコード

php
1(home.php)
2<?php ?>
3<!DOCTYPE html>
4<html>
5    <head>
6        <meta charset="utf-8">
7        <title>Cart Practice</title>
8    </head>
9    <body>
10        <ul>
11            <li><a href="item1.php">item1</a></li>
12            <li><a href="item2.php">item2</a></li>
13            <li><a href="item3.php">item3</a></li>   
14        </ul>
15    </body>
16</html>
17

php
1(item1.php)
2<?php ?>
3<!DOCTYPE html>
4<html>
5    <head>
6        <meta charset="utf-8">
7        <title>Cart Practice</title>
8    </head>
9    <body>
10        <p>Item1</p>
11        <form action="cart-items.php" method="post">
12            <input type="submit" name="btn" value="Add to cart">
13            <input type="hidden" name="item" value="item1">
14        </form>
15
16    </body>
17</html>
18

php
1(item2.php)
2<?php ?>
3<!DOCTYPE html>
4<html>
5    <head>
6        <meta charset="utf-8">
7        <title>Cart Practice</title>
8    </head>
9    <body>
10        <p>Item1</p>
11        <form action="cart-items.php" method="post">
12            <input type="submit" name="btn" value="Add to cart">
13            <input type="hidden" name="item" value="item2">
14        </form>
15
16    </body>
17</html>

php
1(item3.php)
2<?php ?>
3<!DOCTYPE html>
4<html>
5    <head>
6        <meta charset="utf-8">
7        <title>Cart Practice</title>
8    </head>
9    <body>
10        <p>Item1</p>
11        <form action="cart-items.php" method="post">
12            <input type="submit" name="btn" value="Add to cart">
13            <input type="hidden" name="item" value="item3">
14        </form>
15
16    </body>
17</html>

php
1(cart-items.php)
2
3<?php 
4    session_start();
5    $_SESSION['cart']=[];       //SESSION変数cartを配列化
6    $_SESSION['cart'][]=$_POST['item'];     //変数cart内にカートに入れようとしたアイテムを入れる
7       
8                            
9
10?>
11<!DOCTYPE html>
12<html>
13    <head>
14        <meta charset="utf-8">
15        <title>Cart Practice</title>
16    </head>
17    <body>
18        <h1>All items</h1>
19        <?php foreach($_SESSION['cart'] as $item):?>        <!--配列化されたSESSION変数内の商品を１つ１つ出力したいが一つしか出力されない。（データが残らない）-->
20            <h2><?php echo $item?></h2>
21        <?php endforeach?>
22    </body>
23</html>
24

試したこと

できるだけシンプルに書こうとしたため（今後も使うときがあると思うので感覚的に覚えやすいように）、$_POST,$_SESSION, とforeach文で取り組んでみたのですが、データが残らずに苦戦しています。
カート内の削除の処理に関しては、カート内にデータが残らないので、カート内の商品削除のしようがありませんでした。

補足情報（FW/ツールのバージョンなど）

PHP７

m.ts10806

2020/02/10 02:05

>商品の詳細画面(item1~3.php) 商品毎に画面作ってしまっている時点でPHPのメリットほとんど活かせてないですが、そのあたりは大丈夫でしょうか。 1万点あったら1万ファイル作るつもりですか？

PonCotsu

2020/02/10 03:52

返信ありがとうございます。そこはforeach文で追々修正しようと思っています。とりあえずカート機能だけを明確に見せようとした結果、簡易的な形のページをつくるに至りました。

m.ts10806

2020/02/10 03:59

ん。foreachにはならないですよ。あと設計なしに作りながらしてると思想も統一されないしコードもスパゲッティになって後戻りできなくなるので、決めてからやった方がいいです。絶対。

行動規範の内容に同意します

回答1件

ベストアンサー

ここだけかはわからないですが、
$_SESSION['cart']=[]; //SESSION変数cartを配列化
でカートを毎回クリアしちゃってるので、最後に追加した分しか$_SESSION['cart']に格納されていません。

常に初期化するのではなく、
isset($_SESSION['cart'])で存在チェックをして$_SESSION['cart']が存在しない時だけ初期化するなどの修正が必要です。

投稿2020/02/10 01:38

tanat

総合スコア18713

PonCotsu

2020/02/10 05:12

回答ありがとうございます。早速試したところ、カートに複数のitemが入ることができました。ただ一つ疑問が浮かび上がってきました。 session_start()の後に isset($_SESSION['cart])で条件分岐をして、trueになったときにカートに商品を入れるようにしたんですけれども、元々$_SESSION['cart']の定義はしてなかった（未定義）だったにも関わらずisset関数でtrueが返ってくるのはどうしてなんでしょうか。(下のような処理でtrueが返ってきました) <?php session_start(); if (isset($_SESSION['cart'])){ $_SESSION['cart'][]=$_POST['item']; }else{ $_SESSION['cart']=[]; } ?>

tanat

2020/02/10 05:20

<?php session_start(); var_dump($_SESSION); if (isset($_SESSION['cart'])){ $_SESSION['cart'][]=$_POST['item']; }else{ $_SESSION['cart']=[]; } という感じで本当に未定義なのか確認してみてください。 $_SESSIONはリクエストを跨いでデータを持ち越すための仕組みなので、今回のケースだと過去に一度でもこのページにアクセスした場合はカートに投入しなくても $_SESSION['cart']=[]; によって変数自体は定義されることになります。 cookieを削除したり、chromeのシークレットモードでアクセスすると、初めてアクセスした場合の挙動を確認することが出来るので試してみてください。（この辺りは理解出来るまで単純にsessoin_start()と$_SESSION['適当なキー']だけを使う様なテストページを作って確認してみることをお勧めします）

PonCotsu

2020/02/10 18:38

if (isset($_SESSION['cart'])){ $_SESSION['cart'][]=$_POST['item']; }else{ $_SESSION['cart']=[]; $_SESSION['cart'][]=$_POST['item']; } 初めてカートに入れる時だけ、$_SESSION['cart']に空白の配列を入れるようにして、それ以降はひき続きカートに追加できるように条件分岐して、ページ遷移後もカートへ入れることができる処理ができました。教えていただきありがとうございました。 SESSIONについて調べたところ、ブラウザを閉じるとSESSION内に入っているデータは消えてしまうとあったのですが、もしiPhoneなどの携帯からこのサイトに訪れてカートにいくつか商品を入れて、その後ブラウザを閉じるとまた１から商品を入れないといけないのでしょうか。もしそうならば、僕のパソコンでのCookieによるデータ残り（ブラウザを閉じてもCookieを消さないとデータが残る）とネットに書かれているSESSIONの定義？効果？が矛盾しているように思うんですけど、そこはどう認識すべきですか。それとも僕がまだSESSION破棄に関する処理を書いていないのが問題なんですかね。何度も質問してしまいすみません。。。

tanat

2020/02/11 01:35

> このサイトに訪れてカートにいくつか商品を入れて、その後ブラウザを閉じるとまた１から商品を入れないといけないのでしょうか。基本的にはそうです。 > 僕のパソコンでのCookieによるデータ残り実はブラウザが勝手に頑張っています http://www.sakataya.sakura.ne.jp/wp/?p=34 カートの中身を保持したい場合、sessionでは無くcookieにデータを保存したり、カートIDのようなものをcookieに格納してカート内容をDBに保存したりする仕組みが必要になってきます

PonCotsu

2020/02/12 04:06

Chromeの設定を変えて変えてやってみましたが、カートにはデータが残った状態でした。これはSESSION破棄の処理を書いていないからというもので、よろしいですか。

tanat

2020/02/12 04:17

この場合、session破棄の処理は関係ないです。 chromeの設定ミスか検証方法のミスだと思われます。これまで一回も使ったことの無いブラウザで確認してみるのが確実です。

PonCotsu

2020/02/12 13:38

わかりました。色々教えていただきありがとうございました。

行動規範の内容に同意します