レコードの中の文字列を分解して条件文に使用しSQLを実行したいです。

Question

前提・実現したいこと

該当のソースコードの配列の値はカンマ区切りで二つの値を格納しています。

For文で回しながらこの値を分割してデリート文の条件文の値として使用してSQLを実行したいです。
良い方法がありますでしょうか？
宜しければコードを教えて頂けたら幸いです。

質問ががさつでお手数おかけしますが、よろしくお願いいたします。

DB
テーブル名：Employees
ユニークキー：enp_number,id
|enp_number|id|name|
|100|1|田中|
|101|1|山田|
|112|2|渡辺|
|121|3|佐藤|

該当のソースコード

//DBHelperに書かれています。
public function deleteSQL (){ 
		return "delete from Employees ";
	}

//配列 値 = enp_number,id　１要素につき文字列がカンマ区切りで格納されています。　取り出した後、カンマ区切りで分けて使えないか考えています。
 $daleteStorage = array(
[0] => "100 , 1",
[1] => "112 , 2",
);

//SQL作成 For文で回して値を使用して行きたいです。
 $literalStorage = null;
		$sql = $dbHelper->deleteSQL();		
			$sql = $sql . " where enp_number = :enp_number";
// AND id = '' ?
			$literalStorage = array(
						':enp_number' => 実際の値?);
}
//実際にSQLの実行する所
$db = $dbHelper->SQL($sql, $literalStorage);
if(!$db){
			error($dbHelper, 'Employees');
		}

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

NetBeans
MySQL
PHP5.6

Comments

[m.ts10806]

コードが散らかっています。インデントもそうですが、dbHelperクラスの定義がなかったり、deleteSQL()で呼び出しているがdelete ()となっていたり。$daleteStorageも定義はされているものの使われた形跡がありません。1つのコードブロックに全てまとめなくても良いと思うので、コードを整理してください。

[m.ts10806]

ちなみに「文字列の分割」については何か調べましたか？

[xss]

mts10806さん回答ありがとうございます。散らかって申し訳ございません。修正していきます。　文字列の分割についてはsplit、explodeがあることを知りました。

[m.ts10806]

インデントも調整してください。どこにも開始がない } が不意に置かれています。DBHelperに置かれているのであればなおさらコードは別ブロックにした方が良いです。同じコード内に全部書いてあるわけじゃないですよね？

[m.ts10806]

まずは使ってみましょう。splitは非推奨になりPHP7では削除されたものですね。利用されているPHPのバージョンもできれば追記してください。

[m.ts10806]

あとはやはりDBHelperの定義がないとdbHelper->SQLにどのような状態で渡せば良いのかわからないので、具体的な回答にはつながらないと思います。ざっくりの回答で「あとは参考にしてください」で良ければ回答してみますが、いかがいたしましょう？

[xss]

mts10806さん、ご返信ありがとうございます。私自身の知識不足もあり、今回の質問として準備不足でした。ご指摘感謝いたします。dbHelperからFROM句までのSQLを持ってきて別のクラスで持ってきたSQLにWHERE句以降を作成し最後はDBHelperで実行する流れになります。PHPのバージョンは5.6です。回答を頂きたいです。よろしくお願いいたします。

Answer 1

foreachで回しつつ1つずつexplode()して実行するように組むと良いかと思います。

※コメントでも書いたようにdbHelperの定義などが分からないため、参考程度のコードです。そのままでは動かない部分もあると思うので、あとはご自身で現状に合わせて調整してください。

$daleteStorage = array(
    [0] => '100 , 1',
    [1] => '112 , 2',
);

foreach($daleteStorage as $dids){
    $did_array = explode(',',$dids);

    $sql = 'delete from Employees ';
    $sql .= ' where enp_number = :enp_number and id = :id';
    $literalStorage = array(
        ':enp_number' => trim($did_array[0]),
        ':id' => trim($did_array[1])
    );
    $db = $dbHelper->SQL($sql, $literalStorage);
    //エラー処理についてもerror()の仕様が分からないので省略
}

trimしたのは100[ ],[ ]1 [ ]に空白があるためです。
おそらく整数型でDBでは定義されているので、空白が入ると文字列となってエラーとなります。

今回foreachにしましたがforでも不可能ではないです。

$strorage_count = count($daleteStorage);
for($i=0;$i<$strorage_count;$i++){
    $did_array = explode(',',$daleteStorage[$i]);
    //以下同

もし、何回もSQLを実行するのが面倒であれば、下記でもいけるかもしれません。
※dbHelper::SQLの作り次第です。

$sql = '';
foreach($daleteStorage as $n=>$dids){
    $did_array = explode(',',$dids);

    $sql .= 'delete from Employees ';
    $sql .= ' where enp_number = :enp_number'.$n.' and id = :id'.$n.';';
    $literalStorage[':enp_number'.$n]=trim($did_array[0]);
    $literalStorage[':id'.$n]=trim($did_array[1]);
}
$db = $dbHelper->SQL($sql, $literalStorage);

Comments

[xss]

mts10806さんご回答ありがとうございます。大切にご参考にさせて頂きます。ご教授ありがとうございます。

[m.ts10806]

一気に色々とやりすぎているように思うので要件（やりたいこと）を1つ1つ分解してそれぞれ確実に仕上げていくと良いです。

[xss]

mts10806さんご返信ありがとうございます。mts10806さんのアンサーを使わせて頂いて私が求めた結果を実現することが出来ました。大変感謝しております。
質問の正確性の無さや一つの質問にたくさんの要件が存在してしまい、問題解決に対して複雑で難しいものになってしまいました。

今回はmts10806さんに助けられましたが、ひとつひとつ問題を解消し確実に仕上げて行きたいと反省しました。

本当にありがとうございました。

Answer 2

そもそも、$daleteStorageのような配列は、多少を調整を加えても

• PHP 7→文法エラー
• PHP 5系列→同じオフセットに複数の値を入れようとしているため、後の値しか残らない

ということになり、正しくデータを取り出せません（3v4l）。まずは、まっとうに使える形の配列を作ってください。

Comments

[m.ts10806]

そこも含めてコードがかなり散らかってますよね。
たぶんこう書きたかったんでしょうけど。
$daleteStorage = array(
[0] => "100 , 1",
[1] => "112 , 2",
);

[xss]

maisumakunさんご回答ありがとうございます。ご参考にさせて頂きます。
質問内容の不備が多々あります・・申し訳ないです。

Answer 3

delete from Employees where 0
or (enp_number=100 and id=1)
or (enp_number=112 and id=2)

のようになるよう処理を入れればよいでしょう

Comments

[xss]

yambejpさんご回答ありがとうございます。ご参考にさせて頂きます。