2012-02-01 20 views
1

私のStatsクラスで私を助けるためにこの小さなプログラムを書こうとすると、すべてが中央値を除いて計算されているようです。私は何が欠けていますか?モード、メジアン、平均、標準偏差、分散計算機

誰かが私のために分散機能をしたいと思っているならば、余分なクレジットx)。

OSXをGCCコンパイラで実行しています。

#include<stdio.h> 
#include<math.h> 

float mean1(float[],int); 
float median1(float[],int); 
float mode1(float[],int); 
double standarddeviation1(float[],int); 

int main() 
{ 
int i,n,choice; 
float array[100],mean,median,mode; 
double standarddeviation; 

printf("Enter No of Elements\n"); 
    scanf("%d",&n); 
printf("Enter Elements\n"); 

for(i=0;i<=n-1;i++) 
    scanf("%f",&array[i]); 
do 
{ 
    printf("\n\tEnter Choice\n\t1.Mean\n\t2.Median\n\t3.Mode\n\t4.Standard deviation\n\t5.Exit\n"); 
    scanf("%d",&choice); 

    switch(choice) 
    { 
     case 1: mean=mean1(array,n); 
      printf("\n\tMean = %f\n",mean); 
      break; 
     case 2: median=median1(array,n); 
      printf("\n\tMedian = \n",median); 
      break; 
     case 3: mode=mode1(array,n); 
      printf("\n\tMode = %f\n",mode); 
      break; 
     case 4: standarddeviation=standarddeviation1(array,n); 
      printf("\n\tStandard deviation = %f\n",standarddeviation); 
      break; 
     case 5: break; 
     default:printf("Wrong Option"); 
      break; 
    } 

}while(choice!=5); 
    getchar(); 

return 0; 
} 

float mean1(float array[],int n) { 
    int i; 
    float sum=0; 
    for(i=0;i<=n;i++) 
    sum=sum+array[i]; 
return (sum/n); 
} 
float median1(float array[],int n) { 
float temp; 
int i,j; 
for(i=n-1;i>=0;i--) 
    for(j=0;j<=i;j++) 
     if(array[j]>=array[j+1]) 
     { 
      temp=array[j]; 
      array[j]=array[j+1]; 
      array[j+1]=temp; 
     } 

if(n%2==0) 
    return (array[n/2]+array[n/2-1])/2; 
else 
    return array[n/2]; 
} 
float mode1(float array[],int n) { 
return (3*median1(array,n)-2*mean1(array,n)); 
} 
double standarddeviation1(float array[],int n) { 
int j; 
double max[100],sum,variance,mean; 
mean=mean1(array,n); 
sum=0; 
for(j=0;j<=n;j++) 
{ 
    max[j]=pow((array[j]-mean),2); 
    sum+=max[j]; 
} 
variance=sum/(j-1); 
return sqrt(variance); 
} 

答えて

3

mean1も正しくありません。どちらのエラーが同じ理由を持って、あなたはあなたがnによってn+1ここの要素が、格差を追加している割り当てられた番号n

float mean1(float array[],int n) { 
    int i; 
    float sum=0; 
    for(i=0;i<=n;i++) 
    sum=sum+array[i]; 
return (sum/n); 
} 

過去の要素にアクセスします。 n+1 st要素は計算に属しません。ループ条件をi < nにします。あなたのバブルソートで

float median1(float array[],int n) { 
float temp; 
int i,j; 
for(i=n-1;i>=0;i--) 
    for(j=0;j<=i;j++) 

、あなたは任意の値を持っている(とn == 100が割り当てられた配列を過ぎている場合は、それゆえ未定義の動作の原因となる)が、あまりにもarray[n]にアクセスします。内側のループの状態をj < iにするか、外側のループをi = n-2で開始します。

standarddeviation1でも、配列の境界を越えてしまいます。

+4

OPの重要な点は、N個の要素がある場合、有効なインデックスは「0」から「N-1」になります(もちろん0から始まると仮定します)。 – Marlon

+0

@ダニエル・フィッシャー 私は理解していますが、私はここから何を提案しますか? – user1145538

関連する問題