Практика Java: возврат того же объекта, который был передан как параметр

Question

В следующем коде, действительно ли неверна практика для updateWithContex, чтобы вернуть тот же объект, что и в качестве параметра?

class SomeClass{ 
    Foo updateWithContex(Foo foo){ 
     foo.setAppId(i); 
     foo.setXId(index); 
     //..... 
     return foo; 
    } 
} 

class Foo{ 

    public void setAppId(int appId) 
    { 
     // 
    } 
    public void setXId(int appId) 
    { 
     // 
    } 
    public void changeState(X x) 
    { 
     // 
    } 
} 

В C++, я видел такой код:

BigObject& 
    fastTransform(BigObject& myBO) 
    { 
     // When entering fastTransform(), myBO is the same object as the function 
     // argument provided by the user. -> No copy-constructor is executed. 
     // Transform myBO in some way 
     return myBO; // Transformed myBO is returned to the user. 
    } 

Это тоже не так?

Answer 1

Возврат объекта будет предлагать пользователям вашего abi, что переданный объект не будет изменен, и вместо него будет возвращен новый измененный объект. Чтобы было ясно, что это не так, я бы предложил изменить тип возврата на void.

Answer 2

Ваш код должен выглядеть следующим образом:

class SomeClass{ 
    void updateWithContex(Foo foo){ 
     foo.setAppId(i); 
     foo.setXId(index); 
     //..... 
    } 
} 

Это плохая практика, потому что вы передаете ссылку на объект Foo, так что вы можете изменить его в методе updateWithContex без возвращения его обратно метод. Еще раз помните, что вы всегда работаете со ссылкой на Java. И, конечно же, нет способа сделать это в другом месте - это будет всегда ссылка на объект. Java не имеет ничего подобного: fastTransform (BigObject & myBO).

Answer 3

Я не вижу ничего плохого, это вопрос дизайна API. С кодом, который вы выложили вы можете сделать что-то вроде

someClass.updateWithContext(new Foo()).changeState(x); 

вместо

Foo foo = new Foo(); 
someClass.updateWithContext(foo); 
foo.changeState(x); 

Первый фрагмент кода является лучшим примером fluent interface, чем второй.

Answer 4

Output Parameters in Java некоторые объекты являются изменяемыми некоторыми неизменяемыми. Безопасность потоков таких методов также сомнительна. Это можно сделать, но да, это обычно считается нехорошо и google для выходных параметров java. Надеюсь это поможет.