出力結果が思い通りに行きません

Question

7人分の成績を入力した後にその評定が出るようにしたいのですが下記のような表示になってしまいます。
7人分の評点を入力してください。
50 80 70 100 60 0 35
1 : 50 C
2 : 80 A
3 : 70 B
4 : 100 A
5 : 60 C
6 : 0 F
6 : 35 D
^C
一番下の出力を7：３５　D としたいです。またC-cを押さないと次の作業に移れません。
色々探って見たところ自分としてはfor(i=1;N+1;i++)のところがうまく言っていないんじゃないかと思います。

#include<stdio.h>
#define N 7
int main(){
  int i,l,num[N];
  printf("7人分の評点を入力してください。\n");

  
  for(i=1;i < N+1;i++){
    
    for(l=0;l < N;l++){
      
      scanf("%d",&num[N]);

      
      if(num[N] >= 80){
	printf("%d : %d A\n",i,num[N]);
      }
       

      else  if(65 <=num[N] && num[N] < 80){
	printf("%d : %d B\n",i,num[N]);
      }
       

      else if(50 <= num[N] && num[N] < 65){
	printf("%d : %d C\n",i,num[N]);
      }
      

      else if(35 <= num[N] && num[N] < 50){
	printf("%d : %d D\n",i,num[N]);
      }

      else{
	printf("%d : %d F\n",i,num[N]);
      }
     
    }
  
  }
  return 0;
}    

Answer 1

numは7要素の配列ですので、num[0]からnum[6]を利用できます。
逆に言えば、num[7]にアクセスすることは出来ません。

ループはfor(i = 0; i < N; i++)を使い、出力する際に1加えた値を使って下さい。

if(80 <= num[i]) {
    printf("%d : %d A\n", i+1, num[i]);
}
// 直前の条件が偽なので、num[i]が80未満であることは分かりきっている
else if(65 <= num[i]) {  
    printf("%d : %d B\n", i+1, num[i]);
}
略

num[N]に何度もアクセスしようとしているのも大いに謎です。

---

またC-cを押さないと次の作業に移れません。

二重ループにしてるからでは？

Comments

[Teemro_431265]

num[N]になんどもアクセスしようとしているというのはどういうことですか？
一番最後にprintfで表示するということですか？

[LouiS0616]

ご自身のプログラム内にnum[N]と何回書いてあるか見てみましょう。
本来なら宣言時の一回だけしか書いてはなりません。

---
N要素の配列には、0からN-1番目までの要素があります。
N番目にアクセスすると未定義の動作を起こし、何が起きるか予測できなくなります。

[Teemro_431265]

わかりました。scanfの後だからnum[N]の中にすでに要素が入っていてそれを使えるものだと思っていました。
では次の質問なのですが、二重ループだとC-c押さないとプログラムが続いている状態になってしまう感じですか？

[LouiS0616]

二重ループにする意味がそもそもないですからね。
紙に簡単にロジックを書いてみると良いかと。

[LouiS0616]

このプログラム、結局は『数を入力し、それに対応する評価を出力』するミニプログラムをN回繰り返しているだけです。二重ループが登場する余地がありません。
配列や入力バッファが絡むのでちょっとややこしくなってはいますが。

[Teemro_431265]

え、ほんとですか（笑）
そうですね、もう一度ゆっくり考え直して見ます。
ありがとうございます。

[退会済みユーザー]

横から失礼します。
点数を判定するところを一つにまとめると2重for文で書くこともできます。
(私が書いたソースコードを載せておきます。)

[LouiS0616]

@Stars1024 さん
確かに可能ではありますね。コードとして洗練されるかどうかはちょいとわかりませんが。

Answer 2

二重ループで入力待ちをしているため、(N+1)*N回の入力をするまで終わらないプログラムになっています。

  for(i=1;i < N+1;i++){
    for(l=0;l < N;l++){
      scanf("%d",&num[N]);

また、書き込み場所が配列外（num[7]）の位置で固定されている点も問題です。

Comments

[Teemro_431265]

というのはscanfの場所が悪いということですか？

[shiron46]

それよりも、ここではそもそも何回繰り返したいのか考える必要があります。
ご自身のプログラムは、その通りの繰り返し回数になっているでしょうか？

[Teemro_431265]

一応人数が決められているので人数分繰り返したいです。
左側の数字はなっていませんが成績の判定はできていると思います。

[shiron46]

私はだいぶ遠回しに表現していますが、LouiS0616さんと同じことを指摘していることにはお気づきでしょうか。
もう一度、ご自身のプログラムをよく振り返ってみてください。

[Teemro_431265]

なんとなくは同じことを言われている気はします。
わかりました、ありがとうございます。

Answer 3

a.c

#include<stdio.h>

#define N (7)

char grade(int num) {
  char ans;

  if (num >= 80) {
    ans ='A';
  } else if (num >= 65) {
    ans = 'B';
  } else if (num >= 50) {
    ans = 'C';
  } else if (num >= 35) {
    ans = 'D';
  } else {
    ans = 'F';
  }
  return ans;
}

int main() {
  int num[N];

  printf("7人分の評点を入力してください。\n");
  for(int i = 0; i < N; i++) {
    scanf("%d", &num[i]);
  }

  for(int i = 0; i < N; i++) {
    printf("%d: %3d %c\n", i + 1, num[i], grade(num[i]));
  }
  return 0;
}

実行例

Comments

[Teemro_431265]

ありがとうございます。

Answer 4

こんにちは、点数を判定するところをfor文でまとめて書くと以下のようにすることもできます。
(あくまでも点数の最大値が100であることを前提に書いています。)
評価する点数の境界線のところを配列に格納し、それに対応する評価をchar型の配列で用意します。　

また、人数の7などは定数なので#defineで定義しましょう。

#include <stdio.h>
#include <stdlib.h>

#define BUFSIZE 256
#define MEMBER 7
#define SCORE_TYPE 6

int main(void)
{
	char buf[BUFSIZE];
	int data[MEMBER];
	int i,j;
	int score[SCORE_TYPE] = {0,35,50,65,80,101};
	char result[SCORE_TYPE] = "FDCBA";
	printf("%d人分の評点を入力してください。\n",MEMBER);
	
	for(i = 0; i < MEMBER; i++){
		printf("%d人目 ->",i + 1);
		fgets(buf,BUFSIZE,stdin);
		data[i] = strtol(buf,NULL,10);
	}
	for(i = 0; i < MEMBER; i++){
		for(j = 0; j < SCORE_TYPE - 1; j++){
			if(score[j] <= data[i] && data[i] < score[j + 1]){
				printf("%d : %d %c \n",i + 1,data[i],result[j]);
				break;
			}
		}
	}
	return 0;
}

Comments

[Teemro_431265]

ありがとうございます。