プログラマ専用SNS ミクプラ

{
    CString num;
    m_ed.GetWindowText(num);
    if(isdigit(atoi(num))==0){
           MessageBox("数字を入力せよ");
           m_ed.SetWindowText(" ");
     }
}

エデットボデックスに入力した文字列が数字じゃなければエデットボックスの中身を空にするという
簡単なプログラムですが・・例えば6を入れた場合numの中身は"6"でatoiの戻り値は6になっております。
しかしisdigitの戻り値は0になります。　何故でしょうか？

isdigit が必要とする引数は文字コード値です。
この場合では数値を渡しているので、文字コードでの数字ではないと判断して 0 が返ってきます。

上記の場合の条件だと、取得した文字列の各文字に対して isdigit でチェックし、
数字かどうかを判断するといいと思います。

ううむ、仰ってることはわかるんですがどうすればいいのかさっぱりに・・ちょっと混乱してきた。

「数字」を入力させたいのか、「数値」を入力させたいのかを明確にしてください。

「数字」というのは、文字通り「数」を表す「文字」のことです。
具体的には、'0'～'9'のことです。

本当にこれでよいのですか？

ふむふむ、'0'～'9'でいいなら全部「文字」ですよね。そうすると数値の方でしょうか。

isdigit() の結果が真になるのは、引数が'0'～'9'（数字）のときです。
0～9の数値のときではありません。

ということはatoi　にするのがマズイのでしょうか？　しかしisgiditの因数はint型じゃないといけないみたいですが、わかりません・・EDIT箱は文字列じゃないと使えないみたいだからEDIT箱とisdigitと併用しようとしたらどうしてもatoiにしなきゃいけない？と思うのですが、根本的にやり方変化させないとまずいのでしょうか

> ということはatoi　にするのがマズイのでしょうか？

そうですね。

>しかしisgiditの因数はint型じゃないといけないみたいです

引数（ひきすう）です。
isdigitの引数がint型であることは間違いありません。

アスキーコードの場合でいうと、isdigitの結果が真になるのは
'0'：十進で48
'1'：十進で49
...
'9'：十進で57
です。

今一度、'0'と0とは異なることをご確認ください。

ややこしいですけど、文字をint型として扱うことがあります。
そういう場合はatoiやsscanfのような関数で変換するのではなくて、普通にキャストすればいいです。
まぁキャスト無しでそのまま渡せることも多いですが。

型がどうこういうのではなく、そもそも何をしたいのかが明確であれば、それに対して適切な回答ができるのです。

こういうことでしょうか？

{
    CString num;
    m_ed.GetWindowText(num);
    if(isdigit(static_cast<unsigned char>(num[0]))){
           MessageBox("数字を入力せよ");
           m_ed.SetWindowText(" ");
     }
}

あっ、数字を入力していないときだから、判定逆になってますね。

{
    CString num;
    m_ed.GetWindowText(num);
    if(!isdigit(static_cast<unsigned char>(num[0]))){
           MessageBox("数字を入力せよ");
           m_ed.SetWindowText(" ");
     }
}

皆様ご指導ありがとうございました。tkmakwins15様の回答を拝見させていただき、キャストでしょうか、を見て
何これと思ったので知識不足なようです。キャストのやり方は把握しているつもりでしたがstatic_castというのは初めて見ました。今の私には若干難しかったようです。キャストについて勉強しなおします。ありがとうございました。

ちなみに、CEditを継承して数字以外の入力を許さないクラスを作成することもできます。

<参考サイト>
http://athomejp.com/goldfish/mfc/edit/number.asp