PHPの配列によるエラーについて

Question

###前提・実現したいこと
PHPで、アンケートフォームを作っており、
「バリデーションチェックをして下の表に記述。
エラーがあれば表示し、チェックボックスの中身は保持する。
また、値が複数ある場合はすべて表示する」
ということを実現しようと思っています。
###発生している問題・エラーメッセージ

ここを参考にして配列を用いた表示を作成していましたが、
選択していない部分の値が全て
Undefined offset: x(xは選択していない部分の値)と
なって出てきてしまいます。
また、複数を選択しても、最終的に出力される答えは常に、値が大きい物のみになってしまいます。

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

<?php
    if (empty($_POST['attribute'])) {
        $errors['attribute'] = "属性が入力されていません。";

    }/* elseif (htmlspecialchars(ctype_digit(strval($_POST['attribute'])), ENT_QUOTES) == false) {
        $errors['attribute'] = "不正なPOSTです。(属性1)";

    } */elseif (intval($_POST['attribute']) > 22) {
        $errors['attribute'] = "不正なPOSTです。(属性)";

    } else {
      $attribute = $_POST['attribute'];
    }
if (intval($_POST['attribute']) == 22) {
$x = $_POST['attribute_custom'];
    if (($x == null) && (intval($_POST['attribute']) == 23)) {
        $errors['attribute_custom'] = "属性が入力されていません。(カスタム1)";

    }
    else if ((str_replace('    ' ,'', $_POST['attribute_custom']) == "") && (intval($_POST['attribute']) == 23))  {
        $errors['attribute_custom'] = "属性が入力されていません。(カスタム2)";

    } else {
        $attribute_custom = $_POST['attribute_custom'];
    }
}

?>
  <?php
  $attribute2 = array(
  1   => 'a',
  2  => 'b',
  3  => 'c',
 //同じようなソースの繰り返し
  23  => 'x',);
$check_attribute = array();
   if (isset($_POST['attribute']) && isset($attribute)) {

     for($i=0;$i<count(intval($attribute));$i++){
       $check_attribute[$attribute[$i]]="checked";
  }
     $check_attribute[intval($attribute)]="checked";
     foreach ((array)intval($attribute) as $key => $value) {
     $select_attribute = $attribute2[$value]."、";
     }
     $attribute_list=rtrim($select_attribute, "、");
   }
  echo <<<EOD
  <div id="choice">

  <input type="checkbox" name="attribute[]" value="1" id="1" {$check_attribute[1]}><label for="1">1</label><br><br>
  <input type="checkbox" name="attribute[]" value="2" id="2" {$check_attribute[2]}><label for="2">2</label><br><br>
  <input type="checkbox" name="attribute[]" value="3" id="3" {$check_attribute[3]}><label for="3">3</label><br><br>
</label><br><br>
 //同じようなソースの繰り返し
<input type="checkbox" name="attribute[]" value="23" id="other" {$check_attribute[23]}><label for="other">その他</label><br><br>
    <input type="text" name="attribute_custom" placeholder="カスタム" value="" id="attribute-custom"><br>

</p>
EOD;
 ?>
//関係ないので中略

<?php
                  if ((isset($_POST['attribute_custom'])) && ($attribute == "23") && (isset($attribute))) {

            $attribute_custom = htmlspecialchars($_POST['attribute_custom'], ENT_QUOTES);
            echo "その他\($attribute_custom\)";
          }
            elseif (($attribute !== "23") && (isset($attribute))) {
              echo $attribute_list;
            }
                  ?>

###補足情報(言語/FW/ツール等のバージョンなど)
環境
XAMPP最新版、Windows10、Atom1.18.0

Answer 1

ざっとやるとこんな感じです
この手のデータの受け渡しはfilter_input()で処理する方がよいでしょう

$attribute2 = [
  1  => 'a',
  2  => 'b',
  3  => 'c',
  23 => 'x',
  ];
$select_attribute="";
$checked["attribute"] = array_fill(1,23,"");
$attribute=filter_input(INPUT_POST,'attribute',FILTER_VALIDATE_INT,["flags"=> FILTER_REQUIRE_ARRAY]);
if(!is_null($attribute)){
  foreach($attribute as $val){
    $checked["attribute"][$val]=" checked";
  }
  $select_attribute=implode("、",array_filter($attribute2,function($x) use($attribute){
    return in_array($x,$attribute);
  }, ARRAY_FILTER_USE_KEY));
}
echo <<<EOD
<p id="choice">
<form method="post">
<input type="checkbox" name="attribute[]" value="1" id="x1"{$checked["attribute"][1]}><label for="x1">1</label><br><br>
<input type="checkbox" name="attribute[]" value="2" id="x2"{$checked["attribute"][2]}><label for="x2">2</label><br><br>
<input type="checkbox" name="attribute[]" value="3" id="x3"{$checked["attribute"][3]}><label for="x3">3</label><br><br>
</label><br><br>
<input type="checkbox" name="attribute[]" value="23" id="other"{$checked["attribute"][23]}><label for="other">その他</label><br><br>
<input type="text" name="attribute_custom" placeholder="カスタム" value="" id="attribute-custom"><br>
<input type="submit" value="go"><br>
</form>
{$select_attribute}
</p>
EOD;

Answer 2

さくっと見た限り

$attribute

は$_POST['attribute']を代入したものと仮定して、

	<input type="checkbox" name="attribute[]" value="1" id="1" {$check_attribute[1]}><label for="1">1</label><br><br>

とやってるので配列のはずなので、

intval($attribute)

とするのはおかしいですね。

Comments

[penginer]

御返答ありがとうございます。
申し訳ございませんが
ミスが有ったので訂正しました。
その上でもう一度ご教授いただけると
ありがたいです。

[rururu3]

訂正したのを拝見しましたけど、同じ返答になります($_POST['attribute']は配列なのでおかしくなる)
ので、まずは配列と数値をしっかり理解して修正しましょう。

※おかしくなるであろう理由を指摘しておきます
http://php.net/manual/ja/function.intval.php
phpのintval関数において配列を指定したら0か1しか帰ってこないです

Answer 3

ツッコミどころ満載ですね・・・

rururu3さんのおっしゃるとおり、$_POST['attribute']は配列です。

また、$attribute は、PHP5.3以前のphp.iniでは設定可能であった、
register_globals = onを期待する動作なので、
バージョンにも寄りますが、最近の環境では
「isset($attribute)」はfalseですね。

if (empty($_POST['attribute'])) {

別のURLからPOSTされる前提ならありですが、
POSTしていない状態（GET）の時は必ず条件となりますね。

Undefined offset: x(xは選択していない部分の値)と

ですが、参考ページでは foreachで回しているところを、
forで回しているからです。
POSTされていない配列を見ようとするからエラーになってます。
checkboxはチェックされていない要素は送られません。

先頭に、

print_r($_POST);

でも入れて、内容を確認してみるといいと思います。

配列の処理とかをきちんとされるのが先決のようです。