使用指针将数组传递给函数

Question

我正在尝试使用指针而不是数组来打印指针数组，但是在运行时出现此错误Segmentation fault：

enter number of element:5
array[0]=1
array[1]=2
array[2]=3
array[3]=4
array[4]=5
Segmentation fault

这是代码：

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

int *array;
int n;

void input(int *array,int n);
void display(int *array,int n);
int sum(int *array,int n);

int main (void) {
   int result;
   printf("enter number of element:");scanf("%d",&n);
   input(array,n);  
   display(array,n);
   result=sum(array,n);
   printf("sum of array=%d",result);
   return 0;
}

void input(int *array,int n){
   int j;
   array=(int *)malloc(n*sizeof(int));
   for(j=0;j<n;j++){
        printf("array[%d]=",j);scanf("%d",array+j);
   }
}

void display(int *array,int n){
    int j;

    for(j=0;j<n;j++)
        printf("%d\t",*(array+j));
    printf("\n");
}

 int sum(int *array,int n){
     int sum=0,j;

     for(j=0;j<n;j++)
       sum+=*array+j;

    return sum;
}

如何修复此代码？请有人向我解释该代码有什么问题。

Answer 1

变量array是函数input中的局部变量。

同样地，用array = ...进行设置是没有意义的，因为此分配仅在inside函数中生效。通常，您应该将其地址（&array）传递给需要更改其地址的任何函数。

在您的特定示例中，您还具有全局变量array，因此，解决问题的一种快速方法是只调用函数input，而无需将变量array作为参数传递：

void input(int n)
{
    ...
    array = (int*)malloc(n*sizeof(int));
    ...
}

int main()
{
    ...
    input(n);
    ...
}

请注意，这是一个“肮脏”的解决方法，通常应避免使用全局变量。

Answer 2

将纯净版本添加到barak's answer：

int input(int ** array, const size_t n)
{
  int result = 0;

  assert(NULL != array);

  (*array) = malloc(n * sizeof(**array));
  if (NULL == (*array))
  {
    result = -1;
  }
  else
  {
    size_t j;
    for(j = 0; j < n; ++j)
    {
      printf("array[%zu]=", j);
      scanf("%d", (*array) + j); /* still missing error checking here . */
    }
  }

  return result;
}

并这样称呼：

if (-1 == input(&array, n))
{
  perror("input() failed");
  exit(EXIT_FAILURE);
}

Answer 3

尝试此input()：

void input(int **array,int n){
   int j;
   *array=(int *)malloc(n*sizeof(int));
   for(j=0;j<n;j++){
        printf("array[%d]=",j);scanf("%d",*array+j);
   }
}

由于C使用传递值，如果要更改函数中变量的值，则需要将该变量的地址作为该函数的参数传递。

在这种情况下，您想在array中更改input()的值，并且array的类型为int *，因此input()的原型应该类似于void input (int **array, ...)。

Answer 4

这应该做。确保您了解其他人说的话。。

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

int *array;
int n;


void input(int **array,int n);
void display(int **array,int n);
int sum(int **array,int n);

int main (void) {
int result;
printf("enter number of element:");scanf("%d",&n);
input(&array,n);
display(&array,n);
result = sum(&array,n);
printf("sum of array= %d",result);
return 0;
}

void input(int **array,int n){
int j;
*array= malloc(n*sizeof(int));
for(j=0;j<n;j++){
    printf("array[%d]=",j);
    scanf("%d",(*array)+j);
}
}

void display(int **array,int n){
int j;

for(j=0;j<n;j++){
    printf("%d\t",*((*array)+j)); // you can use array notation aswell
                                   //array[0][j] will work
}
printf("\n");
}

int sum(int **array,int n){
int sum=0,j;

for(j=0;j<n;j++){
    sum += *((*array)+j);
}
return sum;
}

Answer 5

*array + j是做什么的？它会评估*array并添加j吗？还是将j添加到数组然后取消引用？如果我告诉你你错了，你愿意下注100美元吗？

通过使用括号，甚至更好地编写数组[j]，可以使您的生活以及任何人阅读代码的生活变得更轻松。