【PHP】ファイルアップロードして画面遷移の機能の実装

Question

ファイルを３つアップロードさせ、３つのファイルアップロードすべてが設定したバリデーションを通過すれば違う画面に遷移、バリデーションを通過しなければ次画面でエラーメッセージを出力させるというのを実装しています。

コード

//登録ボタンクリック後の処理
        if (!empty($_POST)) {
            $this->chkUpfiles();
        }


    /**
     * ファイルが正しくアップロードされているか確認
     *
     * @access private
     * @return string
     */
    private function check($_filename) {
        $ret = false;
        if (is_uploaded_file($_FILES[$_filename]["tmp_name"])) {
            if (preg_match("(.+\.(jpg|pdf|png))", $_FILES[$_filename]["name"])) {
                $filesize = filesize($_FILES[$_filename]['tmp_name']);
                if ($filesize <= 2097152) {
                    $this->getUploadMsg[$_filename] = self::_SUCCESS;
                    $ret = true;
                } else {
                    $this->getUploadMsg[$_filename] = self::_ERR_MSG_FILE_SIZE;
                }
            } else {
                $this->getUploadMsg[$_filename] = self::_NOT_SUITABLE;
            }
        } else {
            $this->getUploadMsg[$_filename] = self::_NOT_EXIST;
        }
        return $ret;
    }


    /**
     * 全部のファイルが正しくアップロードされているか確認
     *
     * @access private
     * @return string
     */
    private function chkUpfiles() {
        $filename = array('certification', 'cert_seal', 'citizen');
        $this->check($filename[0]);
        $this->check($filename[1]);
        $this->check($filename[2]);
        
        // ↓check($hoge)の形式で書いていないので、functionが呼ばれていない。→OK
        // check変数が存在するかという判定になっている
//1回呼べばいいようにする

        if($this->check($filename[0])==TRUE and $this->check($filename[1])==TRUE and $this->check($filename[2])==TRUE){
            $url = 'https://' . $_SERVER['SERVER_NAME'] . '/mg/settle_selectable_result';
            Utils::redirect($url);
        }
    }

functionを一回呼び出して実装させるにはどうしたらいいでしょうか？

Answer 1

ご希望の1回だけになりませんが、filenameが配列になっているので
forかforeachで回しながら、中でcheckメソッドを呼んでみてはどうでしょうか？

for (var $i = 0; $i < count($filename); $i ++)
{
    if (!$this->check($filename[$i]))
    {
        Utils::redirect($url);
        exit;
    }
}

または

foreach ($filename as $file)
{
    if (!$this->check($file))
    {
        Utils::redirect($url);
        exit;
    }
}

Comments

[fearless0603]

private function chkUpfiles()の箇所ですね！試してみます。