Obj-Cでのドット演算子の過剰使用の代替方法は何ですか？

Question

は、あなたがこのコード行を考える何を参照してください：

if ([pickerViewController.picker.bvc.currentResolve.name isEqualToString:message]) 
    ... 

あなたは、これは、ドット演算子の過度の使用であると考えますか？

もしそうでなければ、私はそのままにしておくことができます。

もしそうなら、好ましい選択肢は何ですか？

Answer 1

これは、ドット演算子の問題よりも多く、Law of Demeter違反です。これを行うには、「クリーン」な方法は、あなたがそこにドット演算子を使用するたびに、本当に財産をフェッチしている限り、

if ([pickerViewController hasPickedName:message]) 

Answer 2

私はプロパティ表記の過剰使用は考えられません。オブジェクトにプロパティがある場合は、そのオブジェクトにアクセスします。それはプログラマーが何を意味するかを読者に実証している。

ああ、「構造体のように見える」旅団を先制する。あなたのコード内のオブジェクトからstrucを伝えることができない場合は、あなたのコードをリファクタリングしてください。

Answer 3

ような何かを行うことができるように、自分自身をこれを理解するために、オブジェクトにロジックを与えることであろう（つまり、主に価値を返す以外の目的で仕事をすることに関係する方法ではない）、それは問題ありません。実際、Wil Shipleyのブログをチェックすると、実際には、必要に応じて多くの関数呼び出しを1行にまとめることができます（ローカル変数の過度の使用を嫌う）。

Answer 4

私は、この種のデバッグは常に価値があるよりも問題であることが分かります。したがって、私は各プロパティアクセスの中間変数を作成する傾向があります。または、他の人が示唆しているように、これをリファクタリングすることで、（スマートをメソッドに入れて）使用サイトで簡単に見えます。

Answer 5

私はChuckとコメント投稿者に同意します。あなたの方法は他の多くのオブジェクトに依存していますが、pickerViewControllerにhasPickedName:を入れても、何とかは[picker.bvc.currentResolve.name isEqualToString:message]となります。

代わりに、Interface Builderのを使用して、トップレベルのオブジェクトに（多分id<NamePickerDelegate>を入力）bvcにhasPickedName:を入れて、デリゲートとしてbvcを注入できます。本当にデメテルに準拠するには、currentResolveというメソッドをnameMatches:というショートカット[currentResolve.name isEqualToString:message]にするようにしてください。

問題によって引き起こされる複雑さと、各ソリューションで導入される複雑さを注意深く見てください。元のコードが代替コードより簡単でメンテナンスが簡単であると判断した場合は、そのまま使用してください。