ネットショップの運営に困っています

Question

はじめまして。
どうしてもわからないことがあり質問致しました。
どなたか解決策が分かる方、知識をご教授願います。

現在私は会社のネットショップを始めました。
商品を買い物かごに複数入れたとき、それぞれの商品についている削除ボタンを押したら、その商品だけリストから消えるようにしたいのですが、いまの状態では削除ボタンを押すと買い物かごのすべての商品が消えてしまいます。
是非、解決策を教えていただきたいです。

<概要>
それぞれの商品ページから商品idのみPOSTでcart.phpに送り、それを商品情報が記載されているdata.csvと比較して、商品情報をショッピングカートに表示させています。そのショッピングカートの一つ一つの×印に対応する商品idをGETでdelete.phpに送って、unsetで消去しようとしています。

日本語が下手で申し訳ないですが、どなたか解決策がわかる方がいましたら、何とぞよろしくお願い致します。

20091110 · Accepted Answer

たびたび失礼します。
文字コード、UTF-8ではだめでしたか。。
WindowsServerなんでしょうか。
「能　表　申　十」などの文字が化けなければSJISでもいいのかも。


今日はまとまった時間が取れたので
「商品を買い物かごに複数入れたとき～」の件をじっくり見てみました。
商品を消去して戻ってくると全部消えるのはPOSTデータが原因ですね。
ためしにPOSTの処理と画面表示の処理を分けてみたらうまくいきました。
自分でやってみたファイルをまとめますので、よろしければご確認ください。
※ページ遷移の順番が変わっていますが、
フォームのファイルタイプは.phpにせず.htmlのままと仮定したリロード対策です。
あと、POSTやGETの値の保持はフォームで送った次のページまでです。

参考URL：http://uploadr.net/file/b410902745

bm_hiro · Answer

削除して cart.php にリダイレクトした時と、
「カートの中身」ボタンを押した時に
「空っぽ」の表示になる理由の見当が付きました。

カートに商品を入れたときしか $id = $_POST["select"]; が ないですよね？

で、ここ

if(strcmp($id,$data[0]) == 0){

$id の中身がないので、これ以降は実行されていません。

bm_hiro · Answer

#5です。
---- ---- ---- ---- ---- ---- ---- 
> いきなり長いコードまで記載してしまって申し訳ないです。

長さもさることながら可読性に難があるように感じたのと、
strcmp() を使う必要性に疑問を持った事で、意識が保てなかっただけです。
今後、商品が増えたりした場合の拡張性を考えると、これは ちょっと。。。と思います。
俺なら どうするかと言うと、何通りか浮かびますが、とりあえず、配列使いますね。

//以下商品名からその商品が掲載されているページに飛ばすための処理
・
・
//ここまで

↑の部分の事です。
---- ---- ---- ---- ---- ---- ---- 
あと #6 の 補足の

>下記の方がおっしゃっているように最終確認画面まで行くと、
> 削除した商品は表示されず確かにSESSIONの中身は削除されているみたいですが。。

これは、俺の書き方悪かったですね。
最初、テキトーにぽんぽんカートに入れたり削除したりしてたもんで、
気がついたのが最終画面だったってだけです。
$_SESSIONの中身は 常に正常な状態だと思います。
カートの中身に表示されないだけで。

 昨日、俺が書いた再現実況でも、途中のも ちゃんと削除されて表示されてましたよ。
---- ---- ---- ---- ---- ---- ---- 
> ＜同じ商品の個数が増えていかず上書きされる＞

んー、cart.php の前半だけ見た感じですけど、
一つの商品に対して 一つの$_SESSION["item"][$id] しか持ってないですよね？
既に カートの中にある商品だったら 加算するとかの処理は 全く見当たりません
でした。

$file = fopen("data/data.csv" , "r"); 
while($data = fgetcsv($file)){
if(strcmp($id,$data[0]) == 0){
$price_n = (int)$data[2];
$sum_n = $price_n * $num_n;

$_SESSION["item"][$id] = array("$num_n" , "$data[1]" , "$price_n" , "$sum_n");

↑の部分。

data.csv を総当りで、カートに入れた品物の情報を取り出してきて、
合計額に 単価 * 数量 を追加して、
その商品の情報などを $_SESSION["item"][$id] に 配列で代入。
当然、既にデータが入っていた場合は、無残にも上書きされる。。。と。。
---- ---- ---- ---- ---- ---- ---- 
> ＜個人情報の入力画面＞

一番、手抜きなのは 入力されているか どうかだけの判断をするもので済ましてるところもあります。
厳密にやるなら、メルアドや電話番号の正当性、数字しか入らない所に全角文字が入っているとか、それなりのチェックが必要です。
---- ---- ---- ---- ---- ---- ---- 
> ＜データが正常に表示できていない＞

これに関しては、あとで 見てみます。
眠くなってきたんです。ごめんなさい。
---- ---- ---- ---- ---- ---- ---- 
あと文字コードは パソコン向けなら UTF-8のほうがいいですよ。
SJISだとバグ文字対策とかしなきゃいけなくなりますから。

20091110 · Answer

回答番号No,4 とNo,6を投稿したものです。連投すみません。商品がすべて消えてしまう現象について、 cart.phpでも$_SESSION["item"][$id]と記述されていますので、ファイルの先頭にsession_start()を書き加えてみてください。【cart.php】のの一部などブラウザによっては正確にデータを送ることができないかもしれませんのでLet'sご確認＆修正。

20091110 · Answer

回答番号：No.4です。

文字化け、直りませんでしたか…結果画面のソース内でも化けていますか?
自分も貴サイトのソースをお借りして検証してみました。
※ソースを無断使用してごめんなさい。

ファイルの文字コードはすべてUTF-8とのことなので
検証用のファイルの文字コード、キャラセットともにUTF-8にあわせました。

結果、cart.php内のHTML-MetaのキャラセットをUTF-8にして出力してみたところ文字化けが直りました。

一連のソースをまとめてZipにしましたので、お手数ですがご確認いただけたらと思います。
(ウィルスチェック済みですが、DL後解凍する前にもチェックすることをお勧めします)

参考URL：http://uploadr.net/file/bccd5ee60a

bm_hiro · Answer

どうも、はじめまして。最初にこの質問を見たときにそちらのサイトのほう見て疑問に思った事があったのですが、 #1でそちらの提示されたソースを見て、ゲンナリしてしまいましたので、回答を保留していました。説明が下手クソなので、言い方が回りくどくなるのと、根本的に学がないのとで無駄に長くなるのと、口が悪いのは、ご了承ください。まだ解決していないようでしたので、再度、先ほどそちらのサイトを見て、新たに気がついたことがありましたので、お知らせします。 > var_dump()みたいなので調べてみたら、本来「柿下人麻呂」とかって出ないといけないのに文字化けしているんですよ・・・ var_dump()したのならお気づきになられたかもしれませんが、 $_SESSION の中では、データは「正常に削除したものだけ削除されている」ように見受けられました。ちなみに、文字化けの原因は data.csv が UTF でサイトが SJISなのでおかしな事ではありません。以下に、上記の根拠を説明します。 ----- ----- ----- ----- ----- ----- ----- ----- まず、カートが空っぽの状態で商品A,B,C の３商品をカートに入れる。 ↓ カートの中身が表示されている画面で、右下の「カートの中身」っていうリンクをクリック。 ↓ なぜか、「空っぽ」に表示される。 ← 疑問点その１ ↓ めげずに、ブラウザの戻るボタンを押し、商品 D を追加する。 ↓ ちゃんと、商品 A、B、C、D がカートの中身として表示される。 ↓ この状態で、商品 B を [x] をクリックして削除する。 ↓ またまた「空っぽ」に表示される。 ↓ 再度、折れそうな心を奮い立たせブラウザの戻るボタンを押し、商品 E を追加する。（ここでふと気がついた。「個数の選択」に５が存在しねぇぇぇぇぇぇ）　　 http://shusoan.com/shu_so_an/shu_so_an2.html 　　原因は最後の > がない。　　

20091110 · Answer

>本来「柿下人麻呂」とかって出ないといけないのに文字化けしているんですよ・・・

この部分だけなんですが気になることが。
読み込むCSVファイルの文字コードがEUC-JPやUTF-8Nになっていませんか？
Shift-jisで作られているようなので、CSVファイルの文字コードもShift-jisにあわせて試してみてください。

rakujin · Answer

該当HPに飛んで、少し、ソースコードを見てみました。
　delete.php のソースが解りませんので、どのよう操作をしているのか不明ですが・・・。

idの価（商品のコード）を持たして、delete.phpに送って、そのidに対応した商品を削除するんですよね。組み立てとしては、それで合っていると思いますよ。

しかし、ローカルではうまく動いていてサーバーに移動したら期待した動きにならないというのもおかしな話ですね。

NeilMania · Answer

概要で書かれているので、そのまま組めばいいと思うのですが
どこがわからないか書いた方がよいかもしれませんね。
同じ商品を購入した場合、買い物かごの中に
数が増えていかないのも気にはなりますが…。

inu2 · Answer

>>

それぞれの商品ページから商品idのみPOSTでcart.phpに送り、それを商品情報が記載されている data.csvと比較して、商品情報をショッピングカートに表示させています。そのショッピングカートの一つ一つの×印に対応する商品idをGETで delete.phpに送って、unsetで消去しようとしています。
<<

あなたの言葉がそのまま「仕様」になっていますよね？
その通り組めばよいだけでは？

文章そのままのロジックを具体的にどう書けばよいのか分からない。
というのであれば、逆宣伝している状態ですよ。
つまり。「この程度の事が作れないようなスキルの人が組んだサイトで買い物したくない、セキュリティ的にあぶなっかしくてココでは買い物できない」って思わせてしまうことになっていますよ。

ネットショップの運営に困っています

たびたび失礼します。

この回答への補足

削除して cart.php にリダイレクトした時と、

この回答への補足

#5です。

回答番号No,4 とNo,6を投稿したものです。

回答番号：No.4です。

この回答への補足

どうも、はじめまして。

この回答への補足

>本来「柿下人麻呂」とかって出ないといけないのに文字化けしているんですよ・・・

この回答への補足

該当HPに飛んで、少し、ソースコードを見てみました。

この回答への補足

概要で書かれているので、そのまま組めばいいと思うのですが

この回答への補足

>>

この回答への補足

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング

　該当HPに飛んで、少し、ソースコードを見てみました。