Question

我正在尝试实现以下C ++算法的Java版本：

void constructPrintLIS(int arr[], int n)
{
    std::vector< std::vector<int> > L(n);

    L[0].push_back(arr[0]);

    for (int i = 1; i < n; i++)
    {
        for (int j = 0; j < i; j++)
        {
            if ((arr[i] > arr[j]) &&
                (L[i].size() < L[j].size() + 1))
            {
                L[i] = L[j];
                cout << true << endl;
            }
            else
            {
                cout << false << endl;
            }
        }

        L[i].push_back(arr[i]);
    }

    std::vector<int> max = L[0];

    for (std::vector<int> x : L)
    {
        if (x.size() > max.size())
        {
            max = x;
        }
    }

    printLIS(max);
}

这是Java版本

private static List<Integer> getLongestIncreasingSubsequence(
        List<Integer> sequence
        )
{   
    ArrayList<ArrayList<Integer>> cache = 
            new ArrayList<ArrayList<Integer>>(sequence.size());
    // Populate the elements to avoid a NullPointerException
    for(int i = 0; i < sequence.size(); i++)
    {
        cache.add(new ArrayList<Integer>());
    }
    cache.get(0).add(sequence.get(0));

    // start from the first index, since we just handled the 0th
    for(int i = 1; i < sequence.size(); i++)
    {
        // Add element if greater than tail of all existing subsequences
        for(int j = 0; j < i; j++)
        {
            if((sequence.get(i) > sequence.get(j)) 
                    && (cache.get(i).size() < cache.get(j).size() + 1))
            {
                cache.set(i, cache.get(j));
            }
        }
        cache.get(i).add(sequence.get(i));                  
    }

    // Find the longest subsequence stored in the cache and return it
    List<Integer> longestIncreasingSubsequence = cache.get(0);
    for(List<Integer> subsequence : cache)
    {
        if(subsequence.size() > longestIncreasingSubsequence.size())
        {
            longestIncreasingSubsequence = subsequence;
        }
    }
    return longestIncreasingSubsequence;
}

我不明白我在做什么不同。当测试序列为{9766, 5435, 624, 6880, 2660, 2069, 5547, 7027, 9636, 1487}时，C ++算法会打印正确的结果，正确的结果为624, 2069, 5547, 7027, 9636。但是，我编写的Java版本返回624, 6880, 2660, 2069, 5547, 7027, 9636, 1487的错误结果，我不明白为什么。我试过在调试器中跟踪它，但我无法弄清楚出了什么问题。

我尝试添加一个print语句，指示if语句每次都被评估为true / false，并将其与C ++程序进行比较，并且它是相同的，所以这不是问题所在。

我怀疑它与vector和ArrayList之间的细微差别有关，但我不知道。

Answer 1

我怀疑问题是在Java中，缓存包含引用到列表，而在C ++中它包含列表本身。

因此，在C ++中

L[i] = L[j];

将索引j的列表复制到索引i，而在Java

中

cache.set(i, cache.get(j));

复制参考。这意味着，当您随后向一个项目添加项目时，它们也会添加到另一个项目中。

也许使用

cache.set(i, new ArrayList<>(cache.get(j)));

以便您创建副本，就像在C ++中一样。

这个C ++函数与等效的java函数的工作方式有何不同？

1 个答案: