java - 语法问题 : Ternary and loop inside return statement

Question

我在使用 Java 语法时遇到一些问题。我尝试编写一个素数检查，效果很好

public static boolean isPrime(int n) {
        boolean out;
        if (n < 2)
            out = false;
        else if (n == 2)
            out = true;
        else if (n % 2 == 0)
            out = false;
        else {
            double sqrtN = Math.sqrt(n);
            for (int i = 3; ; i += 2) {
                if (i > sqrtN)
                    return out = true;
                else if (n % i == 0)
                    return out = false;
            }
        }
        return out; 
}

现在我想写得更简洁:去掉变量声明 out 并使用 ternary 代替 if。这是我到目前为止所拥有的:

public static boolean isP(int n) {
        return
            (n < 2) ? false :
            (n == 2) ? true :
            (n % 2 == 0) ? false : {
            double sqrtN = Math.sqrt(n);
            for (int i = 3; ; i += 2) 
                if (i > sqrtN) 
                    return true;
                else if (n % i == 0)
                    return false;
            }
}

我不知道如何处理 block 语句

                {
                double sqrtN = Math.sqrt(n);
                for (int i = 3; ; i += 2) 
                    if (i > sqrtN) 
                        return true;
                    else if (n % i == 0)
                        return false;
                }

我可以在 : 之后合并这个吗？循环内的三元是无意义的/无论如何都是不可能的，如果我没有在这里遗漏一些东西，所以我在那里使用了 if/else 。

那么最外面的return显然缺少一个分号，我不知道该把它放在哪里。

希望我的意图得到传达，那么我怎样才能让我的第二个解决方案发挥作用呢？您对我如何写得更好有进一步的改进/建议吗？

Answer 1

像您建议的那样使用连续的 4 个三元运算符并不是一个好主意，因为它变得有点难以阅读。如果您担心的是代码行数，您可以使用以下方法缩短方法:

public static boolean isPrime(int n) {
  if (n == 2) return true;
  if (n < 2 || n % 2 == 0) return false;

  double sqrtN = Math.sqrt(n);
  for (int i = 3; i <= sqrtN; i += 2) {
    if (n % i == 0) return false;
  }
  return true;
}