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语句。

command-pattern with calculator

Answer 3

更好的方法是返回值。如果您将方法命名为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();
}

println与println的Void方法，带返回方法

3 个答案: