一番大きい奇数を表示する

質問

scanf関数を使って数字を10回入力して一番大きなものを表示させるプログラムをつくったのですが、
さらに一番大きな奇数を表示するにはどうすればいいのでしょうか？

偶数＝割り切れる
奇数＝割り切れない

というところまでは分かるのですが、以下のプログラムに奇数を判別するソースを追加するのにはどうすればいいのでしょうか。

#include <stdio.h>

int main(void)

{
char str[1024];
char buf[10];
int i;
int w;
printf("文字列を10回入力して下さい：\n");

memset(str, 0, sizeof(str));

for (i = 0; i < 10; i++)
{

memset(buf, 0, sizeof(buf));
printf("input>\n");
scanf("%s", buf);

}

for (i = 0; i < 10; i++)
{
if ((buf[i] & 1) == 1) /* 奇数であるか */
{
if (strcmp(buf, str) > 0)
{
strcpy(str, buf);
}
}
}
printf("output>\n%s\n" , str);
getchar();
}

回答No.1

Werner · ベストアンサー

そのプログラムだと
> 数字を10回入力して一番大きなものを表示させる
ことさえできていないように思えるのですが。

> for (i = 0; i < 10; i++)
> {
> 　　memset(buf, 0, sizeof(buf));
> 　　printf("input>\n");
> 　　scanf("%s", buf);
> }
scanfするたびにbufが上書きされるので
結局10回目に入力された文字列しかbufに残りませんが、
それでよいのですか？

> if ((buf[i] & 1) == 1)
これでは文字列bufのi文字目の"文字コード"が奇数かどうかを見ていることになりますが、
それでよいのですか？

> if (strcmp(buf, str) > 0)
strcmpは数値の大小ではなく、文字列の辞書順の比較ですが、意図通りですか？

数値として比較したいなら、scanfで%dを使って数値として取得した方が良くないですか？
（他にも方法はあろうかと思いますが。）

お礼

誤ってメモ帳の試行錯誤している最中のソースをそのままペーストしてしまいました・・・。
ただ、ご回答のようにbufが毎回上書きされてしまうので全く意味をなさないですね。