记录本算法小白刷力扣的这道题遇到的报错

349. 两个数组的交集https://leetcode.cn/problems/intersection-of-two-arrays/

出现报错的代码

/** * Note: The returned array must be malloced, assume caller calls free(). */int* intersection(int* nums1, int nums1Size, int* nums2, int nums2Size, int* returnSize){int hash[1000]={0};int result[1000];//交集是去重的，最多只有1000个数for(int i=0;i<nums1Size;i++){//将nums1录入哈希表hash[nums1[i]]++;}int j=0;for(int i=0;i<nums2Size;i++){//将nums2与哈希表比对if(hash[nums2[i]]!=0){result[j]=nums2[i];j++;hash[nums2[i]]=0;//手动去重}}*returnSize=j;//这个是主函数输出时用的，跟内部计算关系不大return result;}

报错描述

Line 207: Char 3: runtime error: load of null pointer of type ‘int’ [__Serializer__.c]

原因分析

该题需要return一个数组，而在不申请空间（不malloc）的情况下函数内建立的数组是局部变量，无法带回主函数。

解决方法

在函数里malloc需要返回的数组（其实初始代码顶部的note里说的就是这个）。注意建立数组时在中括号里写数组大小跟malloc的效果在这方面是完全不一样的。

这里用于返回的数组是result，所以在建立result数组时要malloc

修改后的代码

/** * Note: The returned array must be malloced, assume caller calls free(). */int* intersection(int* nums1, int nums1Size, int* nums2, int nums2Size, int* returnSize){int hash[1000]={0};int *result=(int*)malloc(sizeof(int)*1000);//交集是去重的，最多只有1000个数for(int i=0;i<nums1Size;i++){//将nums1录入哈希表hash[nums1[i]]++;}int j=0;for(int i=0;i<nums2Size;i++){//将nums2与哈希表比对if(hash[nums2[i]]!=0){result[j]=nums2[i];j++;hash[nums2[i]]=0;//手动去重}}*returnSize=j;//这个是主函数输出时用的，跟内部计算关系不大return result;}

将原来的intresult[1000];

改成了 int *result=(int*)malloc(sizeof(int)*1000);

只修改了这一行，虽然非常浅显但一定要知道的错误

这里因为题目数据限制为

  • 1 <= nums1.length, nums2.length <= 1000
  • 0 <= nums1[i], nums2[i] <= 1000

数字大小不会超过1000，可以直接开一个大小为1000的数组（虽然烧了不必要的内存）。实际上返回数组是去重的，题目给的两个数组的重复数据个数不会超过数据个数比较小的那个，这样会比较省内存（但我懒得写了）。