インプレース操作でオブジェクトを返すのは悪い考えですか？

Question

ここではPythonについて話していますが、これはおそらくほとんどの言語に当てはまると思います。私が可変オブジェクトを持っている場合、インプレース操作でもオブジェクトを返すのは悪い考えですか？ほとんどの例のように、オブジェクトを修正してNoneを返すようです。たとえば、list.sortです。

Answer 1

はい、悪い考えです。その理由は、インプレース操作と非インプレース操作が明らかに同じ出力を持つ場合、プログラマは、インプレース操作と非インプレース操作（List.sort()対sorted()）を頻繁に混在させ、エラーを検出する。

自分自身を返すインプレース操作では、メソッドチェーニングを実行できますが、偶然にチェーンの途中に副作用のある関数を埋め込む可能性があるため、これは悪い方法です。

このようなエラーを防ぐには、メソッドチェーンには副作用を持つメソッドが1つしかなく、その関数はチェーンの最後にある必要があります。チェーン内のそれ以前の関数は、副作用なしに入力を変換する必要があります（例えば、ツリーのナビゲート、文字列のスライスなど）。インプレース操作が自分自身に戻ると、プログラマーはコピーを返す代替関数の代わりに誤って使用することになり、副作用（List.sort()対sorted()）がなくなり、エラーが発生しやすくなりますデバッグ。

これは、Pythonの標準ライブラリ関数が常にコピーを返すかNoneを返して、オブジェクトをインプレースで修正しますが、インプレースでオブジェクトを変更したり、自分自身を戻したりすることがないためです。 Djangoのような他のPythonライブラリもこの方法に従います（Djangoについてはthis very similar questionを参照してください）。

Answer 2

ユースケースによって異なると思われます。私は、結果を使用しない以外に、インプレース操作からオブジェクトを戻すのが痛いのはなぜか分かりませんが、あなたが純粋な機能主義について非常に慎重でない場合は、実際には問題にはなりません。 jQueryのような呼び出し連鎖パターンが好きなので、関数を使用したときに関数がそれを返したときにはそれを感謝します。

Answer 3

変更されたオブジェクトを修正したメソッドから戻すと、いくつかの利点がありますが、Pythonではお勧めできません。変更操作後にselfを返すと、オブジェクト上でmethod chainingを実行することができます。これは、同じオブジェクトに対して複数のメソッドを実行する便利な方法です。オブジェクト指向プログラミングでは非常に一般的なイディオムです。そして、メソッド連鎖は、fluent interfacesの直接的な実装を可能にします。また、いくつかの関数プログラミングイディオムをより簡単に表現することができます。

例を挙げておきます：Pythonでは、Mokaライブラリはメソッドチェインを使用しています。 Javaでは、StringBuilderクラスは同じオブジェクト上に複数のappend()呼び出しを許可します。 JavaScriptでは、JQueryはメソッド連鎖を広範囲に使用します。 Smalltalkは、このアイデアを次のレベルに引き継いでいます。デフォルトでは、すべてメソッドは、特に指定がない限りselfを返します。したがって、メソッド連鎖を推奨します。これはデフォルトでNoneを返すPythonと対照的です。

PythonはCommand/Query Separation Principleに準拠しているため、このイディオムの使用は一般的ではありません。「すべてのメソッドは、アクションを実行するコマンドまたは呼び出し元にデータを返すクエリのいずれかでなければなりません。どちらも"。

最終的にselfを返すことが良いか悪いかに関わらず、すべてのことを考慮して、個人的な味と混合されたプログラミング文化と慣習の問題です。上で述べたように、いくつかのプログラミング言語はこれを（Smalltalkのように）推奨しますが、他のプログラミング言語はPythonのようにそれを妨げます。それぞれの視点には長所と短所があり、熱い議論につながります。もしもあなたが本書のPythonistであれば、selfを返すのをやめてください。このルールを破ることは時には便利なことがあることに注意してください。