2

ビューのサブ要素を操作していて、それらを適切に処理しているかどうかわからない状況にあります。この特定の例は、特定のイベントで消える左ボタンと右ボタンを持つカルーセルがあります (たとえば、一方向のカルーセルの最後に到達した場合)。これらのオブジェクトを操作するために、私はこのようなことをしてきましたが、それは正しくありません:

$(this.el).find('.right-paddle').hide();

または、場合によっては、インスタンス化中に次のように呼び出します。

this.rightPaddle = $(this.el).find('.right-paddle');

後でこのようなことができるように：

this.rightPaddle.hide();

これはフォームが悪いのでしょうか？

4

1 に答える 1

2

これらのオブジェクトを操作するために、私はこのようなことをしてきましたが、それは正しくありません:

なぜだめですか？それの何が問題なのですか？書きすぎじゃない？ビューは「親」の DOM 要素にバインドされており、多くの場合、それらを操作するために見つけなければならないいくつかの子または要素が含まれています。それは完全に有効です

または、場合によっては、インスタンス化中に次のように呼び出します。DOM トラバーサルはコストがかかるため、頻繁に参照する場合は、インスタンスをキャッシュすることをお勧めします。あなたの場合、それは完全に有効であるようです。

これはフォームが悪いのでしょうか？

いいえ、複数の方法があります。アプリケーションの保守性と可読性を向上させることを行い、パフォーマンスが非常に重要でない限り、時期尚早の最適化や読みにくいコードに努力する必要はありません (それでも読みやすいはずです :)

これはあなたの懸念を和らげるのに役立ちますか？

于 2012-08-13T06:10:33.333 に答える