为什么输出总是空的？

Question

我试图解决hackerrank问题Between Two Sets。在下面代码的getTotalX（）函数中，程序的输出总是空的（当我尝试使用cin输出时，计数值总是为零） . 看不懂问题，是不是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. 您在内部 for 循环中递增 i，但实际上您是在迭代，循环使用变量 j
2. 如果flag==1，那么您正在递增计数。但是只要一个数字可整除（数组中的任何数字）就终止。这与问题的假设相反
   

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

基于您编写的代码的建议
1. 读清楚题目，用纸笔解答。
2. 分析您的解决方案的复杂性并尝试改进它。在这里，您应用了排序 + 两个 for 循环
3. 请避免在for循环中使用i、j、k这些类型的变量。这是一种非常糟糕的做法，而且更容易出错。