質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.34%

  • C

    3977questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

  • 配列

    545questions

    配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。

C言語 配列を振り分ける

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 192

carlton

score 7

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

動作としてはある一定の数字を起点にして配列を三つに分けたいです。
(今回でいうと元のWを0~10,11~16,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++;
        }
    }
.
.
.
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

0

一か所おかしい(※)ところは直しましたが、普通に動いていますよ?

#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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

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

for (①; ②; ③)

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

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

という「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がカウントアップされません。こういうミスが起きがちなので、標準の書き方が望ましいです。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/06/29 18:09

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

    キャンセル

同じタグがついた質問を見る

  • C

    3977questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

  • 配列

    545questions

    配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。