列挙型(enum)を使ってユーザー入力値の検証

Question

ユーザー入力値を検証したい。

現在作ったものがこちら

namespace app\model\data;
use app\model\enum;
class paramType extends enum{
	const STRING = 0;
	const NUMERIC = 1;
	const BOOL = 2;
}

namespace app\model\data;
use app\model\data\paramType;

/**
 * 入力値検証
 * Class paramModel
 * @package app\model\data\paramModel
 */
class paramModel {

	const STRING = 0;
	const NUMERIC = 1;
	const BOOL = 2;
	private static $instance;

	private function __construct() {
	}

	public static function getInstance() {
		if (!isset(self::$instance)) {
			self::$instance = new paramModel();
		}
		return self::$instance;
	}

	public function get(string $key, paramType $mode) {
		if(!isset($_GET[$key])) {
			self::_exception('Param ' . $key . 'not defined');
		}
		switch($mode) {
			case paramType::BOOL:
				return $this->_bool($_GET[$key]);
			case paramType::NUMERIC:
				return $this->_numeric($_GET[$key]);
			case paramType::STRING:
				return $this->_string($key);
		}
	}

	/**
	 * @param string $key
	 * @param \app\model\data\paramType $mode
	 * @return int|mixed|null
	 * @throws \Exception
	 */
	public function post(string $key, paramType $mode) {
		if(!isset($_POST[$key])) {
			self::_exception('Param ' . $key . 'not defined');
		}
		switch($mode) {
			case paramType::BOOL:
				 return $this->_bool($_POST[$key]);
			case paramType::NUMERIC:
				return $this->_numeric($_POST[$key]);
			case paramType::STRING:
				return $this->_string(POST_[$key]);
		}
	}

	private function _bool($val){
		if(is_bool($val)) {
			return $val;
		}
		self::_exception('not bool ' . $val);
	}

	private function _numeric($var) {
		if(is_numeric($var)) {
			return (int)$var;
		}
		self::_exception('not numeric ' . $var);
	}

	/**
	 * バイナリの削除
	 * @param $var
	 * @return mixed
	 * @throws \Exception
	 */
	private function _string($var) {
		if(is_array($var)) {
			foreach($var as $str) {
				self::_string($str);
			}
		} elseif (is_string($var)) {
			return preg_replace('/[\x00-\x1F\x7F]/ug', '', strtrim($var));
		}
		$this->_exception('param is not correct');
	}

	private function _exception($msg){
		throw new \Exception($msg);
	}
}
var_dump(paramModel::getInstance()->post('token', paramType::STRING));

こちらのenum クラスを参考にして受け取る $mode をnumber, string, bool に制御したいと思うのですが上手くいきません。enum クラスは記事のものをそのまま使わせていただいています。
https://qiita.com/Hiraku/items/71e385b56dcaa37629fe

エラー

Fatal error: Uncaught TypeError: Argument 2 passed to app\model\data\paramModel::post() must be an instance of app\model\data\paramType, int given,

エラーを見る限り2つ目のパラメータで渡してる paraType::STRING がint なのが問題なのでメソッドのget, post を変えなければいけないと思うのですがどうやって目標の　あらかじめ定義した値のいずれかしか取らない　パラメータを設定できますか？
また$_REQUEST系統のパラメータの取得はこの方式で問題無いでしょうか。

この様に

 paramModel::getIntance()->post('[キー]', [paramType::型]) 

で上手くいけば取得したいと思っています。
よろしくお願いします。

最終的にどうなったか

/**
        public static function get(string $key, paramType $mode) {
		if(!isset($_GET[$key])) {
			self::_exception('Param ' . $key . 'not defined');
		}
		switch($mode->valueOf()) {
			case paramType::BOOL:
				return self::_bool($_GET[$key]);
			case paramType::NUMERIC:
				return self::_numeric($_GET[$key]);
			case paramType::STRING:
				return self::_string($key);
		}
	}

	public static function post(string $key, paramType $mode) {
		if(!isset($_POST[$key])) {
			self::_exception('Param ' . $key . ' not defined');
		}
		switch($mode->valueOf()) {
			case paramType::BOOL:
				 return self::_bool($_POST[$key]);
			case paramType::NUMERIC:
				return self::_numeric($_POST[$key]);
			case paramType::STRING:
				return self::_string(POST_[$key]);
		}
	}

paramTypeは特に変える必要性が無く呼び方を修正すれば大丈夫と教えていただいたので paramType::String() でparamTypeのインスタンス作成
paramModelは直接質問と関係ないけど修正したので一応上げました。
paramTypeのセットされた変数は valueOf() で呼ぶ必要があるみたいです。

$_GET['someVar'] = true;
var_dump(paramModel::get('someVar',paramType::BOOL()));

こんな感じで使えるようになりました。ありがとうございました。

Answer 1

ちゃんと全体を見たわけではないですけど、使い方が間違っていると思います。

PHPで列挙型(enum)を作る

#使い方

//インスタンス化

$suit = new Suit(Suit::SPADE);
echo $suit; //toString実装済みなので文字列キャスト可能

echo $suit->valueOf(); //生の値を取り出す。intやfloat等の場合に。

// 適当な値を突っ込もうとすると、InvalidArgumentExceptionが発生して停止

//$suit = new Suit('uso800');

//__callStaticを定義してあるのでnewを使わずこんな感じでも書ける(PHP5.3以降)

$suit = Suit::SPADE();

つまり、

paramType::STRING()

として使用する必要があるかと。

Comments

[退会済みユーザー]

PHPには列挙型なんてねぇと、おら、思うぞ。
おめぇが貼ったリンクのは列挙型「みてぇな物」の作り方だ。
７年か８年くれぇ前にt_wadaさんがやったのが確か、最初だったな。

te2ji は本当に何年やっても言葉の定義すらわかってねぇな。

[退会済みユーザー]

リンク先はt_wadaさんがプレゼンしたときに「参照」した元記事ですね。
質問者もそれを参照して実装しようとしているようです。
＊質問読めばわかるかと。

ちょっと何が指摘したいのかよく分からないですが。まぁそういうことです。

[退会済みユーザー]

>ちょっと何が指摘したいのかよく分からないですが

ははは、すげぇな。本当にわかんねぇんだ。

[退会済みユーザー]

何回もいってんだけど、「言葉の定義はしっかりしろ」
質問者に回答するならな。

te2ji は本当に何年プログラム書いて飯くってんだよ。

[退会済みユーザー]

>リンク先はt_wadaさんがプレゼンしたときに「参照」した元記事ですね。

はぁ？

おめぇ、頭でぇじょうぶか？
７年か８年前に2017年の記事を参照したっていってんのか？

te2ji おめぇ、ほんとうのアホだろ。
「ビートルズはミスチルのパクリだ」って言ってる奴みてぇだｗ

[退会済みユーザー]

te2ji は、頭がやばすぎんぞ、マジで。

[退会済みユーザー]

リンク先記事の履歴ぐらい見ろよ＾＾；

あとWada さんの資料はコレかなぁ。。。
https://speakerdeck.com/twada/php-conference-2016?slide=39
他にもバージョンありそうだけど。

[退会済みユーザー]

>あとWada さんの資料はコレかなぁ。。。
>https://speakerdeck.com/twada/php-conference-2016?slide=39

おめぇの世界線は、７年か８年前にPHP7があんのか？

[退会済みユーザー]

ぐぐったんだけんど、見つからねぇ。

>リンク先記事の履歴ぐらい見ろよ＾＾；

2019年の7年か8年前って何年になると思ってるんだ？　おめぇ。
2013年だと微妙なラインだけんど、
あんときの和田さんはリンクなんか紹介してなかったはずだ。

[退会済みユーザー]

ずっと枝葉。本質を俯瞰で見ようよ。。。
wadaさんの話は、今回はどーでも良い。

いい記事なんでリンク張ったけどｗ

[退会済みユーザー]

>本質を俯瞰で見ようよ

なら、もう一回言うぞ。

PHPには列挙型なんて、ねぇ。

それを、ちゃんと、質問者に、教えろ。
おめぇらが話してるのは列挙型とは別の物だ。

[退会済みユーザー]

質問のリンク先読もうよ＾＾；
別モンだってちゃんと書いてるよ。。。

[退会済みユーザー]

おめぇは、教えてねぇな。

[退会済みユーザー]

ずっと屁理屈だな。
そろそろ付き合いきれんわ。。。

[退会済みユーザー]

そりゃ、こっちのセリフだぞ。

おめぇは論理が破綻してっから話が全く前にすすまねぇ。
おらが「７、８年くれぇ前ぇに見た」って言ってんのに、「それはこの記事を参照したものだ」って2013年の記事の事を言い出すし、

向こうは向こうで延々と同じこと言わせるし

https://teratail.com/questions/205407
-----------------------------------------
何回おんなじこといわせんだ？
質問と関係ない話すんな。

>だったら各 DB のワイルドカードでエスケープが必要な文字羅列してみ。

何回言ったら分かるんだ？　関係ない話すんなって。

>$statement = 'SELECT * FROM t_sale_den WHERE holiday=false and business_day like $hani%';
>$haniの変数には日付が入っており、他の個所で下記のようにしたら正常に表示されます。
>$sql = "SELECT * FROM t_sale_den WHERE holiday=false and business_day like '$hani%'";
>上記コードのシングルクォートをダブルクォートにするとエラーが出る

こういう話で、「ワイルドカードがどうした」とか「LIKEがどうした」とか、関係あるわけねーだろ。

本当におめぇは論理性も問題解決能力もゼロだな。te2ji

>$haniの変数には日付が入っており、他の個所で下記のようにしたら正常に表示されます。
>（〇年〇月が取得され一覧で表示されます。）

>LIKE はチョットめんどいですよ
>だったら各 DB のワイルドカードでエスケープが必要な文字羅列してみ。

te2ji の世界じゃ、日付に%とかアンダースコアとか使ってんのか。
おもしれぇ世界だなｗ
-------------------------------------------------

te2ji はさ、口先だけでなんとか言い逃れしようとすんの、いい加減にやめろ。

[退会済みユーザー]

ずっと枝葉。本質を俯瞰で見ようよ。。。
wadaさんの話は、今回はどーでも良い。
どうしても気になるならソース提示してくれ。
まぁ提示されても意味ないけど。。。

[mipopon]

すみません。まだこのサイトに登録して新しいものでコメント機能を見逃していました。@te2ji さんが言う通りwadaさんのプレゼンの資料を読んで参考にさせて頂いています。
今回は入力値検証を例にクラスインスタンス作成時の引数を制限して生成できるクラスを制限するという書き方に対する質問でした。最終的に自分の　var_dump が間違っていたのでそもそもそこから問題でした。
var_dump(paramModel::get('someVar',paramType::STRING()));

そもそもparamModelをインスタンス化させる必要が無い気がしたので書き換えてみました。
goku59さんは身内なのかガチなのか判別がつかないので放置ですかね？
te2jiさん　返信ありがとうございました。

[退会済みユーザー]

>goku59さんは身内なのかガチなのか判別がつかないので放置ですかね？

いや、おめぇの力量だと、まだ何言ってるか全く理解出来ねぇだろうから放置でいいぞ。

>@te2ji さんが言う通りwadaさんのプレゼンの資料を読んで参考にさせて頂いています。

全然話の内容わかってねぇみてぇだからな。

あと、PHPにはEnum（列挙型）なんて、ねぇぞ。よく覚えておいた方がいいと、おら、思うぞ。
この質問は、タイトルからしておかしいからな。
だから、te2ji 程度の奴にはちょうど良かったんだと、おら、思ってる。

[退会済みユーザー]

>wadaさんの話は、今回はどーでも良い。

>@te2ji さんが言う通りwadaさんのプレゼンの資料を読んで参考にさせて頂いています。

もぅ、おめぇらの会話、わっけわかんねぇぞｗ
知能レベルが似たもの同士なんだろうな。スカウターで見たら数字いっしょだろ、きっと。

[mipopon]

そうですね　実際に列挙型がどのようなものか　触った事が無いので。。

[退会済みユーザー]

そうだと思ってっけどよ、別物だからな。
PHPに慣れてからでいいけど、C/C++をちょっとだけやってみろ。
随分勉強になんぞ。

[退会済みユーザー]

5年後くらいに、Zend extensionの何かのコードあたりを見てみるとPHPがどうやって動いているのかわかっていいかもな。

[mipopon]

5年後は遠いですね。。。
でもベースとなるc言語を見るのは勉強になりそうなのでみてみます。
アドバイスありがとうございます、