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

问题

您要在内部for循环中增加i，但实际上您正在迭代，并使用变量j
如果flag==1，则您将递增计数。但是，只要数字是可分割的（在数组中也是如此），就终止。这与问题

提示
1.如果您在给定的问题中观察到点1和2，则这些只是LCM和GCD定义的另一种表达方式

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

为什么输出始终为空？

1 个答案: