今天发现的两处代码问题

1. 插入排序

void insert_sort(int *nums, int s, int e)
{
        int i, j;
        for(i = s+1; i <= e; ++i)
        {
                j = i-1;
                int k = nums[i];
                while( j >= s && nums[j] > nums[i])
                {
                        nums[j+1] = nums[j];
                        --j;
                }
                nums[j+1] = num[i];
        }
}

因为插入排序会改变数组元素的位置,所以应该要把被比较的那个元素的值记下来:

void insert_sort(int *nums, int s, int e)
{
        int i, j;
        for(i = s+1; i <= e; ++i)
        {
                j = i-1;
                int k = nums[i];
                while( j >= s && nums[j] > k)
                {
                        nums[j+1] = nums[j];
                        --j;
                }
                nums[j+1] = k;
        }
}

2. 使用 realloc:

int *array = (int *)malloc(sizeof(int) * numsSize);

                                        if( idx + 3 >= size)
                                        {
                                                array= realloc(array, size + numsSize);
                                                size += numsSize;
                                        }
                                        array[idx] = nums[i];
                                        array[idx+1] = nums[j];
                                        array[idx+2] = nums[k];

代码执行的时候出现memory corruption (fast)的问题。用 valgrind跟踪发现有写内存越界的问题。realloc那行需要修改为:

array= realloc(array, (size + numsSize)*sizeof(int));

 

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值