配列内の10個の値でプログラムが機能しない

Question

基本的に、配列の最大サイズは10で、ユーザーは最大10個の値を入力できます。ユーザーが-1または0を入力して10個の値を入力する前にループが停止し、次のループに進みます。私の問題は、10の値を入力するまで完全に動作することです。結果は10の代わりに9で除算され、配列に9つの値があることが表示されます。

#include <stdio.h> 

int main(void) 
{ 
    float numbers[10]; 
    int i; 
    int n; 
    int count =0; 
    for(i = 0; i<10; i++) 
    { 
     scanf("%f", &numbers[i]); 
     if(numbers[i] == -1) 
      break; 
     if(numbers[i] == 0) 
      break; 
     count++; 
    } 


    n = count-1; 
    float sum = 0; 
    float average; 

    for(i = 0; i<=n; i++) 
    { 
     sum = sum + numbers[i]; 
    } 
    average = sum/count; 

    printf("The average price of the %d products is %.2f.\n", count, average); 
    return 0; 

} //this is the fixed solution. 

Answer 1

私はそれが

n = count-1 

ので、このラインであるかなり確信しているあなたは1を減算している理由を説明できますか？ 10個の数字を入力する場合、countは10になります.1を引いた後は、配列の最初の9個のインデックスだけを反復します。

1を減算する必要がある場合（0または-1を入力するユーザーを考慮して）、最後のforループの条件を< =に変更します。

for(i = 0; i<=n; i++) 

Answer 2

代わりn=count-1;
書き込みn=count;を書くべきではありません。 count ++をループの最後に移動します。

#include <stdio.h> 

int main(void) 
{ 
    float numbers[10]; 
    int i; 
    int n; 
    int count =0; 
    for(i = 0; i<10; i++) 
    { 
     scanf("%f", &numbers[i]); 

     if(numbers[i] == -1) 
      break; 
     if(numbers[i] == 0) 
      break; 
count++; 
    } 


    n = count; 
    float sum = 0; 
    float average; 

    for(i = 0; i<n; i++) 
    { 
     sum = sum + numbers[i]; 
    } 
    average = sum/a; 

    printf("The average price of the %d products is %.2f.\n", n, average); 
    return 0; 

} 

Answer 3

このプログラムで考慮する必要があるのは2つあります。

1。 n = count-1;

for(i = 0; i<n; i++) 
The count variable contains the number of element in the array, while n has been assigned to count - 1 to access from the 0th position , but while iterating the loop condition `i<n` make the loop to run n-1 time (i.e 9 times in this case). 

和演算は、最後の配列要素を計算することができなかったように。

nは整数であるため、型変換は、平均値を算出しながら行わなければならないのに対し、

average = sum/a;

変数の平均との和は、フロートです。

average = sum/(float)a;

NOTE : a should be replaced by count , which hold the exact count of the element in array. 

完全に修正されたコードは、 `私は最初のループの後に、あなたが持っている= 10 ''のカウント= 10 'そう' N = 9`については

#include <stdio.h> 

int main(void) 
{ 
    float numbers[10]; 
    int i; 
    int n; 
    int count =0; 
    for(i = 0; i<10; i++) 
    { 
     scanf("%f", &numbers[i]); 
     count++; 
     if(numbers[i] == -1) 
      break; 
     if(numbers[i] == 0) 
      break; 
    } 


    n = count-1; 
    float sum = 0; 
    float average; 

    for(i = 0; i<=n; i++) 
    { 

     sum = sum + numbers[i]; 

    } 
    average = sum/(float)count; 

    printf("The average price of the %d products is %.2f.\n", count, average); 
    return 0; 

}