计算圆的周长和面积

 5.5
Circumference: 
34.55751918948772
Area: 
95.03317777109123
 

#1 楼

在Java中是否习惯将函数调用放在println（）内，还是将返回的值放入一个单独的变量中然后显示该变量更好呢？


我要说的主要是基于意见的。对于这个小应用程序，我认为将函数方法调用放在println内是很好的。重要的是日志记录），因为您不希望在删除System.out.println或日志记录语句时更改程序行为。这是一个玩具程序吗？


在nextInt上使用Scanner时，我看不到需要使用输入验证。也许您想避免使用负数，或者向用户抛出异常或向用户显示错误消息，但这也许就可以了。



使计算函数静态化？


是的，绝对可以。我看不出为什么它们不应该是静态的。鉴于它们的有用性，最好也将它们也具有public。


功能要求：您目前不提示任何可能使用户感到困惑。只需一个简单的System.out.println("Enter radius:");就可以了。


总体而言，您的代码看起来非常完美。一个未成年人nitpick是您在这里超出必要的空间：

System.out.println("Circumference: ");
System.out.println("Area: ");

另一个未成年人nitpick是您的Scanner变量可以命名为scanner或input代替。 （即使sc是Scanner的一个非常普通的变量名）＃perfectionist-naming。

#2 楼

创建一个Circle对象可能是更惯用的Java。该代码将更流畅地读取（例如，circle.getCircumference()）。它还为您提供了一个验证半径是否为负值的好地方。

import java.util.Scanner;

public class Circle {
    private final double radius;

    public Circle(double radius) {
         if (radius < 0) {
             throw new IllegalArgumentException();
         }
         this.radius = radius;
    }

    public double getCircumference() {
        return 2 * Math.PI * radius;
    }

    public double getArea() {
        return Math.PI * radius * radius;
    }

    public static void main(String[] args) {
        try (Scanner sc = new Scanner(System.in)) {
            double radius = sc.nextDouble();
            Circle circle = new Circle(radius);

            System.out.println("Circumference: ");
            System.out.println(circle.getCircumference());
            System.out.println("Area: ");
            System.out.println(circle.getArea());
        }
    }
}

还有两个小点：



import java.lang.Math是不必要的，因为java.lang.*一直都可用。
由于Java 7，try-with-resources为您提供了一种自动关闭像Scanner这样的自动关闭对象的方法。代码或我上面的代码都检查是否存在所有要读取的double。如果输入为垃圾，则将抛出java.util.InputMismatchException，该q4312079q无法处理。

#3 楼

我实际上不喜欢将每个标签和结果显示在单独的行上，而是希望将每个值都放在这样的单独行上：





<我发现5.5 Circumference: 34.55751918948772 Area: 95.03317777109123 可以做到这一点，而print()可以换行。 >我还结合了@Simon关于用户输入提示的建议。

 println() 

这样，我得到以下输出（使用相同的半径）：

 System.out.print("Input radius: ");

// ...

System.out.println();
System.out.print("Circumference = ");
System.out.println(circle.getCircumference());
System.out.print("Area = ");
System.out.print(circle.getArea());