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

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

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

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

Q&A

解決済

3回答

365閲覧

クラス メソッドの呼び出しについて

test.shiro2018

総合スコア16

PHP

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

0グッド

0クリップ

投稿2018/11/07 02:31

初心者でPHPを勉強しております。
現在、お問い合わせフォームを作成しておりバリデーション部分の共通化に取り組んでおります。前回の質問で頂いた回答を基に色々と試しております。

勉強していく中で、しっかりとクラスについて理解できるようになりたいと思うのですが、まだクラスと普通の関数の違いやそれぞれの使いどころが理解できていなかったりします。今回、特に分からなくて困っているのはクラスのメソッドを呼び出すときです。

お問い合わせフォームでバリデーションをクラス化したものを作っています。

class Validate{ public function checkRequierd($aryList,$inputData){ $errMsg = ""; foreach($aryList as $key=>$val){ if($inputData[$key] === ""){ $errMsg[$key] = $val . "を入力してください"; } } return $errMsg; } public function checkTel($tel){ $errMsg = ""; $pattern = "/^(0{1}\d{1,4}-{0,1}\d{1,4}-{0,1}\d{4})$/"; if(!preg_match($pattern, $tel)){ $errMsg = "電話番号を正しく入力してください"; } var_dump($errMsg); return $errMsg; } public function checkMail($mail){ $errMsg = ""; $pattern = "/^([0-9A-Za-z/\_.\-]{1,255})\@([0-9A-Za-z\_\-]+).([0-9A-Za-z\_.\-]*)[a-z]{2}$/"; if(!preg_match($pattern,$mail)){ $errMsg = "メールアドレスを正しく入力してください"; } var_dump($errMsg); return $errMsg; } } ?>

エラーチェックはすべての項目を同時に行いたいので、呼び出す側にはこんな感じで書いています。(この3つ並べる書き方も変な気がしています。。)

//エラーチェックを行う $validate = new Validate(); $err = $validate->checkRequierd($aryRequierd,$post); $err2 = $validate->checkTel($post["tel"]); $err3 = $validate->checkMail($post["mail"]);

ただ、やりたい事は呼び出したメソッドの結果を$errという変数に入れて、この変数の中身に空文字ではないものが0以上だったら入力ページから遷移できないようにしたいので、それぞれの結果がそれぞれの変数に代入されてしまうと条件式も複数書くことなり良くない気がします。ただ、呼び出してからどのような処理を書けばよいのかが分かりません。

まとめると、
・メソッドを複数呼び出すときはどのように書くのか?
・その結果をひとつの変数にすべて代入したいときはどのような処理を書けばよいのか?

上記の2点が一番知りたいことになります。分かりにくい質問ですみません。

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

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

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

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

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

m.ts10806

2018/11/07 07:08

初心者アイコン使ってください。細かいですが「初心者」ということ自体は要件には関係ありません。初心者アイコンがついていれば十分ですし、「初心者」にも色々ありますから、自身の理解度やどのように調べた・どのように試した を書かれたほうが回答者にとっては有意義な情報となります。
guest

回答3

0

ベストアンサー

前の質問の私の回答の後半にそれとなく書いたことが参考になるかとは思います。

入力項目自体を配列で設定ファイルで持っておけば、その入力項目親名だけ渡せばそのフォームの入力チェックを全て行うようにも作れます。

設定を配列で持つ

php

1$input_setting = [ 2 "name"=>["validate"=>"required"], 3 "tel"=>["validate"=>"required|tel"] 4];

設定をチェックで回す。

php

1$validate = new Validate(); 2$err = []; 3foreach($input_setting as $name=>$setting){ 4 if(array_key_exists("validate",$setting)){ 5 $validate_setting = explode("|",$setting["validate"]); 6 foreach($validate_setting as $set){ 7 $validate_method = "check".ucfirst($set); //check+設定の頭文字大文字をチェック用のメソッドとルール付けする 8 //厳密にはここでmethod_existsを入れたほうが良い 9 $errmsg = $validate->$validate_method($post[$name]); 10 if($errmsg !== ""){ 11 $err[$name][] = $errmsg; 12 } 13 14 } 15 } 16} 17var_dump($err);

メソッドも変数にできるのでそれを利用すると効率的に書けます。

何もエラーがなければ
count($err) が0になるのでifで分岐かければ良いです。

設定の配列はvalidateのみにしましたが、ここで色々持たせれば入力フォームとバリデートとDB登録など全部設定に持たせてそこから都度引っ張ってくるように汎用的な作りができます。
1つ親に画面名持たせれば、画面に依存しない形になりますしね。このバリデートの結果処理自体をチェック画面のPHPに書かずにチェック用コントローラに投げるだけにするとか。
オレオレフレームワークの始まりですね(ほんとに・・)

投稿2018/11/07 07:22

編集2018/11/07 07:27
m.ts10806

総合スコア80765

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

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

0

$err=array();

$err[] = $validate->checkRequierd($aryRequierd,$post);
$err[] = $validate->checkTel($post["tel"]);
$err[] = $validate->checkMail($post["mail"]);

エラー結果は
//何かしらエラーが入っていたら
if (!empty(err)) {

}

で判断して

$errをforeachで回して画面に表示させるとかするといいと思います。

投稿2018/11/07 04:38

ms5025

総合スコア292

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

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

0

こんなふうにしてはどうでしょうか?

※値がnullのときにはエラーメッセージに登録

PHP

1class Validate{ 2 function __construct() 3 { 4 $this->errMsg = []; 5 } 6 function test($a){ 7 foreach($a as $key=>$val){ 8 if(is_null($val)) $this->errMsg[$key] = "nulldata"; 9 } 10 } 11} 12 13$v=new Validate(); 14$v->test(["hoge1"=>"fuga1","hoge2"=>null,"hoge3"=>null]); 15print_r($v->errMsg);

投稿2018/11/07 03:02

yambejp

総合スコア114572

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

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

test.shiro2018

2018/11/07 04:41

回答ありがとうございます。 またもや何度もすみません! $v->test(["hoge1"=>"fuga1","hoge2"=>null,"hoge3"=>null]);はどのような意味になるのでしょうか?
yambejp

2018/11/07 05:47

testでバリデートをすると仮定して nullが引っかかるデータだと考えれば 引っかかったエラーが表示されるのがわかりませんか?
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問