C言語 配列を振り分ける

Question

下記文のループ処理が正常に行われないのですが、その理由が分かりません。
printf文を使ってみたところ２回だけ動作をしているようなのですが、どうしたら望みの動作になるか解説をお願いします。

動作としてはある一定の数字を起点にして配列を三つに分けたいです。
（今回でいうと元のWを010,1116,17~20のデータを振り分けたいです）

{
	int W[20] = { 9,19,19,12,12,17,8,8,16,16,16,10,10,10,20,20,20,9,9,9 };	
	int Sare[3][20]={0};
	int SareC=0;
	int up[3]={0};
.
.
.
	for(SareC=0;SareC<=20;){
		if(W[SareC]<=10){
			Sare[0][up[0]]=W[SareC];
			up[0]++;
			SareC++;
		}
		else if(W[SareC]<=16){
			Sare[1][up[1]]=W[SareC];
			up[1]++;
			SareC++;
		}
		else if(W[SareC]<=20){
			Sare[2][up[2]]=W[SareC];
			up[2]++;
			SareC++;
		}
	}
.
.
.
}

Answer 1

printf文を使ってみたところ２回だけ動作をしているようなのですが

という「2回だけ動作」という意味がよくわからないのですがどういう結果になるのでしょうか？そのprintfでの出力コードも載せてもらえますか？

それと、ループ回数が1回多いのと、21以上だった場合が抜けている点を修正しつつ、もう少し簡潔に書くとすると、

    for(SareC=0;SareC<20;SareC++){
        if(W[SareC]<=10){
            Sare[0][up[0]++]=W[SareC];
        }
        else if(W[SareC]<=16){
            Sare[1][up[1]++]=W[SareC];
        }
        else if(W[SareC]<=20){
            Sare[2][up[2]++]=W[SareC];
        }
    }

の様な感じでしょうか。
ループ変数を、forの処理の中でいじるのはわかりにくいコードのなるのでよくはないです。例えば、元のコードだと、もし、21以上の数値があった場合にSareCがカウントアップされません。こういうミスが起きがちなので、標準の書き方が望ましいです。

Comments

[carlton]

ありがとうございます
for文の継続条件術式についても、ありがとうございました

Answer 2

一か所おかしい（※）ところは直しましたが、普通に動いていますよ？

#include <stdio.h>

int main(int argc, char** argv) {
        int W[20] = { 9,19,19,12,12,17,8,8,16,16,16,10,10,10,20,20,20,9,9,9 };
        int Sare[3][20]={0};
        int SareC=0;
        int up[3]={0};

        for(SareC=0;SareC<20;){
                if(W[SareC]<=10){
                        Sare[0][up[0]]=W[SareC];
                        up[0]++;
                        SareC++;
                } else if(W[SareC]<=16){
                        Sare[1][up[1]]=W[SareC];
                        up[1]++;
                        SareC++;
                } else if(W[SareC]<=20){
                        Sare[2][up[2]]=W[SareC];
                        up[2]++;
                        SareC++;
                }
        }

        for(int i=0; i<3; i++) {
                for(int j=0; j<20; j++) {
                        printf("%2d,", Sare[i][j]);
                }
                puts("");
        }
        return 0;
}

結果（Ubuntu Linux amd64, gcc 5.4.0)

 9, 8, 8,10,10,10, 9, 9, 9, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
12,12,16,16,16, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
19,19,17,20,20,20, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,

※SareC のループは 0～19 までなので、 <=20 ではなく <20

Answer 3

各ifの中で'SareC++'を実行していますが、for分は次の繰り返し処理に入る際、

for (①; ②; ③)

③に書かれた処理を実行します。
つまり、SareC++が二回行われてしまいます。

また、②に書かれたSareC<=20はSareCが20の時も「真」になりループ内に突入しますが、
W[SareC]（W[20]へのアクセス）はメモリアクセス違反です。
SareC<20で良いです。