条件に合致したカラムを抽出して処理を分ける

Question

下記コードで該当する日付（holiday）を「削除」か「登録」のif文に入れて「削除」か「登録」の処理を行いたいのですが行き詰まっております。複数の日付の登録はできますが、複数の日付の削除はできません。単体はどれもできます。
たぶんif文に問題があると思いますが、どう修正すればよいかどなたか教えて頂けますでしょうか。

		$paraholiday = explode(',',$para['holiday']);
		$ct2 = 0; 
				foreach ($ret as $h){
			$is_deleted[$ct2] = $h['is_deleted'];
			$ct2 += 1; 
			if(count($paraholiday) > 1 && array_search("y", $is_deleted) !== FALSE){
		登録処理
			}else if(count($paraholiday) > 1 && array_search("n", $is_deleted) !== FALSE){
		削除処理
			}
		}
		

DB

-- Table "calendartbl" DDL

CREATE TABLE `calendartbl` (
  `holiday` date NOT NULL DEFAULT '0000-00-00' COMMENT '休日年月日',
  `is_deleted` char(1) NOT NULL COMMENT '削除フラグ（y, n）',
  PRIMARY KEY (`holiday`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8 COMMENT='休日';

var_dump

var_dump($h);
array(2) {
  [0]=>
  string(1) "y"
  ["is_deleted"]=>
  string(1) "y"
}

var_dump($ret);
array(2) {
  [0]=>
  array(2) {
    [0]=>
    string(1) "n"
    ["is_deleted"]=>
    string(1) "n"
  }
  [1]=>
  array(2) {
    [0]=>
    string(1) "n"
    ["is_deleted"]=>
    string(1) "n"
  }

var_dump($paraholiday);

array(2) {
  [0]=>
  string(10) "2016-02-01"
  [1]=>
  string(10) "2016-02-02"
}

var_dump($is_deleted);

array(2) {
  [0]=>
  string(1) "n"
  [1]=>
  string(1) "n"
}

Answer 1

まずはif文の中にechoで数字でも出力してみて、本当に個数分ループしているか確認してみたらどうでしょうか。

 if(count($paraholiday) > 1 && array_search("y", $is_deleted) !== FALSE){
    echo 0;
    登録処理
 }else if(count($paraholiday) > 1 && array_search("n", $is_deleted) !== FALSE){
    echo 1;
    削除処理
 }

これで削除処理も複数回ループしているなら、SQL構文の中、たとえば削除条件の変数が同じ値を指し続けていた、など問題があると分かります。

Comments

[leoairen]

ご回答、ありがとうございました。できました。

Answer 2

回答依頼ありがとうございます。

正直ちょっとコードが読みにくいですね、あと必要な部分は意図的に載せてないのでしょうが何？
ってなります。例えば、$ret ってどこから来たの？とか

変数名もわかりにくいので意図しているところが読みにくいです。($hって？)
適切にコメントを入れるなりして自分が何をしたいのかはっきりしておかないと後々メンテ不能になると思われますよ。 (普通に見たらスルーする質問ですｗ)

で、あまり時間をかけて読んでいないですが、
if 文が書けているならあとは、登録ならInsert、削除ならDeleteのSQLを発行するだけでしょ？と
思うのですが、何に詰まっているのでしょうか？

私の読解力も足りないですが、もう少しシンプルにして頂けると助かります ^_^；

Comments

[leoairen]

ご回答、ご指摘、ありがとうございます。わかりにくい内容で失礼いたしました。質問については補足情報を付け加えました。今回詰まった内容はif文です。書けていないため、複数登録ができても複数削除ができずにおります。

Answer 3

そもそも登録と削除が同じインターフェースになるのは考えにくいのだけど
UIを調整すれればもっと単純なモデルになりそうな気がします

入力はHTMLですよね？
どういったものを想定されていますか？

追記

<form>
<table>
<thead>
<th>日付<ht>
<th>削除</th>
<th>登録</th>
</tr>
</thead>
<tbody>
<tr>
<td>2016-02-01</td>
<td><input type="radio" name="d['2016-02-01']" value="del"></td>
<td><input type="radio" name="d['2016-02-01']" value="add"></td>
</tr>
<tr>
<td>2016-02-02</td>
<td><input type="radio" name="d['2016-02-02']" value="del"></td>
<td><input type="radio" name="d['2016-02-02']" value="add"></td>
</tr>
<tr>
<td>2016-02-03</td>
<td><input type="radio" name="d['2016-02-03']" value="del"></td>
<td><input type="radio" name="d['2016-02-03']" value="add"></td>
</tr>
</tbody>
</table>
<input type="submit" value="go">
</form>

<?PHP
$d=filter_input(INPUT_GET,'d', FILTER_DEFAULT , FILTER_REQUIRE_ARRAY);
foreach((array) $d as $key=>$val){
  $sql="";
  if($val=="del") $sql="DELTE FROM テーブル WHERE 日付カラム='$key'";
  if($val=="add") $sql="INSERT INTO テーブル(日付カラム) VALUES='$key'";
  print $sql."<br>";//実際にはprepare処理でsqlを発行
}
?>

Comments

[leoairen]

ご回答、ありがとうございます。
管理画面のHTMLがありまして、その画面にて入力した日付は登録しますので、ここまではできておりますが、
逆に入力された日付の一覧から選んだら削除にしたいです。

[yambejp]

ソース追記しました
HTML側で調整すれば、あとは単純な分岐になると思います

[leoairen]

ご回答、ありがとうございました。FKMさんのご回答で引っ掛かったところに気づけたので、できました。