下のコードの修正をお願いします。

質問

終了処理、バッファのクリアなど、ほとんどの処理は出来たのですが、唯一a,b,cの入力数が上限の桁数LIMITをオーバーしてしまった場合に本来エラーが表示されなければならないのにそのまま正常処理で最大値が出力されてしまいます。つまり、サブ関数void disp_errorでの処理がmain関数で実行されません。
どこを直せばいいのでしょうか？

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <limits.h>

#define LIMIT (5)
#define NULLB '\0'
#define ECOUNT (3*LIMIT + 4)
#define EWORD (3)
#define NORMAL (0)
#define ABNORMAL (-1)
#define LF '\n'

int word_judge(char array[]);
void disp_error(int x_len , int y_len , int z_len) ;

int main(int argc , char *argv[])
{
int number_a ;
int number_b ;
int number_c ;

int input_len ;
int a_len ;
int b_len ;
int c_len ;
int end_len ;
int max ;

char input[ECOUNT+1] ;
char a[ECOUNT+1] ;
char b[ECOUNT+1] ;
char c[ECOUNT+1] ;
char array[ECOUNT+1] ;
char end[EWORD+1] ;

end[0] = 'n' ;
while(end[0] == 'n' )
{
while(1)
{
number_a = 0 ;
number_b = 0 ;
number_c = 0 ;
input_len = 0 ;
a_len = 0 ;
b_len = 0 ;
c_len = 0 ;
end_len = 0 ;
max = 0;

memset(input , NULLB , sizeof(char) *ECOUNT+1) ;
memset(a , NULLB , sizeof(char) *ECOUNT+1) ;
memset(b , NULLB , sizeof(char) *ECOUNT+1) ;
memset(c , NULLB , sizeof(char) *ECOUNT+1) ;
memset(end , NULLB , sizeof(char) *EWORD+1) ;
printf("a,b,c ?:") ;

fgets(input , ECOUNT+1 , stdin) ;
input_len = strlen(input) ;

if(input_len <ECOUNT)
{
a_len = strlen(a) ;
b_len = strlen(b) ;
c_len = strlen(c) ;

sscanf(input , "%[^,], %[^,], %s" , a , b , c) ;

if(a[0] != NULLB && b[0] != NULLB && c[0] != NULLB)
{

if(word_judge(a) == NORMAL && word_judge(b) == NORMAL && word_judge(c) == NORMAL)
{

if(a_len > LIMIT || b_len > LIMIT || c_len > LIMIT)
{
disp_error(a_len , b_len , c_len) ;
}

else
{
number_a = strtol(a , NULL , 10) ;
number_b = strtol(b , NULL , 10) ;
number_c = strtol(c , NULL , 10) ;

max = number_a ;

printf("最大値:") ;

if(number_b > number_a)
{
max = number_b ;

if(number_c > number_b)
{
max = number_c ;
}
}
else
{
if(number_c > number_a)
{
max = number_c ;
}
}
printf("%d\n" , max) ;

break ;
}

}
else
{
printf("文字が混ざっています。　\n") ;
}
}
else
{
printf("正しく、入力をしてください。\n") ;
}
}
else
{
if(input[ECOUNT-1] == LF)
{

}

else
{
/* バッファのクリアを行う */
while(getchar() != LF);
}

printf("入力数が多いです。\n") ;
}
}

while(1)
{
printf("終了しますか？ y/n:" );

fgets(end , EWORD+1 , stdin ) ;
end_len = strlen(end) ;

if(end_len < 3)
{
if(end[0] != 'y' && end[0] != 'n')
{
printf("y or nを入力して下さい。\n") ;
}
else
{
break ;
}
}
else
{
if(end[EWORD-1] == LF)
{

}
else
{
while(getchar() != LF) ;
}

printf("入力が間違っています。y or nを入力しなおしてください。\n") ;
}
}

}
return 0 ;
}

int word_judge(char array[])
{
int array_len ;
int i ;
int sub_number ;
int r_value ;

sub_number = 0 ;
r_value = NORMAL ;

array_len = strlen(array) ;

for(i=0 ; i<array_len ; i++)
{
sub_number = isdigit(array[i]) ;
if(sub_number != 0)
{
r_value = NORMAL ;
}
else
{
r_value = ABNORMAL ;
break ;
}
}
return r_value ;
}

void disp_error(int x_len, int y_len , int z_len)
{
int fc ;
fc = 0 ;

if(x_len > LIMIT)
{
fc += 1 ;
}

if(y_len > LIMIT)
{
fc += 10 ;
}

if(z_len > LIMIT)
{
fc += 100 ;
}

switch(fc)
{

case 11 :
printf("aとbの桁が多いです。 \n") ;
break ;

case 110 :
printf("bとcの桁が多いです。 \n") ;
break ;

case 101 :
printf("aとcの桁が多いです。 \n") ;
break ;

case 1 :
printf("aの桁が多いです。 \n") ;
break ;

case 10 :
printf("bの桁が多いです。 \n") ;
break ;

case 100 :
printf("cの桁が多いです。 \n") ;
break ;
}
}

回答No.1

kmee · ベストアンサー

> a_len = strlen(a) ;
> b_len = strlen(b) ;
> c_len = strlen(c) ;
としてから
> sscanf(input , "%[^,], %[^,], %s" , a , b , c) ;
としたのでは、このsscanfで代入されたa,b,cの長さにはならないですよね?
strlenの前に、a,b,cの文字列をゼロクリアしているので、a_len,b_len,c_lenは0です。よって
> if(a_len > LIMIT || b_len > LIMIT || c_len > LIMIT)
> {
> disp_error(a_len , b_len , c_len) ;
> }
このif文の条件は真にはなりません。

他はゴチャゴチャしすぎで、ちょっと読む気がしないので気付いたとこだけ

> if(a[0] != NULLB && b[0] != NULLB && c[0] != NULLB)
から
>number_c = strtol(c , NULL , 10) ;
までの処理、
> sscanf(input , "%[^,], %[^,], %s" , a , b , c) ;
の書式と戻り値を活用すれば、もっと簡単にかけるはずです。

お礼

すいません、おかげでできました。
ありがとうございます。

確かに、戻り値を活用した方がいいかもしれません。
整理してみることにします。