[已解决]求助，在函数的默认参数这里出错了

gkz525

#include <iostream>
using namespace std;

void get_min_max(int src[], int arr_len, int* max_v = NULL, int* min_v = NULL)
{
        if (arr_len < 0 || (!max_v && !min_v)) return;

                int min_value = src[0];
                int max_value = src[0];
        for (int i = 1; i < arr_len; ++i)
        {
                        if (min_value > src[i]) min_value = src[i];
                        if (max_value < src[i]) max_value = src[i];
        }
        if (min_value) *min_v = min_value;
        if (max_value) *max_v = max_value;
}
int main()
{
        int src[] = { 66, 33, 55, 22, 88, 89, 19 };
        int max_v = 0;
        get_min_max(src, sizeof(src) / sizeof(int), &max_v);

        cout << "max_v = " << max_v << endl;

        return 0;
}

一运行就说：exe已停止工作。
调试时：引发了异常: 写入访问权限冲突。min_v 是 nullptr。
求大家帮忙看看要怎么改？

最佳答案

月排行榜 / 总排行榜

User_Ghost

2020-8-7 13:11:48

gkz525 发表于 2020-8-7 12:06
我只想要最大值，不想要最小值

您是说只要最大值，所以才只传入接收最大值的变量

但是函数在执行的时候，注意一下下边代码的10和12行，min_value和max_value是函数的局部变量，也就是说，做完了for循环以后，min_value和max_value已经分别得到最小值和最大值。但是，你的if语句却是用这两个最值作为判断，那么只要两个最值不为0，if语句都会执行该句的代码。

而您只想要最大值，所以没有传入接受最小值的地址，默认参数为NULL，也就是空地址。此时注意上边讲的话，您用的是最值局部变量作为判断，只要不为0都会执行，做完for循环以后，两个变量可能都不为0，那执行到 if (min_value)以后，就会做这句的代码块，但是没有传入min_v的地址，所以为空地址，那你说说空地址能用来读写吗？所以程序才会报错。我估计您是打错了代码，if里面应当是对两个指针变量地址的空地址判断，也就是if ( min_v != NULL ) 和 if ( max_v != NULL )

1. void get_min_max(int src[], int arr_len, int *max_v = NULL, int *min_v = NULL)
   
2. {
   
3.     if (arr_len < 0 || (!max_v && !min_v))
   
4.         return;
   
5. 
   
6.     int min_value = src[0];
   
7.     int max_value = src[0];
   
8.     for (int i = 1; i < arr_len; ++i)
   
9.     {
   
10.         if (min_value > src[i])
    
11.             min_value = src[i];
    
12.         if (max_value < src[i])
    
13.             max_value = src[i];
    
14.     }
    
15.     if (min_value)
    
16.         *min_v = min_value;
    
17.     if (max_value)
    
18.         *max_v = max_value;
    
19. }

复制代码

关于您说只想要最大值不想要最小值。总而言之，您只想要最大值，没打算要最小值，而函数却帮你算出了最小值，又因为判断标准错误，对空地址读写，才导致程序崩溃的。

另外，您可以尝试在数组里加上0，然后程序也可以正常运行

跳转到最佳答案楼层

User_Ghost

gkz525 发表于 2020-8-7 12:06
我只想要最大值，不想要最小值

您是说只要最大值，所以才只传入接收最大值的变量

但是函数在执行的时候，注意一下下边代码的10和12行，min_value和max_value是函数的局部变量，也就是说，做完了for循环以后，min_value和max_value已经分别得到最小值和最大值。但是，你的if语句却是用这两个最值作为判断，那么只要两个最值不为0，if语句都会执行该句的代码。

而您只想要最大值，所以没有传入接受最小值的地址，默认参数为NULL，也就是空地址。此时注意上边讲的话，您用的是最值局部变量作为判断，只要不为0都会执行，做完for循环以后，两个变量可能都不为0，那执行到 if (min_value)以后，就会做这句的代码块，但是没有传入min_v的地址，所以为空地址，那你说说空地址能用来读写吗？所以程序才会报错。我估计您是打错了代码，if里面应当是对两个指针变量地址的空地址判断，也就是if ( min_v != NULL ) 和 if ( max_v != NULL )

1. void get_min_max(int src[], int arr_len, int *max_v = NULL, int *min_v = NULL)
   
2. {
   
3.     if (arr_len < 0 || (!max_v && !min_v))
   
4.         return;
   
5. 
   
6.     int min_value = src[0];
   
7.     int max_value = src[0];
   
8.     for (int i = 1; i < arr_len; ++i)
   
9.     {
   
10.         if (min_value > src[i])
    
11.             min_value = src[i];
    
12.         if (max_value < src[i])
    
13.             max_value = src[i];
    
14.     }
    
15.     if (min_value)
    
16.         *min_v = min_value;
    
17.     if (max_value)
    
18.         *max_v = max_value;
    
19. }

复制代码

关于您说只想要最大值不想要最小值。总而言之，您只想要最大值，没打算要最小值，而函数却帮你算出了最小值，又因为判断标准错误，对空地址读写，才导致程序崩溃的。

另外，您可以尝试在数组里加上0，然后程序也可以正常运行

User_Ghost

这一句，你传入参数的时候没有传入min_v，所以是NULL，但是你的min_value可能并不为0，而min_v又可能为NULL

1. if (min_value)
   
2.         *min_v = min_value;

复制代码

改的话，我不知道你想要什么。我这是解决上边问题后可以运行，不会报错
可能的话，可以对两个指针变量判断一下是否为空地址，然后就可以正常运行了

tony666

你 min_v  就没传啊，可不是nullptr吗

gkz525

tony666 发表于 2020-8-7 11:41
你 min_v  就没传啊，可不是nullptr吗

我不想要最小值min,所以NULL不用传入也是可以的啊

gkz525

User_Ghost 发表于 2020-8-7 11:28
这一句，你传入参数的时候没有传入min_v，所以是NULL，但是你的min_value可能并不为0，而min_v又可能为NULL ...

我只想要最大值，不想要最小值

gkz525

User_Ghost 发表于 2020-8-7 13:11
您是说只要最大值，所以才只传入接收最大值的变量

但是函数在执行的时候，注意一下下边代码的10和12 ...

谢谢你，已解决

gkz525

tony666 发表于 2020-8-7 11:41
你 min_v  就没传啊，可不是nullptr吗

好的，谢谢你