値が空からだったらkey消去したい

Question

現在状態
$errorsが空でもkeyもっているので表示されてしまう。

やりたいこと
$errorsが空だったらkyeを持たず$errorsが空ではなかったら表示させるようにする。

session_start();
$message = $_POST['message'];
$mail = $_POST['mail'];
$comment = $_POST['comment'];
$upfile = $_FILES['upfile'];

Class Errorchak {

    public $message;
    public $mail;
    public $comment;
    public $upfile;
    public $errors;

    //メッセージエラーチェック
    public function chakmes($message) {
        $errors = "";
        if ((mb_strlen($message) >= 10 || mb_strlen($message) <= 5)) {
            $errors = "5文字以上10文字以下で記入してください。";
        }
    }

    public function chakmessa($message) {
        $errors = "";
        $pattern = "/^[a-zA-Z0-9]+$/";
        if (preg_match($pattern, $message)) {
            $errors = "ひらがなで入力してください。";
        }
    }

    //メールのエラーチェック
    public function chakmail($mail) {
        $errors = "";
        $pattern = "/^[a-zA-Z0-9_¥.¥-]+@[a-zA-Z0-9¥-]+.[a-zA-Z0-9]{2,4}$/";
        if (!preg_match($pattern, $mail)) {
            $errors = "メールアドレスを正しく入力してください<br>";
        }
    }

    //画像のエラーチェック
    public function cheakfile($upfile) {
        $errors = "";
        if (is_uploaded_file($_FILES['upfile']['tmp_name'])) {
            if (move_uploaded_file($_FILES['upfile']['tmp_name'], "./" . $_FILES['upfile']['name'])) {
                echo "uploaded<br>";
            } else {
                $errors = "アップロードできませんでした。";
            }
        }
    }

    public function cheakfiletype($upfile) {
        $errors = "";
        $upfil = $_FILES['upfile']['type'];
        if (!preg_match("/.*.jpg|.*.png||.*.gif/i", $upfil)) {
            $errors = "対象ファイルはPNG，JPG，GIFのみです。<br>";
        }
    }
    
    public function totalerror($errors){
        
        if (!empty($errors)) {
            $this->errors = $errors;
            echo $errors;
        }else{
            unset($errors);
        }
        return $errors;
    }
}

$errorchak = new Errorchak($errors);
$errors = array();
$errors[] = $errorchak->chakmes($_POST['message']);
$errors[] = $errorchak->chakmessa($_POST['message']);
$errors[] = $errorchak->chakmail($_POST['mail']);
$errors[] = $errorchak->cheakfile($_POST['upfile']);
$errors[] = $errorchak->cheakfiletype($_FILES['upfile']);

?>

やってみたこと

 public function totalerror($errors){
        $errors = array();
        $errors[] = chakmes($_POST['message']);
        $errors[] = chakmessa($_POST['message']);
        $errors[] = chakmail($_POST['mail']);
        $errors[] = cheakfile($_POST['upfile']);
        $errors[] = cheakfiletype($_FILES['upfile']);
        
        if (!empty($errors)) {
            $this->errors = $errors;
            echo $errors;
        }else{
            unset($errors);
        }
        return $errors;
    }
}

$errorchak = new Errorchak($errors);
$errorchak->totalerror(); 
?>

これですとFatal error: Uncaught ArgumentCountErrorになってしまっていました。
よろしくお願いします

Comments

[m.ts10806]

kye　ではなく key です。
細かいですが決まっている文言についてはスペルミスは命取りになりますのでご注意を。

Answer 1

記述されているコードだとエラー(Notice)でまくるけど、

これですとFatal error: Uncaught ArgumentCountErrorになってしまっていました。

の原因は
public function totalerror($errors){なのに$errorchak->totalerror(); で引数がないからです

Comments

[yuyuyuyugi2]

ありがとうございます！

Answer 2

値が空からだったらkye消去したい

の意味がよくわかりません。
そもそも「空」とは送られてきていない状態のことなのか
""（長さ0の文字列）ということなのかにもよります

おそらくfilter_inputである程度かいけつできると思います

$message = filter_input(INPUT_POST,'message');
$mail = filter_input(INPUT_POST,'mail');
$comment = = filter_input(INPUT_POST,'comment');

ただ$_FILESは任意の処理が必要かも

$upfile = isset($_FILES['upfile'])?$_FILES['upfile']:null;

Comments

[yuyuyuyugi2]

ありがとうございます。試してみます！

Answer 3

そもそもですが、Errorchakクラスの各functionで$errorsと言う変数に文言を入れていますが、returnしていません。
何も返してないので基本nullが格納されるのでは？

まずは「エラーがあったかどうか｣を確認する必要があります。
Errorchakクラスの各functionでreturn $errors;でも入れておいて1度適当な変数に受け取っておいて、判断すると良いです。

$message_error = $errorchak->chakmes($_POST['message']);
if($message_error !== ""){
  $errors[] = $errorchak->chakmes($_POST['message']);
}

1点注意。
$_POSTはスーパーグローバル変数で、どこからでも同じ形で参照できるため、引数で渡す意味がありません。
nameだけ渡すようにして$_POST[$name]のように受け取っても良いのでは。

Comments

[yuyuyuyugi2]

ありがとうございます。やってみます！

Answer 4

$errors = array();
$errors[] = $errorchak->chakmes($_POST['message']);
$errors[] = $errorchak->chakmessa($_POST['message']);
$errors[] = $errorchak->chakmail($_POST['mail']);
$errors[] = $errorchak->cheakfile($_POST['upfile']);
$errors[] = $errorchak->cheakfiletype($_FILES['upfile']);
$errors = array_filter($errors, function($e) { return $e !== "";});

↑これで空のやつが消えるんで、そしたらempty($errors)がtrueになるんで、とりあえず、全部の関数でreturn書きましょう。

Comments

[papinianus]

chakmesとかchakmessaとかchakmailとか全部でreturn $error;してねってことです。

[yuyuyuyugi2]

ありがとうございます
$errors = array_filter($e, function($e) { return $e !== "";});

は、returnの結果がtrueじゃなかったら配列されないってことでよいのでしょうか？

[papinianus]

記載ミスがあった。
$errors = array_filter($erros, function($e) { return $e !== "";});
array_filterは、第一引数の配列から、おっしゃるようにreturnがtrueのものだけ残した別の配列を作ってくれる子です。
なんで、全部、空文字だったら、配列されないというか0個しかない配列になって、これはempty

[yuyuyuyugi2]

ありがとうございます。
なんとかなりそうなので、後は自力でがんばってみます！

Answer 5

このソースだけでは再現できないのでなんともですが、エラーを見る限り、
以下でtotalerror()に渡す引数が指定されていないことが原因ではないかと。

$errorchak->totalerror();