C指针，最小值和最大值

Question

我需要编写一个将num1，num2和num3作为参数的函数。 一旦函数运行完毕，发送到主函数的参数（num1，num2和num3）应该更改，以使num1包含最小值，num2包含第二大值，num3包含最大值。 我还需要编写交换功能并使用它。 这是我的代码：

#include <stdio.h>
#include <stdlib.h>
void SmallBig(int* num1, int* num2, int* num3);
void swap(int* x, int* y);
void main()
{
    int *num1, *num2, *num3;
    *num1 = 1;
    *num2 = 69;
    *num3 = 22;
    printf("%d %d %d\n", *num1, *num2, *num3);
    SmallBig(*num1, *num2, *num3);
    printf("%d %d %d\n", num1, num2, num3);
    system("PAUSE");

}
void SmallBig(int* num1, int* num2, int* num3)
{
    if (*num1 > *num3 && *num2 > *num3 && *num1 > *num2)
    {
        swap(num1, num3);
        swap(num2, num2);
        swap(num3, num1);
    }
}
void swap(int* x, int* y)
{
    int temp;
    temp = *x;
    *x = *y;
    *y = temp;
}

这些是我得到的错误：

Warning 1   warning C4047: 'function' : 'int *' differs in levels of indirection from 'int' 
Warning 2   warning C4024: 'SmallBig' : different types for formal and actual parameter 1   
Warning 3   warning C4024: 'SmallBig' : different types for formal and actual parameter 2   
Warning 4   warning C4024: 'SmallBig' : different types for formal and actual parameter 3   
Error   5   error C4700: uninitialized local variable 'num1' used   
Error   6   error C4700: uninitialized local variable 'num2' used   

错误7错误C4700：使用了未初始化的局部变量'num3'

请帮我修复程序！ 谢谢。 阿农

Answer 1

您遇到的第一个问题是

SmallBig(*num1, *num2, *num3);

您正在将引用的指针作为参数传递-这些指针的类型为int而不是int* 。 您的SmallBig函数需要指针，因此应这样称呼它

SmallBig(num1, num2, num3);

您的代码至少现在应该可以使用此更改进行编译。

第二个主要问题是您在执行此操作时将值写入未初始化的内存

int *num1, *num2, *num3;
*num1 = 1;
*num2 = 69;
*num3 = 22;

写入未初始化的内存是未定义的行为，充其量将导致段错误。 一种更安全的方法是

int num1,num2,num3;
num1 = 1;
num2 = 69;
num3 = 22;

然后将这些int变量的地址传递给需要int*参数的函数。 如果您不喜欢这种方式，则需要动态分配指针。 您不能在未预料到麻烦的情况下写未初始化指针指向的内容。

最后，这条线

swap(num2, num2);

没有什么用

Answer 2

错误和问题

关于您的错误：

int *num1, *num2, *num3;  // you define 3 pointers to int but let them unitinialised !
*num1 = 1;                 // you write something to the poinded adress, which might be random !!
*num2 = 69;
*num3 = 22;

当您使用指针而不初始化它们（仅初始化所指向的值）时，会收到此错误消息

然后，您的第二个printf()打印指针，而不打印指针的值。

最后，SmallBig（）仅在少数情况下有效！

解

在main()使用整数，而不是指针：

int num1, num2, num3;
num1 = 1;
num2 = 69;
num3 = 22;
printf("%d %d %d\n", num1, num2, num3);
SmallBig(&num1, &num2, &num3);   // use the address 
printf("%d %d %d\n", num1, num2, num3);

我差点忘了你的SmallBig()函数！ 在那里，继续像您一样使用指针。 但是您必须预见更多情况：

   if (*num1 > *num3) 
        swap (num1, num3); 
   if (*num1 > *num2) 
        swap (num1, num2); 
   if (*num2 > *num3) 
        swap(num2, num3);