Question

Codeforces问题119A - http://codeforces.com/problemset/problem/119/A

我的解决方案：

#include<iostream>
using namespace std;
int a,b,i,n;
int gcd(int x,int y)
{
    int z,c;
    c=min(x,y);
    for(i=1;i<=c;i++)
    {
        if(((x%i)==0)&&((y%i)==0))
        z=i;
    }
    return z;
}
int main()
{
    cin>>a>>b>>n;
    while(1)
    {

        if(n<gcd(a,n))
        {
            cout<<"1";
            break;
        }
        else
        n-=gcd(a,n);
        if(n<gcd(b,n))
        {
            cout<<"0";
            break;
        }
        else
        n-=gcd(b,n);
    }   
}

我不熟悉时间限制。我的解决方案是在正常时间在我的计算机上运行时给出正确的结果。我应该在我的解决方案中更改它以便在时间限制下运行？我应该避免使用用户定义的函数？测试案例为-3,5,9和1,1,100。

Answer 1

您的代码肯定是准确的并且会得到正确的答案，但是可以通过一些小的改进来提高效率并成功通过时间限制。

不要在while循环中计算GCD两次。 GCD是一种昂贵的算法，运行它两次会使程序的整体完成时间翻倍。这可能是您可以进行的最重要的修复，以使您的程序不会导致TLE。
潜在地优化您的GCD算法，虽然不需要通过所有测试用例（如下所示）

仅使用gcd一次并传递代码中所有测试用例的实现可能如下所示（提交here）：

#include <iostream>
using namespace std;

int gcd(int, int);

int main()
{
    int a, b, n;
    cin >> a >> b >> n;
    while (1) {
        n -= gcd(a, n);
        if (n == 0) {
            cout << "0" << endl;
            return 0;
        }
        n -= gcd(b, n);
        if (n == 0) {
            cout << "1" << endl;
            return 0;
        }
    }
}

int gcd(int u, int v)
{
    int t;
    while ((t = u % v) != 0) {
        u = v;
        v = t;
    }
    return v;
}

我的解决方案在测试用例中超出时限。我的解决方案出了什么问题？

1 个答案: