在C中更改数字数组中的值

Question

我想制作一个使用我创建的函数的程序，该函数将数组X（长度为N）的所有元素交换为某个数字K（仅当该元素大于K时）。我在这里哪里出错了？ ？

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

int swap_K(int *, int);
int main()
{
    int N,i,K;
    printf("Enter N: ");
    scanf("%d",&N);
    printf("Enter K: ");
    scanf("%d",&K);
    int X[N];
    for (i=1; i<=sizeof(X)/sizeof(int); i++){
       printf("Enter %d. element: ",i);
       scanf("%d",&X[i]);
    }
    swap_K(X,K);

    for (i=1; i<=sizeof(X)/sizeof(int); i++){
        printf("%d",X[i]);
    }
}

int swap_K(int *X, int K)
{
    int i;
    for (i=1; i<=sizeof(X)/sizeof(int); i++){
        if (X[i]>K)
            X[i]=K;
    }
    return X;
}

Answer 1

在swap_K(int *X, int K) ， sizeof(X)是sizeof(int *) ，而不是数组的大小。

在C语言中，指针与数组并不完全相同。

要修复此问题，请在所有地方使用N代替sizeof(X)/sizeof(int) ，尤其是。 在swap_K()内部。

Answer 2

1）数组从索引0开始。

2）在您的main函数中，您无需在for循环中使用sizeof(X)/sizeof(int) ，因为您已经知道它等于N

3）当您将数组传递给函数时，您要发送的数组的基地址将衰减为指针，因此在swap_K函数中， sizeof(X)将返回sizeof(int) ，其值为4 （通常）。

为了克服这个问题，您应该从main函数发送数组的大小。 例如： swap_K(X,K,N);

4）当您从main函数发送X的基地址时，不需要从swap_K返回X

例如：

---

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

int swap_K(int *, int, int);
int main()
{
    int N,i,K;
    printf("Enter N: ");
    scanf("%d",&N);
    printf("Enter K: ");
    scanf("%d",&K);
    int X[N];
    for (i=0; i<N; i++)
    {
       printf("Enter %d. element: ",i);
       scanf("%d",&X[i]);
    }
    swap_K(X,K,N);
    for (i=0; i<N; i++)
    {
        printf("%d",X[i]);
    }
}

int swap_K(int *X, int K,int N)
{
    int i;
    for (i=0; i<N; i++)
    {
        if (X[i]>K)
            X[i]=K;
    }
   //return X;  //This is not required
}

Answer 3

您的循环不正确

for (i=1; i<=sizeof(X)/sizeof(int); i++)

它应该是

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

Answer 4

发布的代码存在几个问题：

1. C中的数组是0索引的，因此for循环应始终从0迭代到N-1。迭代N是缓冲区溢出
2. 指向数组的指针只是指向数组第一个元素的指针。 交换函数无法知道传递给它的指针是数组的一部分还是单个值。 考虑到这一点，将需要采用另一个参数，该参数告诉传入数组作为指针的大小。 循环内部的迭代将使用该值，而不是sizeof（X）/ sizeof（int）= 1
3. 您将X定义为完全在堆栈上分配的可变大小数组。 引入一个相当大的N将使您的程序崩溃。 如果您不知道输入的大小，最好在堆中分配数组。