フィボナッチ数列をfor文で

Question

100までのフィボナッチ数列をfor文で書きたいのですが、

#include <stdio.h>
int main (void) {
    
unsigned short data[3];
int i;

data[0]=0;
data[1]=1;

printf("%d\n", data[0]);
printf("%d\n", data[1]);

for(i=1; data[2] < 100; i++) {
    
 data[2] = data[1]+data[0];
 printf("%d\n", data[2]);
 data[0] = data[1];
 data[1] = data[2];
 } 
 
return 0;
}

これだと100を一個超えて出力されます。どのように改善したら良いでしょうか。

Answer 1

質問のコードの改善ではありませんが、こんなコードも書けるということで、

#include <stdio.h>

int main(void)
{
	for (int a = 0, b = 1, c; a <= 100; c = a + b, a = b, b = c)
		printf("%d\n", a);
}

printf がひとつというのが改善かな？

追記
次のように書いたら、質問のコードを改善したと思ってもらえますか？

#include <stdio.h>

int main(void)
{
	unsigned short data[3];

	data[0] = 0;
	data[1] = 1;

	for (; data[0] < 100; ) {
		printf("%d\n", data[0]);
		data[2] = data[1] + data[0];
		data[0] = data[1];
		data[1] = data[2];
	}
	return 0;
}

a, b, c を data[0], data[1], data[2] と書き換えただけです。

Answer 2

正しいプログラムについては、他の方が回答してくださっているので、100を超えたものが出力される原因を説明します。
https://docs.microsoft.com/ja-jp/cpp/c-language/for-statement-c?view=vs-2019
こちらを参照すると分かるように、今回であれば

for (i=1; data[2] < 100; i++)

の2番目が、data[2]が100より小さい間は実行し続けることになります。144が表示されるのは、
まだこの時はdata[2] = 89になっているからです。
正しいプログラムの回答ですと出力してから計算しているため、89を出力してからdata[2]の値が144になり、for文が次は実行されないことになります。
ちなみに、data[2]が初期化されていないため、正しく動作しない可能性があります。for文の前にdata[2]を初期化してください。

Answer 3

for(i=1; data[2] < 100; i++) {

data[2]には何も代入されていなくてここで使用されてます。
ローカル変数は、そのままではでたらめな値が入りますんで、このfor文が実行される可能性は非常に低いですね

Answer 4

forループの中でiを使っていないのでi++する意味もないし、whileで無限ループを書いて、計算したdata[2]の値が100を超えたら脱出すればいい。「100までの~」が曖昧だけど。

#include <stdio.h>

int main (void) {
    unsigned short data[3];

    data[0] = 0;
    data[1] = 1;

    printf("%d\n", data[0]);
    printf("%d\n", data[1]);

    while (1) {
        data[2] = data[1] + data[0];
        if (data[2] > 100) {
            break;
        }
        printf("%d\n", data[2]);
        data[0] = data[1];
        data[1] = data[2];
    } 

    return 0;
}

Answer 5

int main (void) {

unsigned short data[3];
int i;

data[0]=0;
data[1]=1;
data[2] = data[1]+data[0];

printf("%d\n", data[0]);
printf("%d\n", data[1]);

for(i=1; data[2] < 100; i++) {
 printf("%d\n", data[2]);
 data[0] = data[1];
 data[1] = data[2];
 data[2] = data[1]+data[0];
 } 

return 0;
}