全角文字の判定

現在Ｃ言語を勉強しております。

そこで、キーボードから入力された文字列のチェックを行う関数を作成したいのですが、実装方法が分かりません・・・。

以下に仕様と私の作成したソースを貼ります。
【許可する文字】は、
全角のひらがな
全角の英語（大文字も小文字もＯＫ）
全角の数字
です。

チェックする関数を作成したいのですが、以下ではうまくいかず・・・
教えてくださいorz
比較の仕方がおかしいでしょうか？
関数は、引数として渡された文字列に許可以外の文字が含まれている場合は１を、そうでなければ０を返す、という仕様にしたいです。

int checkName(char *str)
{
int i;

for(i = 0; str[i] != '\n'; i++){
/* 奇数バイトをチェック */
if(str[i] == 0x82){
i++;
/* ひらがなの場合 */
if(str[i] >= 0xa0 && str[i] <= 0xf1)
printf("ひらがな\n");
return 0;
/* 英語（大文字）の場合 */
if(str[i] >= 0x60 && str[i] <= 0x79)
printf("英語大\n");
return 0;
/* 英語（小文字）の場合 */
if(str[i] >= 0x81 && str[i] <= 0x9a)
printf("英語小\n");
return 0;
/* 数字の場合 */
if(str[i] >= 0x50 && str[i] <= 0x58)
printf("数字\n");
return 0;
}
return 1;
}
return 0;
}


以上、よろしくお願いいたします。

No.2 回答者： Yanch 回答日時： 2008/11/08 13:19

if 文の使い方が変ですよ。

if 文の条件が真の時に実行する文が複数ある場合は、ブロック化してください。
ブロック化しておかないと、ifの後の１文しか、真の時に実行する文として認識しませんよ。

これは×
----------------------------------------------------------------------
/* ひらがなの場合 */
if(str[i] >= 0xa0 && str[i] <= 0xf1)
　　printf("ひらがな\n");
　　return 0;
----------------------------------------------------------------------

こっちが〇
----------------------------------------------------------------------
/* ひらがなの場合 */
if(str[i] >= 0xa0 && str[i] <= 0xf1) {
　　printf("ひらがな\n");
　　return 0;
}
----------------------------------------------------------------------

この回答への補足

ご指摘ありがとうございます。
No1様からも指摘があった内容ですが、ソース中のprintfの行は削除しないといけないです・・・・。

申し訳ありませんでした。

補足日時：2008/11/08 13:46

この回答へのお礼

ソースがおかしいために、質問した内容に対する回答がいただけそうにないので、一度締め切って、再度質問しなおします。

本当にすいませんでした。

お礼日時：2008/11/08 13:48

No.1 回答者： edomin2004 回答日時： 2008/11/08 13:18

何のチェックもしていませんが、ソースを見る限り

先頭バイトが0x82だったら、「ひらがな」と表示するかしないかの判断はしますが、その後の「return 0」で必ずリターンしています。
先頭バイトが0x82じゃ無かったら「return 1」でリターンしています。
なので、ループにする必要は全くないルーチンになっています。（どちらにしても、一回しか通りません。）

もう少し、処理の見直しが必要でしょう。

この回答へのお礼

ご回答ありがとうございます。

申し訳ありませんでした。
自分でデバッグするためにprintf埋め込んだまま貼り付けてしまいました・・・。

おかしなソースを貼り付けてすいませんでした・・・。

お礼日時：2008/11/08 13:46