5

我已经编程了一段时间了,我有一个关于公共方法使用的问题。我正在开发一个自动售货机程序,我有一个私有方法setUpMachine()来初始化游戏和设置对象。我有另一个私有方法startMachine()可以启动游戏并提示用户输入。然后它将输入传递给另一个私有方法checkInput(),该方法检查输入是否有效......但这并不是我遇到的“问题”,而是我没有正确做某事的奇怪感觉。setUpMachine()对于我的第三种方法,我需要访问第一种方法中的对象checkInput()。问题是我有很多对象(糖果、薯条、苏打水、饼干),并且将它们全部传递给检查周边似乎并不正确。换句话说,这样做:

checkInput(FoodType candy, FoodType chips, FoodType soda, FoodType cookie)

似乎不对。这是否意味着,如果我使用私有方法,每次我想使用它们时都必须传递对象?我读到公开方法是不好的做法。

对此的解释会很好,与其说是告诉我我的编码效率低下的解释,不如说是描述何时以及如何使用私有方法的解释,或者是否有其他方法可以做到这一点。

4

3 回答 3

3

如果您不想传递对象,可以将它们配置为实例变量:

public class VendingMachine {
    private FoodType candy;
    private FoodType chips;
    private FoodType sodas;
    private FoodType cookies;

    private static String errorMessage = "A really bad error occurred.";

    public VendingMachine(){
        this.setupMachine();
    }

    private void setUpMachine(){
        this.candy = new FoodType();
        this.chips = new FoodType();
        this.sodas = new FoodType();
        this.cookies = new FoodType();
    }

    private boolean checkInput(){
        if (this.candy==null || this.chips==null || this.sodas==null || this.cookies==null)
            return false;
        else
            return true;
    }

    public void doSomething() throws Exception() {
        if (!this.checkInput()) throw new Exception(VendingMachine.errorMessage);
        // do things
    }
}

然后可以将此类称为

VendingMachine vendingMachine = new VendingMachine();
try {
    //vendingMachine.checkInput() is not available because it is private
    vendingMachine.doSomething(); // public method is available
} catch (Exception e){
    // validation failed and threw an Exception
}
于 2012-10-21T00:58:49.547 回答
2

通常,当您想向外界公开某些内容时,应该使用公共方法。通常,它是一种更高级别的方法,其中私有方法用于提供详细信息。私有方法可以很容易地更改,而不会影响公共方法不能影响的其他类。

考虑这个例子

public void run() {

  setUpGame();
  validateInput();
  doSomethingInteresting();
  endGame();
  notifyUserGameIsComplete();

}

在此示例中,run是公共的,但其他方法都不是。这是因为公共方法的调用者只知道他们想要运行游戏,而不关心它究竟是如何工作的细节。

拥有许多私有方法是可以肯定的。方法应该是单一的,每个只做一件事。这使您可以更轻松地阅读、调试和维护代码。

如果多个方法需要共享数据,可以将它们声明为类的成员变量。这意味着它们是班级的状态。例如:

public class Person {
   private String name;

   public String getName() { return name; }
   public boolean nameMatches(String pattern) { // do some regex testing against the name }
}

在此示例中,方法共享对 的访问权限name,并且name是人员的一部分。

如果你真的需要传递一堆参数,你可以创建一个对象来保存所有参数。因此,您可以创建一个数据传输对象,而不是传递 param1、param2、param3 等,例如

class MyStuff {
   private final String param1;
   private final String param2;
   private final String param3;
}

现在您可以MyStuff方便地传递而不是列出单个参数。

希望这有助于增加一些清晰度。

于 2012-10-21T00:52:57.590 回答
2

外部世界看不到私有方法。您只能从同一类的方法中调用私有方法。

私有方法可用于在逻辑上将一个大方法分解为多个较小的部分(步骤)。

此外,如果您有一些代码似乎在您的类中的多个方法中重复,那么您可以使用私有方法。

如果对象也是类中的字段,则不需要将对象定义为私有方法的参数,但如果不是,则必须这样做。

于 2012-10-21T00:55:41.960 回答