特定行の削除

###前提・実現したいこと
特定行削除したい。

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

エラーはとくにありませんが１行目しか消えない状態です。
また、色々試していますが、連続チェックだと消せたり
１行目３行目と飛んでチェックつけると消えません。

###該当のソースコード

PHP
1for($i=0; $i<count($this->request['a']); $i++)
2					
3if($this->request['a'][$i]== "ok")

###試したこと
html側のvalueは「"ok"」になってます。

###補足情報(言語/FW/ツール等のバージョンなど)
初心者で情報不足でしたら申し訳ありません。
よろしくお願い致します。

kei344

2016/08/17 06:52

回答が付いた質問の編集は慎重に行ってください。回答でタイプミスについての指摘がある場合は「直したこと」がわかるようにしてください。

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

回答3件

ところで、

for($i=0; $i<count($this->request['checkbox']); $i++)

の意味は、理解されていますか？

プログラムが一部だけなので、推測部分が多いですが、count($this->request['checkbox']) は、チェックをつけた数しか返さないと思います。

この for ループでは、データの先頭から、チェックボックスをチェックした数しかループ処理されません。

例えば、チェックボックスが、5行あったとして、1行目と5行目にチェックした場合、チェックが2個なので、1行目と2行目しかループで、処理されません。

count の部分を「全行のデータがある項目」にしたら、想定する動作をするようになると思います。

投稿2016/08/17 03:51

CHERRY

総合スコア25171

eatomochi

2016/08/17 04:39

ご回答頂きありがとうございます。ご指摘の通り$this->request['checkbox']の部分が違っていたようです・・・こちらを修正すると無事思い通りの動きになりました。丁寧にご説明いただきありがとうございました。

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

ここのif文が正しく処理されていないような気がします。

PHP
1if(false == $rs = mysql_query($sql))

このようにしたらどうでしょうか？

PHP
1$rs = mysql_query($sql);
2if($rs === FALSE){

投稿2016/08/17 03:53

ttyp03

総合スコア16998

ぱっとみて思ったことは、$i++の位置正しいでしょうか？
for文が繰りかえす度に1ずつ増えるようにしているのに、最初のifに入った時にさらに増えるということになっているので、チェックによって動きが安定しないのかなと思いました。

言いたいことは以下の通りです。

PHP
1//ここの$i++でforが１回まわるたびにiが増える設定をしている
2for($i=0; $i<count($this->request['checkbox']); $i++)
3                {
4
5                    if($this->request['checkbox'][$i]== "ok")
6                    {
7
8                        $sql = "DELETE
9                                        FROM
10                                            YASAI
11                                        WHERE
12                                            kyuuri='".$this->request['kyuuri'][$i]."';";
13
14　　　　　　　　　　　　　　　//ここの書き方も気になる・・・これ大丈夫ですかね？？
15                        if(false == $rs = mysql_query($sql))
16                        {
17                            return false;
18                        }
19               　　　　　//ここでさらにiが+1される（最初のifでtrueの時のみ増える)のでいらない？？
20                        $i++;
21                    }