Java Practice：パラメータとして渡された同じオブジェクトを返す

Question

次のコードでは、updateWithContexがパラメータとしてとった同じオブジェクトを返すのは本当に悪い習慣ですか？ C++で

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) 
    { 
     // 
    } 
} 

、私はこのようなコードを見てきました：

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); 
     //..... 
    } 
} 

あなたがメソッドを、それをバック戻さずupdateWithContex方法でそれを変更することができるようにあなたは、fooというオブジェクトへの参照を渡すので、それは悪い習慣です。もう一度、Javaを使用して常に参照することを忘れないでください。そして、確かに、他の場所でそれを行う方法はありません - それは常にオブジェクトへの参照になるでしょう。 Javaには、fastTransform（BigObject & myBO）のようなものはありません。

Answer 3

私は何も間違っているとは限りません。それはAPIデザインの問題です。コードを使用すると、投稿しているあなたは

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

代わりの

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

ような何かを行うことができます最初のコードスニペットは、第1よりもfluent interfaceのよりよい例です。

Answer 4

Output Parameters in Java一部のオブジェクトは不変です。このようなメソッドのスレッドの安全性にも疑問があります。これは実行できますが、それは通常、出力javaパラメータに対しても良いことではなく、Googleとはみなされません。お役に立てれば。