初期化

[研修生]

空白を数えるプログラムを作りました

main()
{
   int c, n1;
   n1 = 0;
   while(( c = getchar()) != EOF){
     if(c == ' '){
         ++n1;
         printf("%d\n", n1);
     }
   }
  
}

としました。これはどんどん空白をカウントしていきます
１回打ち込む度n1をリセットしたいのですがが望む形は空白を１個、２個、４個をそれぞれ入力すると
1
2
4
と出力する形です。今だと
1
3
7になります。
printfの↓で初期化をいれると
1
1
1
になってしまいます。
whileループの後で入れるとループが終わらないらしくそこまでいきません。

どのようにすればうまく初期化できますか？

[初級者]

１回打ち込むたび、というのは、
Enter キーを押すたび、ということですか？

であれば、'\n' を検知するたびに
カウンターをリセットすればよいでしょう。

[dic]

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

void	main()
{
	int n1;
	char c[100];
	n1 = 0;
	while(( gets(c)) ){
		int	i;
		for( i=0; i<strlen(c); i++ )
		{
			if( c == ' ' )
			{
				++n1;
				printf("%d\n", n1);
			}
		}
		n1 = 0;
	}
}

ではいかがでしょうか？

[研修生]

そんな手があったとは気が付きませんでした。
できましたありがとうございました。

[初級者]

main関数の型がvoidというのは、
ありなのでしょうか。

[研修生]

一応動いてますが、気になるようでしたらintにすればいいのでは

[dic]

私はコマンドラインでパイプライン処理などを行わないので
main関数はほとんどvoid型です

[Mist]

Windowsのコマンドラインはmain関数の戻り値を無視するから、戻り値を参照するバッチファイルなどを組まなければ問題ありませんが、ISO-C/ANSI-Cではmain関数は

int main(void)
または
int main(int argc, char *argv[/url])
int main(int argc, char **argv)

と決められているのでvoidにはしないほうがいいです。

[toyo]

私はvoid main( )と書いてある本やサイトは信用しないことにしています
http://www.kouno.jp/home/c_faq/c11.html#15

[hss12]

ANSI-Cを持ち出されると間違いになってしまいますが、
別のプログラムの関数にするときにvoid型とすぐに分かりますし
return 0;も省略できますし。
エラーにならないなら柔軟に書いても良いと思いますけどね。

入門書でなんの説明もなしにいきなりvoid main()になっているのはどうかと思いますが。
規格を知らずに書いているのなら、ほかの部分も知らずに書いているのだろうから
信用できないというのは同意できますね。