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

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

新規登録して質問してみよう
ただいま回答率
85.50%
HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

Q&A

2回答

3082閲覧

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

fearless0603

総合スコア41

HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

0グッド

0クリップ

投稿2016/03/23 04:07

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

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

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

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

PHP

1 2 if(file_exists($_FILES['certification']['name'])){ 3 if(preg_match("(.+\.(jpg|pdf|png))",$filename)){/*jpgかpdf,pngしかアップできない*/ 4 $filesize = filesize($_FILES['certification']['name']); 5 if($filesize <= 2097152){ 6 //2MB以下のもののみ 7 move_uploaded_file($filename,$updir); 8 /*ファイルをfilesフォルダに保存*/ 9 echo "ファイル名" . $filename . " アップロード成功!"; 10 echo $_FILES['certification']['error']; 11 12 } else { 13 echo "$filesize Byte ファイルサイズが大きすぎます。<br>2MB 以下にしてください。"; 14 } 15 } else { 16 echo "jpgかpdf,png のファイルしかアップできません。或いはハイフン以外の記号があります。"; 17 } 18 } else { 19 echo "ファイルが参照されていません。"; 20 } 21

HTML

1 2コード 3 4<input type="hidden" name="MAX_FILE_SIZE" value="2097152" /> 5<input type="file" name="certification" size="30" /> 6<input type="submit" value="アップロード"> 7

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

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

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

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

・エラーが見つかった場合はクラス内(グローバル)変数にエラーメッセージを設定する

→エラーメッセージの内容はそれらしいものを設定してください
・バリデーション処理はfunctino化し、戻り値はbooleanとする

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

投稿2016/03/23 04:53

t_obara

総合スコア5488

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

fearless0603

2016/03/23 05: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; } } ``` このような感じでしょうか。アドバイスお願いします。
t_obara

2016/03/23 18:25

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

0

php

1function validate(欲しいものを何か) { 2 if (ファイルが選択されていなければ) { 3 message = "選択してくださいね。"; 4 return false; 5 } 6 7 if (拡張子がpdf,jpg,pngのいずれでもなければ ) { 8 message = "pdf,jpg,pngのどれかにしてくださいね。"; 9 return false; 10 } 11 12 if (ファイルサイズが2メガを超えていれば) { 13 message = "2メガ以下のファイルにしてくださいね。"; 14 return false; 15 } 16 17 return true; 18}

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

投稿2016/03/23 04:51

root_jp

総合スコア4666

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問