将类实例变量作为参数传递

Question

这是我一直在考虑的一个设计问题，但没有找到令人信服的信息。

假设我的类中有一些实例变量，现在想象我想使用该值为我的类编写一些私有功能。写这样的东西没问题：

public class Example{

    private String attribute1;

    public void setAttribute1(String att){
        this.attribute1 = att;
    }

    private void processAttribute(){
        //do something with attribute1
    }

    public void executeExample(){
        processAttribute();
    }

}

processAttribute() 在内部使用 attribute1 值。但是，许多文档说我们应该尝试限制全局变量的使用。编写这样的东西会是一种更可重用且设计良好的方式吗？

public class Example{

    private String attribute1;

    public void setAttribute1(String att){
        this.attribute1 = att;
    }

    private void processAttribute(String att){
        //do something with attribute1
    }

    public void executeExample(){
        processAttribute(this.attribute1);
    }

}

集思广益。

Answer 1

许多反对全局状态的论点在这里也适用：

• 如果该属性在 processAttribute 方法之外的其他地方使用，则很难推断程序的正确性
• 使用全局状态的代码更难并行化：如果在处理属性时修改了属性会发生什么？
• 更多：http://c2.com/cgi/wiki?GlobalVariablesAreBad

另一方面，它是一个私有方法，您可以随意实现它，只要您履行该类的合同即可。

Answer 2

但是，许多文档说我们应该尝试限制使用 global 变量。

我认为您误解了这个概念。通常全局变量是那些声明为public static 的变量，以便可以从应用程序的任何其他部分直接访问它。

因此，在您的两个示例中，变量属性 1 都不是全局变量。它只是类的一个成员变量。

因此，我认为这两个不同的代码之间没有太大区别。

如果设计是固定的，那么我认为最好使用第一个使其更具可读性。如果将来有其他机会将其他变量作为参数而不是成员变量发送，那么您可以使用第二个实现。 （虽然我认为这完全取决于编码员的个人选择）

Answer 3

首先，attribute1 不是全局属性，它只是一个类变量。使用此操作，类变量将在所有类方法中可用，因此您无需将它们作为方法参数传递。

由于这里不需要传递方法参数，所以实现它似乎不合逻辑。顺便说一句，这是我个人的看法，其他人可能有不同的想法。