为什么输出总是空的？

Question

我想解决黑客rank的问题。两套之间.在下面代码的getTotalX()函数中，proggram的输出总是空的（当我试图用cin输出时，count值总是给零），我不明白这个问题，是不是count变量的声明有什么问题？

    #include <bits/stdc++.h>

using namespace std;

string ltrim(const string &);
string rtrim(const string &);
vector<string> split(const string &);

/*
 * Complete the 'getTotalX' function below.
 *
 * The function is expected to return an INTEGER.
 * The function accepts following parameters:
 *  1. INTEGER_ARRAY a
 *  2. INTEGER_ARRAY b
 */

int getTotalX(vector<int> a, vector<int> b) {
    sort(a.begin(),a.end());
    sort(b.begin(),b.end());
    int n1 = a.back();
    int n2 = b.front();
    cout<<n1<<n2<<endl;
    int count=0;
    for(int i=n1;i<=n2;i+=n1)
    {
        int flag = 1;
        for(int j=0;j<a.size();j++ )
        {
            if (i % a[j])
            {
                flag =0;
                break;
            } 
        }
        if (flag)
        {
            for(int j=0;j<b.size();i++)
            {
                if (b[j]%i)
                {
                    flag = 0;
                    break;
                }
            }
        }
        if (flag)
            count++;
    }

    return count;
}

int main()
{
    ofstream fout(getenv("OUTPUT_PATH"));

    string first_multiple_input_temp;
    getline(cin, first_multiple_input_temp);

    vector<string> first_multiple_input = split(rtrim(first_multiple_input_temp));

    int n = stoi(first_multiple_input[0]);

    int m = stoi(first_multiple_input[1]);

    string arr_temp_temp;
    getline(cin, arr_temp_temp);

    vector<string> arr_temp = split(rtrim(arr_temp_temp));

    vector<int> arr(n);

    for (int i = 0; i < n; i++) {
        int arr_item = stoi(arr_temp[i]);

        arr[i] = arr_item;
    }

    string brr_temp_temp;
    getline(cin, brr_temp_temp);

    vector<string> brr_temp = split(rtrim(brr_temp_temp));

    vector<int> brr(m);

    for (int i = 0; i < m; i++) {
        int brr_item = stoi(brr_temp[i]);

        brr[i] = brr_item;
    }

    int total = getTotalX(arr, brr);

    fout << total << "\n";

    fout.close();

    return 0;
}

string ltrim(const string &str) {
    string s(str);

    s.erase(
        s.begin(),
        find_if(s.begin(), s.end(), not1(ptr_fun<int, int>(isspace)))
    );

    return s;
}

string rtrim(const string &str) {
    string s(str);

    s.erase(
        find_if(s.rbegin(), s.rend(), not1(ptr_fun<int, int>(isspace))).base(),
        s.end()
    );

    return s;
}

vector<string> split(const string &str) {
    vector<string> tokens;

    string::size_type start = 0;
    string::size_type end = 0;

    while ((end = str.find(" ", start)) != string::npos) {
        tokens.push_back(str.substr(start, end - start));

        start = end + 1;
    }

    tokens.push_back(str.substr(start));

    return tokens;
}

Answer 1

遇到的问题

1. 你正在增加 i 内的for循环，但实际上你是在迭代，循环中使用变量 j
2. 如果 flag==1 那么你就是在递增计数。但只要一个数是可除数（也就是数组中的任何一个数）就终止。这违背了问题的假设。

提示 1. 如果你观察所给问题中的第1点和第2点，那只是LCM和GCD定义的另一种说法。

根据你所写的代码提出建议 1. 读清题目，用纸笔解决。 2. 分析你的解法的复杂程度，并尝试改进。这里你应用了排序+两个for循环3. 请避免使用 i, j, k 这些类型的变量在for循环里面。这是一种非常不好的做法，更容易出错。