これらのバリデーションをfunction化させたい。

Question

これらのバリデーションをfunction化させたい。

受付中

回答 2

投稿 2016/03/23 13:07

評価
クリップ 0
VIEW 1,724

fearless0603

score 41

アップロードされたファイルについて、phpで以下の順序でバリデーションを実施する。
1.ファイルが選択されていること
2.拡張子がpdf,jpg,pngのいずれかであること
3.ファイルサイズが2M以下であること

◆補足
・2.3で使用する値は定数化すること
・1～3のいずれかのチェックでエラーと判定された場合は後続のチェックを打ち切る
・エラーが見つかった場合はクラス内（グローバル）変数にエラーメッセージを設定する
→エラーメッセージの内容はそれらしいものを設定してください
・バリデーション処理はfunctino化し、戻り値はbooleanとする
→true：エラーなし、false：エラーあり

バリデーションをfunction化させたものを作成したいです。どのようにすればいいのでアドバイスや、回答お願いします。

if文はうまく抜けました。

if(file_exists($_FILES['certification']['name'])){
        if(preg_match("(.+\.(jpg|pdf|png))",$filename)){/*jpgかpdf,pngしかアップできない*/
            $filesize = filesize($_FILES['certification']['name']);
            if($filesize <= 2097152){
                //2MB以下のもののみ
                        move_uploaded_file($filename,$updir);
                        /*ファイルをfilesフォルダに保存*/
                        echo "ファイル名" . $filename . " アップロード成功！";
                        echo $_FILES['certification']['error'];

                    } else {
                    echo "$filesize Byte ファイルサイズが大きすぎます。<br>2MB 以下にしてください。";
                        }
            } else {
                    echo "jpgかpdf,png のファイルしかアップできません。或いはハイフン以外の記号があります。";
                }
        } else {
            echo "ファイルが参照されていません。";
                    }

コード

<input type="hidden" name="MAX_FILE_SIZE" value="2097152" />
<input type="file" name="certification" size="30" />
<input type="submit" value="アップロード">

またfileタグは３つありますので、簡単にしたいです。

気になる質問をクリップする

クリップした質問は、後からいつでもマイページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

クリップを取り消します
良い質問の評価を上げる

以下のような質問は評価を上げましょう
- 質問内容が明確
- 自分も答えを知りたい
- 質問者以外のユーザにも役立つ
評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。
質問の評価を上げたことを取り消します

評価を下げられる数の上限に達しました

評価を下げることができません
- 1日5回まで評価を下げられます
- 1日に1ユーザに対して2回まで評価を下げられます
質問の評価を下げる

teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。
- プログラミングに関係のない質問
- やってほしいことだけを記載した丸投げの質問
- 問題・課題が含まれていない質問
- 意図的に内容が抹消された質問
- 過去に投稿した質問と同じ内容の質問
- 広告と受け取られるような投稿
評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。
質問の評価を下げたことを取り消します

この機能は開放されていません

評価を下げる条件を満たしてません

評価を下げる理由を選択してください
プログラミングに関係のない質問やってほしいことだけを記載した丸投げの質問問題・課題が含まれていない質問意図的に内容が抹消された質問過去に投稿した質問と同じ内容の質問広告と受け取られるような投稿
詳細な説明はこちら

上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。
質問の評価を下げる機能の利用条件

この機能を利用するためには、以下の事項を行う必要があります。
- 質問回答など一定の行動
- メールアドレスの認証
  メールアドレスの認証
- 質問評価に関するヘルプページの閲覧
  質問評価に関するヘルプページの閲覧

回答 2 件

評価が高い順
新着順
古い順

同じタグがついた質問を見る

score 1 · Answer 1 · 2016-03-23T13:51

+1

function validate(欲しいものを何か) {
    if (ファイルが選択されていなければ) {
        message = "選択してくださいね。";
        return false;
    }    

    if (拡張子がpdf,jpg,pngのいずれでもなければ ) {
        message = "pdf,jpg,pngのどれかにしてくださいね。";
        return false;
    }

    if (ファイルサイズが２メガを超えていれば) {
        message = "２メガ以下のファイルにしてくださいね。";
        return false;
    }

    return true;
}

validateをファイル分ループで呼ぶなりすればいいのではないでしょうか？

投稿 2016/03/23 13:51

回答の評価を上げる

以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
回答の評価を下げる

下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

score 1 · Answer 2 · 2016-03-23T13:53

+1

とりあえず、以下のようなことをすればそれっぽくなるかも。

既存のチェック部分を関数の定義部に移動
関数内で解決できない情報を引数で取得するようにする
あるいは、変動させたい情報を引数で
引数が多くなりすぎるようなら、複数の関数に分けることを考慮する

・エラーが見つかった場合はクラス内（グローバル）変数にエラーメッセージを設定する
→エラーメッセージの内容はそれらしいものを設定してください
・バリデーション処理はfunctino化し、戻り値はbooleanとする

ということであれば、関数部分がメソッドになる感じでしょうか。

投稿 2016/03/23 13:53

回答の評価を上げる

以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
回答の評価を下げる

下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

2016/03/23 14:48

```
function validate($filename){

if(file_exists($filename){
if(preg_match("(.+\.(jpg|pdf|png))",$filename)){/*jpgかpdf,pngしかアップできない*/
$filesize = filesize($filename);
if($filesize <= 2097152){
//2MB以下のもののみ
move_uploaded_file($filename,$updir);
/*ファイルをfilesフォルダに保存*/
echo "ファイル名" . $filename . " アップロード成功！";
echo $_FILES['certification']['error'];

} else {
message $filename."は$filesize Byteです。ファイルサイズが大きすぎます。ので2MB以下にしてください。";
return false;
}
} else {
message $filename."はjpg,pdf,png の形式ではありません。jpg,pdf,png形式のファイルのみアップロードできます。";
return false;
}
} else {
message $filename."ファイルが参照されていません。";
return false;
}
}

```

このような感じでしょうか。アドバイスお願いします。

``` function validate($filename){ if(file_exists($filename){ if(preg_match("(.+\.(jpg|pdf|png))",$filename)){/*jpgかpdf,pngしかアップできない*/ $filesize = filesize($filename); if($filesize <= 2097152){ //2MB以下のもののみ move_uploaded_file($filename,$updir); /*ファイルをfilesフォルダに保存*/ echo "ファイル名" . $filename . " アップロード成功！"; echo $_FILES['certification']['error']; } else { message $filename."は$filesize Byteです。ファイルサイズが大きすぎます。ので2MB以下にしてください。"; return false; } } else { message $filename."はjpg,pdf,png の形式ではありません。jpg,pdf,png形式のファイルのみアップロードできます。"; return false; } } else { message $filename."ファイルが参照されていません。"; return false; } } ``` このような感じでしょうか。アドバイスお願いします。

キャンセル
2016/03/24 03:25

基本はそれで良いと思いますが、validateに成功したらtrueを返し、呼び出し側で戻り値を判断して処理をすべきでしょう。

//2MB以下のもののみ
move_uploaded_file($filename,$updir);

少なくとも、上記はvalidateの処理ではないですよね。

また、elseをする必要がありません。設定した規則に沿わない条件の場合には、falseで戻るのだから、以下のようなチェックを列挙するだけ。
if (check statement) {
// 規則に沿っていない
必要に応じてメッセージ
return false;
}

validateでメッセージまで表示するか否かは色々と好みなどありそうですが。

基本はそれで良いと思いますが、validateに成功したらtrueを返し、呼び出し側で戻り値を判断して処理をすべきでしょう。 //2MB以下のもののみ move_uploaded_file($filename,$updir); 少なくとも、上記はvalidateの処理ではないですよね。また、elseをする必要がありません。設定した規則に沿わない条件の場合には、falseで戻るのだから、以下のようなチェックを列挙するだけ。 if (check statement) { // 規則に沿っていない必要に応じてメッセージ return false; } validateでメッセージまで表示するか否かは色々と好みなどありそうですが。

キャンセル

質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう！

これらのバリデーションをfunction化させたい。

関連した質問

同じタグがついた質問を見る

質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう！

これらのバリデーションをfunction化させたい。

15分調べてもわからないことは、teratailで質問しよう！

関連した質問

同じタグがついた質問を見る

SNSアカウントでログイン

teratailアカウントでログイン