C言語初心者の者です。

質問

C言語初心者の者です。
現在、構造体を使った名簿を作成しているのですが、今回どうしても分からないことがあり、質問させて頂きました。
ソースの全文は下記のURLに添付しております。
（字下げ投稿が上手く出来ませんでした、申し訳ありません）
http://blog.livedoor.jp/airin7/archives/2010-10.html#20101026

分からない部分は下記の２点です。

(1)入力判定時に先頭が０入力のものをエラーにしたい。
まず、人数を確保してからメニュー画面へと移行するプログラムなのですが、
１～２５以外の数字は弾くものの、「01」などの入力は判定を通ってしまいます。
これをエラーとして返したいのですが、その方法と記述がどうしても分かりません。
メニューの判定についても上記と同じ現象が起こっているので、こちらもエラーを返したいです。

＜＜入力判定部分ソース抜粋＞＞
//main省略//
printf("何人分？(1～25件まで)");
fgets(buffer, sizeof(buffer), stdin);
if(sscanf(buffer, "%d %c", &num, &c) != 1)
{
printf("範囲外です\n");
stop_func();
continue;
}
else if(num < 1 || num > 25)
{
//**25人以上を設定した場合**/
printf("範囲外です\n");
stop_func();//←これは画面を止める関数です//
continue;
}
else
{//**確保できた場合はメモリを動的に確保**//
kok = (Meibo *)malloc( sizeof(Meibo) * num);
if(kok==NULL)
{
stop_func();
}
free(kok);
}

(2)一覧判定部分に「12345678901234・・・・」などの不正な文字を入力した場合、エラーを返したい。
一覧表示を５件ずつ分けているのですが、デバッグした後
ページ判定部分に「12345678901234・・・・」と入力してみると、

0x003028e6 でハンドルされていない例外が発生しました:
0xC0000005: 場所 0xc4e0bf70 を読み込み中にアクセス違反が発生しました。

という表示が出て、処理が強制終了してしまいます。
このような入力の場合もエラーを返したいのですが、どの部分が原因なのかが分かりません。

＜＜表示判定部分ソース抜粋＞＞
fgets(buffer, sizeof(buffer), stdin);
sscanf(buffer, "%s%*c", &hantei);
//**前の５件へ移動**//
if(hantei=='B'&&sno>5){ sno=sno-5;}
else if(hantei=='b'&&sno>5){ sno=sno-5;}
//**次の５件へ移動**//
else if(hantei=='N'&&(sno+4)<max){sno=sno+5;}
else if(hantei=='n'&&(sno+4)<max){sno=sno+5;}
//**メニュー画面へ戻る**//
else if(hantei=='Q'){return n;}
else if(hantei=='q'){return n;}
else
{
//**対応文字以外を入力した場合**//
printf("入力文字が不正です\n");
stop_func();
continue;
}
eno=sno+4;
if(eno>max){eno=max;}

その他にも間違っている部分などがあるかと思います。
勉強不足で申し訳ございません。
長文で申し訳ありませんが、ご教授お願い致します。

回答No.1

kmee · ベストアンサー

せっかく fgets→ sscanf としているのだから
・fgetsの戻り値を確認。
行が長すぎて、読み込んだ配列の長さと同じになってないか、逆に読み込み失敗で0になってないかを確認する。
・bufferの中身を確認。
期待している文字列になっているかを確認する。
例えば、人数を入力するところでは、最初に出てきた文字が0でない数字になっているか、数字がintで収まらないくらい連続していないか確認する。

とすればよいのでは?

> {//**確保できた場合はメモリを動的に確保**//
> kok = (Meibo *)malloc( sizeof(Meibo) * num);
> if(kok==NULL)
> {
> stop_func();
> }
> free(kok);
> }

これを含めてなのですが、コメントはわざとですか?
C言語の従来型のコメントは /* から */までです。改行等も含めて */ が現れるまでがコメントの範囲で、入れ子不可です。逆に1行の途中の部分だけコメントアウトすることもできます
printf( "%d", /* 1, */ 2 ) ; /* 2を出力する。 1はコメントアウトされている */
C++で // が追加になりました。これは //から行末までがコメントになります。後に、C言語でも // を行末までのコメントとして使えるようになりました。正し、古いコンパイラ等、対応していないものもあります。
これらを理解しているのならよいのですが、全部 //** **//になっているのを見ると、誤解している気がします。

さて、ここですが、 kokの領域を確保してすぐに解放しています。これでは、まったく意味がありません。

>case 1 ://**登録**//
>n = add(data, n,*kok,num);
>break;
>case 2 : //**一覧**//
>n = printall(data, n,*kok,num);
>break;

しかも、そのあとでkokが使われています。
free関数は、領域の解放はしますが、変数の初期化はしません。よって、この *kok は kokが指すMeibo型(しかし、その領域はすでにfreeで解放済み)ということになります。
現在はたまたま動作しているかもしれませんが、いつ不具合が起ってもおかしくない状態です。

> sscanf(t->from,"%s",&kok);
このような行がいくつもありますが
・scanf系関数の"%s"に対応するのは、char *型です。 kokが引数で「Meibo kok」と宣言されているので、 &kokは「Meibo *型」です。scanfはこのような場合、「何もチェックせずchar *型だと勝手に思いこんで処理します」
・しかも、kokは関数でのローカル変数ですから、returnした段階でメモリから無くなります。参照渡しをしたいのなら、ポインタと実体が逆です。(しかも、先に書いたように、呼び出しがわでは、解放済みの領域を使おうとしています)

もう一度落ち着いて設計をやりなおしたほうがよいのではないでしょうか。

お礼

素早い回答有難うございます。
kmee様の助言お陰で、疑問点は無事に解決することが出来ました。

また、他の点もご指摘頂き有難うございます。
コメント行に関しましては、自分の勘違いが原因です。
Cを始めたての頃「//**～**//」でコメントアウトされているものを見て、
その記述が正しいものだと誤解していたようです。

疑問が解決でき、その他の部分も修正が出来たので本当に助かりました。
また何かありましたらご相談するかと思いますが、その時はまた宜しくお願い致します。