バリデーションについて(正規表現)

Question

前提・実現したいこと

ユーザーネームを英数字のみで入力させるときに「フォーマットが不適切です」と表示させたい。

該当のソースコード

    if (name.match(/[亜-熙ぁ-んァ-ヶ]+/)) {
      $('#error-name').text("フォーマットが不適切です");
    };

試したこと

この場合だと、「太郎」と入力したときは「フォーマットが不適切です」と表示されますが、「t郎」と英語と漢字を混合にすると表示されません。

正規表現のサイトを見て色々試したのですが、15分たっても分からず...
何卒ご教授の程よろしくお願いします。

Answer 1

ホワイトリストにするのが適切です。

Comments

[_sora_]

すみません。ホワイトリスト調べましたが意味が分からないです...

[退会済みユーザー]

どぞ
https://ja.wikipedia.org/wiki/%E3%83%9B%E3%83%AF%E3%82%A4%E3%83%88%E3%83%AA%E3%82%B9%E3%83%88

[_sora_]

ありがとうございます！

Answer 2

この場合だと、「太郎」と入力したときは弾きはすが、「t郎」と英語と漢字を混合にすると

DBに保存されてしまいます。

根本的な問題ですが、「DBに保存」するためのバリデーションは、フロントエンドのJavaScriptでかけるのは適当ではありません。

フォームにJavaScriptがあろうが、不正なリクエストはいくらでも送れますので、サーバサイドでチェックすることが必須となります。

Comments

[maisumakun]

もちろん、その場でチェックしてくれる「使い勝手」のためにJavaScriptでのバリデーションを行うのは問題ないですが、データ整合性を保つためのバリデーションはサーバサイドで行うことが必須です。

[maisumakun]

そして、Unicodeでは「亜-熙」に「郎」はヒットしません。

[_sora_]

すみません、自分が今やりたいこととしては「t郎」と入力された時に「フォーマットが不適切です」と表示させたいだけです。

ちゃんとしたバリデーションはmodel内に記述しています。

[maisumakun]

> ちゃんとしたバリデーションはmodel内に記述しています。

DBに保存されてしまったのであれば、モデルサイドのバリデーションも適切でない、ということになります。

[_sora_]

「フォーマットが不適切です」と表示されないのはなぜでしょうか？？

[maisumakun]

上にコメントしたように、「郎」はこの正規表現にヒットしません。

[_sora_]

どのように改善するとヒットするようになりますか？
[亜-熙ぁ-んァ-ヶ]+の部分をどう変えたらいいですか？

[maisumakun]

> [亜-熙ぁ-んァ-ヶ]+の部分をどう変えたらいいですか？

そのやり方をそもそも変えて、ホワイトリスト方式を使うのが適当です。

[_sora_]

te2jiさんもおっしゃっているやり方ですね。
ありがとうございます。スッキリしました。