qq_17591789
shackerist
采纳率75%
2021-03-02 22:39

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;
}

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

 

  • 点赞
  • 写回答
  • 关注问题
  • 收藏
  • 复制链接分享
  • 邀请回答

4条回答

  • Huoon Huoon 1月前
    // 我先帮你把我看到的问题指出来
    
    
    #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;
    }
    点赞 2 评论 复制链接分享
  • qq_43414873 小白小白你好菜 1月前

    点赞 1 评论 复制链接分享
  • qq_43414873 小白小白你好菜 1月前

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

    点赞 1 评论 复制链接分享
  • qq_43414873 小白小白你好菜 1月前

    参考程序:

    #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;

    点赞 1 评论 复制链接分享