Question

我正在练习最长递增序列问题，并且正在使用以下代码：

int lis(vector<int>& a)
{
    int s = a.size();
    vector<int> prev(s);
    vector< vector<int> > pile;
    int m = -1; // lis size
    for(int i=0; i<s; i++)
    {
        // initialize piles
        vector<int> vt;
        pile.push_back(vt);
    }

    pile[0].push_back(0); // a[0] must go to the first pile
    prev[0] = -1;

    for(int i=1; i<s; i++)
    {
        cout << "i: " << i << endl;
        for(int j=0; j<s; j++)
        {
            cout << "j: " << j << endl;
            if(pile[j].size() == 0)
            {
                printf("pile %d is empty, push %d (%d)\n", j, i, a[i]);

                pile[j].push_back(i);
                if(j == 0)
                {
                    prev[i] = -1;
                }
                else
                {
                    prev[i] = pile[j-1].back();
                }

                m = max(j, m);
                break;
            }
            else if(a[pile[j].back()] < a[i])
            {
                printf("go to pile %d\n", j+1);
                continue;
            }
            else
            {
                printf("push %d (%d) to pile %d\n", i, a[i], j);

                pile[j].push_back(i);
                if(j == 0)
                {
                    prev[i] = -1;
                }
                else
                {
                    prev[i] = pile[j-1].back();
                }

                m = max(j, m);
                break;
            }
        }
    }

    // print out the sequences
    int ind = pile[m][0];
    vector<int> res;
    while(ind != -1)
    {
        res.push_back(a[ind]);
        ind = prev[ind];
    }
    for(int i=res.size()-1; i>=0; i--)
    {
        cout << res[i] << " ";
    }

    return m+1; // return lis size
}

过去，工作正常。然后，我想简化代码并提出：

int lis(vector<int>& a)
{
    /* same */

    for(int i=1; i<s; i++)
    {
        cout << "i: " << i << endl;
        for(int j=0; j<s; j++)
        {
            cout << "j: " << j << endl;
            if(a[pile[j].back()] > a[i] || pile[j].size() == 0)
            {
                printf("push %d to pile %d\n", a[i], j);

                pile[j].push_back(i);
                prev[i] = (j == 0) ? -1 : pile[j-1].back();                    
                m = max(j, m);

                break;
            }
            printf("go to pile %d\n", j+1);
        }
    }

    /* same */
}

使用此代码，当移至下一个堆（{j增加）时，程序突然退出。（流程返回了0xC0000005）
这不可能是缓冲区溢出或分段错误，因为每个向量/数组都已正确声明且索引未超出界限。
这也不会是内存泄漏或悬空指针，因为我没有使用任何手动分配。
我无法解决这个问题！

我做错了什么？

Answer 1

首先，在以下语句中：

if(a[pile[j].back()] > a[i] || pile[j].size() == 0)

我相信您应该先检查表达式的第二部分。

换句话说，请先检查pile[j].size() == 0，以避免调用pile[j].back()（如果为true）。

程序退出原因不明（0xC0000005）

1 个答案: