Question

编辑：解决了！我将负数测试用例视为0，而不是输出也是负数。谢谢你的帮助！

以下是挑战说明：https://www.codeeval.com/open_challenges/17/
我一直得到部分解决的分数。我想知道为什么。就像在我看来，这段代码有效。我相信它是O（N）时间。谢谢你的期待！

这是我的代码：

    #include <iostream>
    #include <fstream>
    #include <cstdlib>
    #include <vector>
    #include <string>
    #include <sstream>

    using namespace std;

    int max(int a, int b)
    {
        if (a > b)
            return a;
        else return b;
    }


    int maxSubArray(vector<int> values)
    {
        int max_so_far = values[0];
        int curr_max = values[0];
        for(int i = 1; i < values.size(); ++i)
        {
            curr_max = max(values[i], curr_max + values[i]);
            max_so_far = max(max_so_far, curr_max);
        }
        return max_so_far;
    }

    int main(int argc, char *argv[]) 
    {
        std::vector<vector<int> > Values; //to hold the values of the stock price change
        ifstream file(argv[1]);
        std::string line; //for the txt file input
        int value = 0; //for holding the value of stock change

        while (std::getline(file, line)) 
        {
            int pos = 0;
            if(line.length() == 0)
                continue;
            else
            {
                std::istringstream iss(line);
                std::vector<int> list; // temporary list of values to be pushed back into the 2-d vector
                while (iss >> value)
                {   
                    list.push_back(value);  
                }

                Values.push_back(list);
            }
        }

        for(int i = 0; i < Values.size(); ++i)
        {
            cout << maxSubArray(Values[i]);
            cout << endl;
        }
    /*  
        cout << " Printing the values : " << endl;
        for (int j = 0; j < Values.size(); ++j)
        {
            for (int k = 0; k < Values[j].size(); ++k)
                cout << Values[j][k] << " ";
            cout << endl;
        }
    */
        return 0;
    }

Answer 1

所以我现在换掉了一些代码。我得分更高，但我还是偏袒。

#include <iostream>
#include <fstream>
#include <cstdlib>
#include <vector>

using namespace std;

int max(int a, int b)
{
    if (a > b)
        return a;
    else return b;
}


int maxSubArray(vector<int> values)
{
    int max_so_far = values[0];
    int curr_max = values[0];
    if (curr_max < 0) 
    {
        curr_max = 0;           
        max_so_far = 0;
    }
    for(int i = 1; i < values.size(); ++i)
    {
        curr_max = max(values[i], curr_max + values[i]);
        curr_max = max(curr_max, 0);
        max_so_far = max(max_so_far, curr_max);
    }
    return max_so_far;
}

int main(int argc, char *argv[]) 
{
    std::vector<vector<int> > Values; //to hold the values of the stock price change
    ifstream file(argv[1]);
    std::string line; //for the txt file input
    std::string token; //for the subtring that will be converted from char to int
    int value = 0; //for holding the value of stock change
    int count = 0;// for holding how many total cases

    while (!file.eof()) 
    {
        int pos = 0;
        getline(file, line);
        if(line.length() == 0)
            continue;
        else
        {

            std::vector<int> list; // temporary list of values to be pushed back into the 2-d vector

            while ((pos = line.find(",")) != std::string::npos )
            {
                token = line.substr(0,pos);
                value = atoi(token.c_str());
                line.erase(0, pos + 1); 
                list.push_back(value);  
            }
            value = atoi(line.c_str());
            list.push_back(value);

            Values.push_back(list);

            ++count;
        }
    }

    for(int i = 0; i < Values.size(); ++i)
    {
        cout << maxSubArray(Values[i]);
        cout << endl;
    }

    cout << " Printing the values : " << endl;
    for (int j = 0; j < Values.size(); ++j)
    {
        for (int k = 0; k < Values[j].size(); ++k)
            cout << Values[j][k] << " ";
        cout << endl;
    }

    return 0;
}

Answer 2

为什么要在这里按值传递矢量？

int maxSubArray(vector<int> values)

这看起来是一个重要的优化机会。
我认为你没有完全正确地阅读这个问题。当他们说“所有连续的子公寓”时，他们意味着您必须对i的所有j和for(idx = i; i < j; ++i) { total += vec[idx]; }采取最大值。现在你的代码基本上假设i = 0，这不是你应该做的。

只是看看他们提供的输出示例，我可以看到你的代码不会给出他们期望的答案。

Answer 3

看起来是正确的，我唯一能想到的是，当列表变长时，结果会溢出，所以将int更改为long long。

Answer 4

除了在其他答案中建议的技术优化，关于算法，我认为一点修复可以使您的算法工作。当curr_max下降到负值时，由于遇到超过curr_max的负整数，您可以简单地删除所有先前的整数，包括当前值并重新开始。这个修复很简单，你可以像这样在你的循环中添加一行：

for(int i = 1; i < values.size(); ++i)
{
    curr_max = max(values[i], curr_max + values[i]);
    curr_max = max(curr_max, 0); // <---------------- add this line
    max_so_far = max(max_so_far, curr_max);
}

最大整数总和

4 个答案: