Я пытаюсь решить это упражнение в HackerEarth. Но у меня ошибка срок превышен. Это код, который я написал:
import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.util.*;
class TestClass {
//gcd
public static long gcd(long num1, long num2) {
if (num2 != 0) {
return gcd(num2, num1 % num2);
} else {
return num1;
}
}
public static void main(String args[] ) throws Exception {
//BufferedReader
BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
int T = Integer.parseInt(br.readLine()); // Reading input from STDIN
while (T-- > 0) {
StringTokenizer st1 = new StringTokenizer(br.readLine());
long a = Long.parseLong(st1.nextToken());
long b = Long.parseLong(st1.nextToken());
long A = a/gcd(a,b);
long B = b/gcd(a,b);
System.out.printf("%d%1s%d%n",B,"",A);
}
}
}
Возможно, вам следует попробовать итеративную реализацию GCD, чтобы избежать вызовов функций. Также примите во внимание то, что отметил @amit (gcd
вычисляется дважды для одних и тех же аргументов).
Кроме того, вы можете использовать int
вместо long
везде. Указанный диапазон ввода будет аккуратно помещаться в int.
Но в любом случае большая часть времени фактически тратится на ввод-вывод, чем на фактические вычисления.
Выполнение ввода-вывода и загрузка кода JVM/загрузки и т. д.
Также обратите внимание, что javadoc для StreamTokenizer
указывает «StringTokenizer
— это устаревший класс, который сохранен по соображениям совместимости, хотя его использование — обескуражен в новом коде».
Ваше решение немного медленное из-за плохой реализации. Я переписал ваше решение в лучшей реализации с той же логикой и временной сложностью, что и ваше решение, и получил Принятый, и ни один из тестовых случаев не превысил 0,8 секунды.
import java.util.*;
class TestClass {
// Same gcd function but it's better code :)
public static int gcd(int a, int b) {
return b == 0 ? a : gcd(b, a % b);
}
public static void main(String args[] ) throws Exception {
Scanner s = new Scanner(System.in);
int t = s.nextInt(); // this way of reading input is faster alot.
while(t-- > 0) {
int a = s.nextInt();
int b = s.nextInt(); // No need to use long it's just 1e9
int tmp = gcd(a, b); // It's better to save the value of the gcd(a, b) instead of calculate it twice.
int A = a/tmp;
int B = b/tmp;
System.out.println(B+" "+A);
}
}
}
Во-первых, вы не должны вычислять
gcd(a,b)
дважды. Теперь, как правило, это обычно оптимизируется для вас, но, возможно, эта оптимизация компилятора неактивна для онлайн-судьи.