外部関数 呼ばれた回数を返す関数

Question

「外部変数を利用して、関数 times() が呼ばれた回数を返す関数 int times(void) を作成し、main 関数で times() を 5 回使い、それぞれの戻り値を表示するプログラムを作成せよ。」という問題で以下のようにプログラムを作成しました。

#include<stdio.h>

int times(void);
int count=0;
int times(void){
	return count++;
}

int main(void)
{
	int i;
	for(i=0;i<5;i++){
	printf("%d\n",times());
	}
	return 0;
}	

ここで疑問なのがtimes()を５回使った時、実行結果の正解がなぜ0 1 2 3 4となるのですか？5回呼んでいるので1 2 3 4 5じゃないのでしょうか？
(そもそもプログラムが違っていた時、質問が成り立っていないです。その場合は申し訳ありません。)

Answer 1

++演算子の使い方は他の回答者の方が述べている通りです。よくわからないときは素直に更新処理とreturnを分けて書いてください。

あとは、変数countがグローバル変数なら、そもそもcountを直接インクリメントしたり表示したりすればいいじゃないかという話になり、そしてcountの値がどこかの関数で変更されておかしなことになるかもしれないので、times関数内に入れてしまいましょう。

#include <stdio.h>

int times(void) {
    static int count = 0; /* 「= 0」を書かなくても初期値は 0 */
    return ++count;
}

int main(void)
{
    for (int i = 0; i < 5; i++) {
        printf("%d\n", times());
    }
    return 0;
}

Comments

[tikuno]

静的なローカル関数を使用するということでよろしいでしょうか？

[Daregada]

そうですね。staticを付けた場合、数値タイプの変数は0に初期化され、その後はプログラムが終了するまで値が保持されます。

[tikuno]

丁寧に教えて下さりありがとうございます。

Answer 2

count の初期値が 0 だからじゃないですか？
ああ、でも count++ を返しているから 1 からなのか...

【追記】
return count++ は count を return してから 1 を足すみたいですね。
以下のコードだと質問者の期待している結果になります。

#include<stdio.h>

int times(void);
int count=0;
int times(void){
    // count++;
    // return count;
    return ++count;
}

int main(void)
{
    int i;
    for(i=0;i<5;i++){
    printf("%d\n",times());
    }
    return 0;
}    

参照: i++と++iの違い

Comments

[bboydaisuke]

return count++ は count を return してから 1 を足すみたいですね。
以下のコードだと質問者の期待している結果になります。

```
#include<stdio.h>

int times(void);
int count=0;
int times(void){
count++;
return count;
}

int main(void)
{
int i;
for(i=0;i<5;i++){
printf("%d\n",times());
}
return 0;
}
```

[bboydaisuke]

コメントでは Markdown が使えないので回答の方に追記しました。

[otn]

> count++;
> return count;

return ++count;
ですね。

[bboydaisuke]

ありがとうございます。