PHPでチェックボックスを配列をしようしてチェックをつける

Question

PHPでチェックボックスでチェックがついていたら

チェックをするような動作をしています。

ですが「CheckboxColor」の関数を使用すると

なぜかチェックが付きません。

原因を調べても分からないのでどなたか教えて下さい。

ーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーー

こっちのべた書きは動作します。

    echo '<font class="chk_color">RED</font>';
    echo '<input type="checkbox" name="F_adData[color][]" value="1" '.Checkboxchecker($p1,'1').'> ';

    echo '<font class="chk_color">Blue</font>';
    echo '<input type="checkbox" name="F_adData[color][]" value="2" '.Checkboxchecker($p1,'2').'> ';

    echo '<font class="chk_color">Yellow</font>';
    echo '<input type="checkbox" name="F_adData[color][]" value="3" '.Checkboxchecker($p1,'3').'> ';

    echo '<font class="chk_color">Green</font>';
    echo '<input type="checkbox" name="F_adData[color][]" value="4" '.Checkboxchecker($p1,'4').'> ';

    echo '<font class="chk_color">Black</font>';
    echo '<input type="checkbox" name="F_adData[color][]" value="5" '.Checkboxchecker($p1,'5').'> ';

ーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーー
こっちは動作しません。

function CheckboxColor($p1){
global $FavoriteColor_Arr;

    foreach($FavoriteColor_Arr as $key=>$value){
        echo '<font class="chk_color">'.$value.'</font>';
        echo '<input type="checkbox" name="F_adData[color][]" value="'.$key.'" '.Checkboxchecker($p1,$key).'> ';
    }

}

---- 関数 ----

$FavoriteColor_Arr = array(
1=>'Red',
2=>'Blue',
3=>'Yellow',
4=>'Green',
5=>'Black'
);

function Checkboxchecker($color,$p2){
if( strpos($color, $p2) !==false) {
return "checked";
}else{
return "";
}
}

Answer 1

質問の回答ではありませんが、関数の引数の変数の名前がわかりにくいと感じました。

引数に何を使うのかがわかりにくいのでrikさんの回答でもcheckboxColorの引数に「'Blue'」が使われている方と「2」が使われている方が示されていると思います。

数字なら$color_index、文字なら$color_nameにするとか、もう少しわかりやすい名前にした方がコードがわかりやすくなっていいと思います。

Answer 2

$keyと$valueの扱いが混同されています。
２例を記載しました。

$FavoriteColor_Arr = array(
	1=>'Red'
	,2=>'Blue'
	,3=>'Yellow'
	,4=>'Green'
	,5=>'Black'
);

function checkboxColor($p1) {
	global $FavoriteColor_Arr;
	foreach ($FavoriteColor_Arr as $key => $value) {
		print '<font class="chk_color">' . $value . '</font>';
		print '<input type="checkbox" name="F_adData[color][]" value="' . $key . '" ' . checkboxchecker($p1, $key) . '> ';
	}
}

function checkboxchecker($color, $p2) {
    return ($color == $p2) ? 'checked' : '';
}

checkboxColor(2);

$FavoriteColor_Arr = array(
	1=>'Red'
	,2=>'Blue'
	,3=>'Yellow'
	,4=>'Green'
	,5=>'Black'
);

function checkboxColor($p1) {
	global $FavoriteColor_Arr;
	foreach ($FavoriteColor_Arr as $key => $value) {
		print '<font class="chk_color">' . $value . '</font>';
		print '<input type="checkbox" name="F_adData[color][]" value="' . $key . '" ' . checkboxchecker($p1, $value) . '> ';
	}
}

function checkboxchecker($color, $value) {
    return (strcmp($color, $value) == 0) ? 'checked' : '';
}

checkboxColor('Blue');

Comments

[pirlo21]

返信が遅れてすみません。

「$color」の変数の中身は「１　２　３」のように数字とスペースが入ります。
なので　checkboxColorの関数では　$FavoriteColor_Arrの$keyで判断をするようにしています。ちょっとどつぼにはまってどのへんが$keyと$valueのを混同しているか。。。といった感じです。もう少し粘ってみます。

[rik]

そういう事であれば、
function Checkboxchecker($color,$p2){
の直後に
$p2 = strval($p2);
を入れてみてください。
文字列と数値の比較で意図しない動作になっています。

[pirlo21]

すみません。
無事解決しました。