正規表現がうまくマッチしません。

Question

###前提・実現したいこと

パスワードの入力チェックをしたいです。

条件
・半角英数記号混合、ただし英数は必須
・3文字以上連続の場合NG
・許可文字以外が含まれている場合NG

###発生している問題・エラーメッセージ

・記号の3文字連続がひっかからない
・許可以外の記号（たとえば<>など）が含まれていてもひっかからない

###該当のソースコード

// 許可記号
$escaped = preg_quote('!#$%()*+-./:;=?@[]^_`{|}', '/');

// 3文字以上連続の場合、NG
if (preg_match("/([{$escaped}a-zA-Z0-9])\1{2,}/", $value) !== 0) {
	return error();
}

// 半角英＋数字記号混在でない場合、NG
if (preg_match("/[a-zA-Z]+/", $value) == 1){
	if(preg_match("/[{$escaped}0-9]/", $value) == 0) {
		return error();
	}
}else{
	return error();
}

// 許可文字以外が含まれている場合、NG
if (preg_match("/[^{$escaped}a-zA-Z0-9]/", $value)) {
	return error();
}

###その他

内部エンコーディングはEUCです。

Answer 1

""が一つphpで食べられてしまった為ではないでしょうか？

if (preg_match("/([{$escaped}a-zA-Z0-9])\1{2,}/", $value) == 1) {

Comments

[pgn]

ありがとうございました！！
ご指摘通り修正するとうまくいきました。

あともう一つの問題
・許可以外の記号（たとえば<>など）が含まれていてもひっかからない
についても間違いがありますでしょうか・・・

[pgn]

すみません！他の問題があったようで、2つ目の問題は解決しました！
ありがとうございました！！

[A.Ichi]

その様ですね。正しく1が戻っています。

Answer 2

preg_match()!==0
って変じゃないですか？
くらべるならfalse
（間違った回答ですが自戒もこめて残しておきます）

#追記
「\1」→「\1」はどうでしょうか？

Comments

[KiyoshiMotoki]

・マッチしなかった場合だけ、return error(); を実行する。
・マッチした場合、およびエラーが発生した場合は 以降の処理を実行する。

という意図であれば、正当な実装です。

http://php.net/manual/ja/function.preg-match.php
> 返り値
> preg_match() は、pattern が指定した subject にマッチした場合に 1 を返します。 マッチしなかった場合は 0、エラーが発生した場合は FALSE を返します。

エラーの場合にどうするのが良いかは議論が分かれそうですが。

[yambejp]

KiyoshiMotokiさん、ご指摘ありがとうございます
スミマセン適当なことを書いちゃったみたいで
たしかにpreg_matchの戻り値はintでした

[KiyoshiMotoki]

yambejp様

私の指摘も間違いでした。
正しくは

・マッチした場合、およびエラーが発生した場合は、return error(); を実行する。
・マッチしなかった場合だけ、以降の処理を実行する。

ですね (^^;

[yambejp]

とりあえず本題の指摘だけはしておきます

[pgn]

ありがとうございます。
なんとなくの理解でしたので、勉強になりました！！
preg_match()!==0 と
preg_match()==1 は同じと思っていました・・・

・記号の3文字連続がひっかからない
はご指摘通りの修正でうまくうごきました！

・許可以外の記号（たとえば<>など）が含まれていてもひっかからない
は、やはりうまくいきません。
どこが問題でしょうか（＞＜）
よろしくお願いいたします。。。

[yambejp]

許可文字以外はご提示のものでヒットしますよね？

$value="a<b";
$escaped = preg_quote('!#$%()*+-./:;=?@[]^_`{|}', '/');
$pattern="/[^{$escaped}a-zA-Z0-9]/";
if (preg_match($pattern, $value)) {
print "error";
}

[pgn]

すみません！他の問題があったようで、2つ目の問題は解決しました！
ありがとうございました！！