2012-02-01 157 views
1

試圖編寫這個小程序來幫助我在我的Stats類中,除了中位數外,一切似乎都在計算。我錯過了什麼?模式,中位數,平均數,標準差,方差計算器C

額外的信貸,如果有人願意爲我做的方差功能x)。

使用GCC編譯器運行OSX。

#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

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++) 

在你冒泡排序,您可以訪問array[n]太多,具有任意值(如果n == 100是過去分配的陣列,從而導致不確定的行爲)。使內部環路條件爲j < i或使用i = n-2啓動外部環路。

standarddeviation1你也超出了數組邊界。

+4

OP要注意的重要一點是,如果你有'N'個元素,有效索引從'0'到'N-1'(假設你從0開始)。 – Marlon 2012-02-01 23:20:38

+0

@丹尼爾菲捨爾 啊我有點理解,但你會建議我從這裏做什麼? – user1145538 2012-02-02 08:40:03

相關問題