我需要一个测试用例来证明我的算法/代码是错误的

时间:2012-11-05 09:58:21

标签: c++ algorithm testcase

我正在创建一个程序,它找到最大长度的子数组,其总和最接近0。 实际上我试图解决这个问题:http://opc.iarcs.org.in/index.php/problems/DEVIOUS,但是当我提交我的代码时,它只给出了50%的测试用例的正确答案。 我的代码工作正常,甚至为我自己的测试用例提供了正确的答案,所以我需要一个测试用例来证明我的代码/算法错误,这样我才能在代码中找到错误。

这是我的算法,

  
      
  1. 创建一个等于输入数组大小的数组'temp',初始化temp的所有元素,temp [i] =所有元素之和直到input [i]
  2.   
  3. 对数组temp进行排序,在temp中找到'i',使temp [i-1] - temp [i]最接近0
  4.   

以下是我从上面的链接解决问题的代码。

#include <iostream>
#include <algorithm>
#include <stdlib.h>
#include <limits.h>

using namespace std;

struct element{
    int index;
    int value;
};

bool compare(element a, element b){
    return a.value < b.value;
}

int main(){
    int n;
    cin >> n;
    int array [n];
    element elements[n];
    int sum = 0;

    for(int i = 0 ;i < n; i ++){
        cin >> array[i];
        sum = sum + array[i];
        elements[i].index = i;
        elements[i].value = sum;
    }

    sort(elements,elements+n,compare);

    int diff = INT_MAX;
    int i1 = 0,i2 = 0;
    for(int i = 1;i < n;i ++){
        int temp = elements[i].value - elements[i-1].value;
        if(temp < diff){
            diff = temp;
            i1 = elements[i].index;
            i2 = elements[i-1].index;
        }
        else if(temp == diff){
            if(abs(elements[i].index - elements[i-1].index) > abs(i1-i2)){
                i1 = elements[i].index;
            i2 = elements[i-1].index;
            }
        }
    }

    diff = 0;
    int s,e;
    if(i1 >= i2){
        e = i1, s = i2+1;
    }else{
        s = i1+1, e = i2;
    }

    for(int i = s;i <= e ; i++){
        diff = diff+array[i];
    }
    cout << diff << endl;

    cout << s+1 << " " << e+1;

    return 0;
}
编辑:我在代码中发现了一个错误并对其进行了编辑,它现在为80%的输入提供了正确的答案

1 个答案:

答案 0 :(得分:1)

  

换句话说,如果车站的利润/损失是p1,p2,......,   pN部长希望移交序列i,i + 1,...,j等   pi + pi + 1 + ... + pj的绝对值被最小化。如果有   是一个具有这个最小绝对值的序列,然后是他   我想交出最长的一个

我认为你的算法有些错误:

  

temp [i] =输入[i]之前的所有元素的总和   对数组临时排序

序列的开头可能是数组开头的另一个位置 - 它可以是一个不要忘记的子序列。

-1 | -2 | 10 | 5 |-10 | 6

   |    |    | x | x  | x

edit2:现在删除了错误的解释我看了代码 -

if(temp < diff){
        diff = temp;
        i1 = elements[i].index;
        i2 = elements[i-1].index;
    }

您确实偏好-ve数字(丢失)。可能是因为它有一个更接近的数字为零,因为它要么稍微小于-ve或+ ve(但是在abs值较小的地方)?