C# と VB.NET の質問掲示板

■No29576 (ヤンサン さん) に返信
> >double に Parse
> >ToString() して、元の文字列と比較 → 異なっていたら NG

double ではマズイかと。桁数が保持されないので、たとえば、
"1.10" は "1.1" という文字列になり、NG と判定されかねません。
　double a = double.Parse("1.10");
　double b = double.Parse("1.1");

　string s1 = a.ToString();　// "1.1"
　string s2 = b.ToString();　// "1.1"


みきぬさん( No29539 )が書かれていたように、decimal の方が良いんじゃないかな?
　decimal a = decimal.Parse("1.10");
　decimal b = decimal.Parse("1.1");

　string s1 = a.ToString();　// "1.10"
　string s2 = b.ToString();　// "1.1"



なお今回のパターンは、double を使うにせよ decimal を使うにせよ、
Parse/TryPase だけでは検証しきれないと思うので(NumberStyles を指定したとしても)、
正規表現によるチェックと、数値範囲のチェックの両方を行う事をおすすめします。

No29536 の正規表現で一か所間違いを見つけましたorz
整数部0の小数のチェック部分で、小数部2桁のパターンだけに対応する形になっていたため、
例えば0.1でマッチしないになってました。

>     string s = 
>         @"^(-?(1000(\.0{1,2})?|[1-9][0-9]{0,2}(\.[0-9]{1,2})?|0\.([1-9][0-9]|[0-9][1-9]))|0(\.0{1,2})?)$";

         @"^(-?(1000(\.0{1,2})?|[1-9][0-9]{0,2}(\.[0-9]{1,2})?|0\.([1-9][0-9]?|[0-9][1-9]))|0(\.0{1,2})?)$";
                                                                             ~

のように修正(~の箇所の「?」を追加)すれば、この間違いについては大丈夫だと思います。

こういう凡ミスをしていても、このくらいの複雑さになるだけで簡単に見落としてしまうので、
単純な正規表現で済まないような条件では正規表現の使用は個人的に避けますね。


@"^-?[0-9]{1,4}(\.[0-9]{1,2})?$";

というような正規表現で、
-9999.99〜9999.99の範囲の数値表現(ただし-0や002のようなパターンは許容される)である
ことだけをチェックする部分に正規表現を用いるのがよいかなと思います。