println和println的空洞法与返回法

Question

在我编写计算器应用程序的时候，我只是想不出什么是最好的方法：

private void calculate(String command) {
    System.out.print("value1: ");
    int value1 = reader.readInteger();
    System.out.print("value2: ");
    int value2 = reader.readInteger();

    if (command.equals("sum"))
        System.out.println("sum of the values " + sum(value1 + value2));
}

private int sum(int value1, int value2) {
    return value1 + value2;
}

或

private void calculate(String command) {
    System.out.print("value1: ");
    int value1 = reader.readInteger();
    System.out.print("value2: ");
    int value2 = reader.readInteger();

    if (command.equals("sum"))
        sum();
}

private void sum(int value1, int value2) {
    System.out.println("sum of the values " + value1 + value2);
}

第二种方法使calculate()方法更加简洁，但是它通常倾向于使用返回方法还是void(仅用于打印行)？

Answer 1

一般来说，最好返回值-然后您可以测试您的方法并在更复杂的计算中重用它。

一种很好的方法是定义一些计算和返回值的calculate方法，并定义void show方法，该方法将接受作为参数值在屏幕上显示并正确显示。

Answer 2

如果要将其扩展到其他命令，我认为您应该看看command pattern。当您想要扩展此计算方法时，它将减少您的if语句。

带计算器的命令模式

Answer 3

更好的方法是返回值。如果您将方法命名为sum，您希望得到sum的值。

您可以创建一个方法showResult()。你希望得到一条信息。

public int getSum() {
    return this.sum;
}

public void showResult() {
    System.out.println("sum = " + this.sum); // add some description 
}

private void calculate() {
    this.sum = reader.readInteger() + reader.readInteger();
}