c语言小白,程序有问题,望大佬指点

题目如下:

给定一个整数数组 nums 和一个整数目标值 target,请你在该数组中找出 和为目标值 的那 两个 整数,并返回它们的数组下标。

你可以假设每种输入只会对应一个答案。但是,数组中同一个元素不能使用两遍。

你可以按任意顺序返回答案。(力扣上面的一道题)

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

int* twoSum(int* nums, int numSize, int target)
{
    int i = 0;
    int j = 0;
    int *ret = NULL;
    for (i = 0; i < numSize - 1; i++) {
        for (j = 1; j < numSize; j++) {
            if (nums[i] + nums[j] == target) {
                ret = (int*)malloc(sizeof(int) * 2);
                ret[0] = i;
                ret[0] = j;
                break;
            }
        }
    }
    return ret;
}

int main()
{
    int nums[] = { 2,7,11,15 };
    int target = 9;
    int numSize = 4;
    int ret[] = *twoSum(nums, numSize, target);
    for (int i = 0; i < numSize; i++) {
        printf("%d", ret[i]);
    }
    return 0;
}

想问问大佬们程序哪里有问题

 

// 我先帮你把我看到的问题指出来


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

int* twoSum(int* nums, int numSize, int target)
{
    int i = 0;
    int j = 0;
    int *ret = NULL;
    for (i = 0; i < numSize - 1; i++) {
        for (j = 1; j < numSize; j++) {
            if (nums[i] + nums[j] == target) {
                ret = (int*)malloc(sizeof(int) * 2);
                ret[0] = i;
                ret[0] = j;    // 这里因该改为ret[1] = j;
                break;         // 这里要优化一下,应该是跳出最外层的循环
            }
        }
    }
    return ret;
}

int main()
{
    int nums[] = { 2,7,11,15 };
    int target = 9;
    int numSize = 4;
    int ret[] = *twoSum(nums, numSize, target);    // 这里函数前面不能加*,加了*,整个返回值就是个整形了,而且等号右边数组初始话这种写法也不对。应该改成int* ret = twoSum(nums, numSize, target);
    
    // 下面循环次数应该是2,不是numSize,而且 ret有可能是NULL, 使用前应该判断NULl, 如果返回值不为NULL, 后面也要记得释放内存
    for (int i = 0; i < numSize; i++) {
        printf("%d", ret[i]);
    }

    // 修改如下:
    if(ret != NULL){
        for(int i = 0; i < 2; i++){
            printf("%d\n", ret[i]);
        }
        free(ret);
    }
    
    return 0;
}

参考程序:

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

int* twoSum(int* nums, int numSize, int target)
{
    int i = 0;
    int j = 0;
    int *ret = NULL;
    for (i = 0; i < numSize - 1; i++) {
        for (j = i+1; j < numSize; j++) {
            if (nums[i] + nums[j] == target) {
                ret = (int*)malloc(sizeof(int) );
                ret[0] = i;
                ret[1] = j;
                ret[2] = nums[i];
                ret[3] = nums[j];
                break;
            }
        }
    }
    return ret;
}

int main()
{
    int nums[] = { 2,7,11,15 };
    int target = 9;
    int numSize = 4;
    int *ret = twoSum(nums, numSize, target);
    printf("下标:%d %d\n数值:%d %d", ret[0], ret[1], ret[2], ret[3]);
    return 0;

首先,你的main函数中int ret[]那里,先不说没有定义数组等于指针的;再且你的函数返回指针加*运算符也不能当做数组整体来用,
用个指针去定义ret等于函数的返回值就可以了。
然后你的自定义函数中,你的ret既然是指针,你去sizeof(int)*2干嘛,你只要申请内存并强制转化成指针就行了
ret[0] = i和ret[0] = j不冲突吗?
当你获得了下标之后, 回到主函数中,for循环下标越界,你后面的ret[2]和ret[3]为随机值