2015-05-10 16 views
-1

当我在交换函数中传递参数时,我期望指数为a[0]a[1]的输出。但输出结果分别显示a[1]a[2]的数据。将数组作为参数传递并交换数据 - 意外的数据

但是为什么?以及如何改进它?

以下是我的代码。

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

int add(int *array,int max); 
void swap(int *array,int max,int *x,int *y); 

int main() 
{ 
    int sum,i,j=0,a[10],max; 

    scanf("%d",&max); 

    for(i=0;i<max;i++) 
    { 
     scanf("%d",&a[i]); 
    } 

    for(i=0;i<max;i++) 
    { 
     printf(" %d ",a[i]); 
    } 

    printf("\n"); 

    //for(i=0;i<max;i++) 
    { 
     swap(a,max,&a[0],&a[1]); 
    } 
    sum=add(a,max); // array name is same as address of array//or sum=add(&a[0],max); 
    printf("%d",sum); 
} 

int add(int *array,int max) 
{ 
    int sum=0,i=0; 
    while(i<max) 
    { 
     sum=sum +array[i]; 
     i++; 
    } 
    return sum; 
} 

void swap(int *a,int max, int *x,int *y) 
{ 
    int i; 
    /* int temp=0; 

    temp=a[*x]; 
    a[*x]=a[*y]; 
    a[*y]=temp;*/ 
    printf("\n%d %d",a[*x],a[*y]); 
    for(i=0;i<max;i++) 
    { 
     printf("\n %d",a[i]); 
    } 
    printf("\n"); 
} 

回答

0

您的问题是在下面的行

printf("\n%d %d",a[*x],a[*y]); 

有了这样

swap(a,max,&a[0],&a[1]); 

swap()呼叫如果a[0]a[1]的值大于10多,你正在访问阵列越界。请检查并更正您的逻辑。

FWIW,

但输出表示从数据A [1]和[2]分别

这是因为,最可能的是,a[0]包含1a[1]包含'2

+0

它并没有超出限制,因为我总是访问数组,对于i <10,也是a [0] = 1,a [1] = 2,a [2] = 3等等直到a [4] = 5,因为我采取最大= 5。对不起,我是新手编程 –

+0

@deepakpandey在询问用户输入时,你如何保证'[0] == 1'总是如此? –

+0

@deepakpandey没什么可对不起的朋友,只要逻辑思考。 :-) –

0

我试过了,现在工作正常。谢谢

int add(int *array,int max); 
void swap(int *array,int max,int *x,int *y); 

    int main() 
    { 
    int sum,i,j=0,a[10],max; 

    printf("enter the size \n"); 

     scanf(" %d",&max); 

     for(i=0;i<max;i++) 
    { 
     scanf("%d",&a[i]); 
     } 

      for(i=0;i<max;i++) 
    { 
     printf(" data entered a[%d] = %d \n ",i,a[i]); 
     } 

printf("\n"); 


printf("%d",&a[1]); 

     //for(i=0;i<max;i++) 
    { 

     swap(a,max,&a[0],&a[1]); 

     } 
     sum=add(a,max); // array name is same as address of array//or sum=add(&a[0],max); 
     printf("%d",sum); 
} 

    int add(int *array,int max) 
    { 
     int sum=0,i=0; 
     while(i<max) 
     { 


    sum=sum +array[i]; 
    i++;} 
    return sum; 
    } 

void swap(int *a,int max, int *x,int *y) 
{ 
int i,temp; 
    temp=*y; 
    *y=*x; 
    *x=temp; 

    printf("\n%d %d",*x,y); 
    for(i=0;i<max;i++) 
    { 
     printf("\n %d",a[i]); 
    } 

printf("\n"); 

}