我正在考虑我的应用程序的解决方案。这是一种情况:我有一个类,该类的方法将ObjectA作为输入参数并调用几个小方法,这些方法中的每个方法都需要ObjectA的某些部分(它们不重叠,即method1()需求ObjectA.field1和ObjectA.field2,method2()需求ObjectA.field3等等)。 …)现在的问题是:鉴于一般的良好代码实践和性能,将ObjectA传递给这些方法中的每一个方法更好,以便它们可以自己提取所需的值,还是仅传递它们的值更好?我的意思是:
method1()
ObjectA.field1
ObjectA.field2
method2()
ObjectA.field3
method1(ObjectA); method2(ObjectA);
要么
method1(Object1.getField1(), ObjectA.getField2()); method2(ObjectA.getField3());
感谢您的任何答案!
请记住,使用您的代码,实际上并没有通过ObjectA。也就是说,您要将引用类型传递给ObjectA,因此在性能注意事项上,传递String对象引用和ObjectA对象引用之间的区别可以忽略不计。
ObjectA
String
我写的方式
如果该方法与该类相关,则将传递整个对象。我这样做的原因是尽可能多地分散课堂知识。我的意思是以下内容。
public void doSomethingRelatedToTheClass(String param) { // Do something with param. }
我在这里的第一个批评是,此方法假定输入是正确的字段。我的第二个是,现在,调用此代码的类需要更多地了解此方法,因为它必须像这样调用它:
doSomethingRelatedToTheClass(myObject.getValue());
这意味着,如果您发现该方法的另一个成员ObjectA在该方法中效果更好,或者您想要访问的其他成员ObjectA,并且doSomething()为了反映此更改而进行更改,则还需要将方法调用更改为:
doSomething()
doSomethingRelatedToTheClass(myObject.getOtherValue(), myObject.getValue());
因此,通过传入整个对象,您可以将该细节抽象化,并且该方法可以处理它。即:
doSomethingRelatedToTheClass(myObject); // Doesn't need to know what you do with it. public void doSomethingRelatedToTheClass(ObjectA object) { String val = object.getValue(); String otherVal = object.getOtherValue(); }
当更改为一个类别而导致更改其他类别时,这是一种称为Shotgun Surgery的反模式。
编辑
我有机会在这里查看我的答案,并且我对原始答案进行了稍微的修改,因为我认为这并不是在 所有 情况下的最佳解决方案。如上所述,如果方法与某个类具体相关,则该类的实例(或更优选地,其超类或已实现的接口)应作为参数。
当功能可以通用时,情况并非如此。通用函数的示例可能是:
public String[] findNouns(String sentence);
在这种情况下,查找句子中的名词可能适用于许多用例,而不仅仅是您定义的用例。因此,传递值是唯一明智的方法,因为否则,您会将没有直接关系的两个逻辑耦合在一起。查找名词和您定义的任意对象。
综上所述
如果方法是与对象相关的逻辑,请传入对象
如果该方法与对象无关,并且该对象只是将其用作实用程序函数,则传入值并以通用方式命名该函数。