Ext.pass がどうしても使いづらい…

Ext.pass または、Ext.Function.pass のお話(2つは同じ動作します)

さて、Extjs4 以降はMVCになったので、いろいろと楽です(設計とか)
今回探し回っていたのが下記URLの「関数委譲関数を生成する」の部分
http://code.xenophy.com/wp-trackback.php?p=1882

要するに
”コントローラーでイベントを受信した時に、引数を追加して共通処理を呼びたい。”
って事なんですよ。(要約してない…)
で、このURLを探し当てたわけですよ。

見つけて理解した時「さすがです。痒い所に手が届く！」とか思って読んでました。



しかし、詳しく理解するためにドキュメントを読むと…あれ？
追加の引数が前に来ちゃってますよ…？
そう、Extjsのドキュメントを見ても、src/lang/Function.jsソースの中を見ても、実装して確認しても、
追加する引数が"前(第1引数)"に使われているではありませんかっ！

なぜ？

追加引数が前だと、引数を指定したくない場合(デフォルト動作)の処理が面倒です。

例えば：
　　１．イベント引数の場所が決まって居ないので「第1引数の型がAの場合、型がBの場合、型がC…」と処理をわけなければいけない。

　　２．引数が不要なのに、空白引数を指定しなければいけない。
Ext.Function.pass(this.onButtonClick, [''], this)

　　３．引数の数によって入れ替えをする。
function( arg1, button, e, eOpts ) {

if(arguments.length == 3 ) {

    eOpts = e;

    e = button;

    button = arg1;

    arg1 = '';

}

// なんだかの処理

}


などなど、面倒がたくさんです。
ちなみに私の意見
１．本末転倒過ぎる…メソッド分けるわ…
２．あほらしすぎる＋チーム全体＆保守チームに連携するのダル…
３．ありえ無い…でも僕の頭ではこれが限界…orz
となり、３．を採用しました。
(ちなみに引数が2個、3個と増えれば、その分ずれるので叱らず…)

追加引数が後に付けばこんなこと一切する必要がありません。
というわけで、
なぜ追加引数が前なのだーーーー理由を教えてくれーーーー開発者ーーーー

とか叫びたくなったので書いてみました(笑)



さて、ここまでいろいろ書きましたが、参照URLでこの事について何も記載されていません。
と言う事は「追加引数が前に有ったの方が良い理由があるはずだ！」と思い考えてみても、私の頭では到底思い付きませんでした。
謎は深まるばかり…私には精神ポイントが足りなく、"ひらめき"が使えないようです。

Extjsのソースを見る限り、追加引数を後ろにするのは簡単だけど…
・影響範囲が広すぎるから、後ろにはされないだろうなー…
・前に追加・後ろに追加フラグ持たせるのもなんだかなー…
と言う事で、ずっとこのままでしょう。個人的には残念です。



修正案：(未テスト)

ファイル：src/lang/Function.js

メソッド：pass: function(fn, args, scope) {

修正前：

return function() {

            var fnArgs = [].concat(args);

            fnArgs.push.apply(fnArgs, arguments);

            return fn.apply(scope || this, fnArgs);

        };

修正後：

return function() { // 配列に入れる順番を変えれば良いのでは？

            var fnArgs = [];

            fnArgs.push.apply(fnArgs, arguments);

            fnArgs.push.apply(fnArgs, args);

            return fn.apply(scope || this, fnArgs);

        };


遅くなっちゃうのかな？そのほか理由があるのかな…？ん～悩ましい…



************** コメントへの返信 **************
> Unknown (匿名さん)
> 2013-08-08 16:15:32
いろいろ考えながら書いたので、別記事にしました。
こっち見てね。
http://blog.goo.ne.jp/pianyi/e/71bc79cfee4ee1135583a063063c1cff

コメント

コメントを投稿

このブログの人気の投稿

ヨドバシカメラの店舗購入履歴を見るには…

イメージ
・2025年04月08日 現在の情報です。今後使えなくなる可能性があります。 ・本記事に関する内容は分かる方のみ行ってください。ヨドバシさんの店舗店員、ヨドバシ・ドット・コムさんに質問等しないでください。 ・ヨドバシ関連様とは全く関係ありません。ご迷惑をかけない様よろしくお願いします。 ヨドバシ・ドット・コム で ヨドバシカメラ店舗で購入した履歴を見る方法です。 2種類の検索方法を記載しています。おすすめは上部の方(再注文：2025年04月08日掲載)です。 共通の注意点： １．ヨドバシ・ドット・コムのユーザが作成されている。（利用している？） ２．ヨドバシゴールドポイントカードは同じものを共通で登録・購入時提出してないと駄目だと思う。 ３．ヨドバシ・ドット・コムと連携する前のデータは連携されないと思う。 ４．Webと店舗のどちらで買ったか区別付きません。（思い出してください） ５．表示される値段は現在の値段です。（購入時の値段はわかりません。） 　　ヨドバシさんのSwitch2抽選の話が出たからかアクセスが急増していますが、【購入時の金額は分かりません】 (なんでこんなに画像が見にくいのだろうか…) 2025年04月08日掲載の検索方法 【カテゴリ検索】 「ま」様よりコメントを頂きましたので反映します。 こちらの方法は下記3パターンの場合、とても優良ですのでお試しください。 １．商品の「カテゴリ」が分かっている場合 ２．過去５年間に購入した商品を検索する場合（2025年の場合、2025年～2021年の5年間) ３．少しでも早く全データを確認したい場合？（ページングがあるのでこちらが速いと思うが検証してないです） 全ては「ま」様のコメントに記載されていますが、一応記載しておきます。 やり方： １．ヨドバシ・ドット・コムにログイン ２．会員ページに移動（画面上部の登録者名から移動） ３．画面上部にある「ご注文履歴」内にある「ご注文履歴から再注文する」のリンクを選択する（ボタンじゃないです） ４．パスワード入力 ５．「ご注文履歴から再注文」画面が表示されますのでそこから検索してください。 検索方法について： ・「ご注文カテゴリをお選びください」は【絞り込み】のため、検索した結果のデータしか出てきません。 　そのため下記画像の「①」で「全てのご注文」で検索した後（検索結果...
続きを読む

C# の WPF の DataGrid で 行を交互に背景色を変える＋選択色を変える＋カラムが無い所も変える…

イメージ
盛りだくさんのタイトルになりましたがまぁそういう事… 注意：個々のコードは最終確認してません。（動いている物から必要ない箇所を削除してます） DataGrid を使って表を作ることはよくあると思います。 で、見やすくするために「1行ごとに背景色を変える」っていうのは「AlternationCount」と「AlternatingRowBackground」を使えばすぐ出来ます。 こんな感じですね <DataGrid AlternationCount="2" AlternatingRowBackground="SkyBlue" SelectionMode="Single" SelectionUnit="FullRow"> で、そこにユーザが選択した時の色を変えるのもその辺に良く落ちてます。 こんな感じですね。 <DataGrid AlternationCount="2" AlternatingRowBackground="SkyBlue" SelectionMode="Single" SelectionUnit="FullRow" Background="Transparent"> <DataGrid.RowStyle> <Style TargetType="DataGridRow"> <Setter Property="Background" Value="White"/> <Setter Property="Foreground" Value="Black"/> <Style.Triggers> ...
続きを読む

Visual Studio の ホットリロードが動かない場合に確認するところ

とりあえずはここを読む ttps://learn.microsoft.com/ja-jp/troubleshoot/developer/visualstudio/tools-utilities/xaml-hot-reload-troubleshooting 英語版： ttps://learn.microsoft.com/en-us/troubleshoot/developer/visualstudio/tools-utilities/xaml-hot-reload-troubleshooting 今回困ったのは、デバッグ中の画面上部にホットリロード用のマーク（緑のレ点・赤の×）マークが有効（緑のレ点）になっているにもかかわらず、ホットリロードが効かない状態でした。 調べてもそれらしきものが見つけられずに早半年… やっと見つける事が出来ました。 きっかけはここ： ttps://zv-louis.hatenablog.com/entry/2023/03/21/110547 「Debug構成以外でホットリロードを有効にしたい」そんなことがあるかどうかは別にして… <XamlDebuggingInformation>true</XamlDebuggingInformation> の記述が… これってデフォルトで有効なんじゃないの…？ 【ちなみに最初のマイクロソフトのHPにも記載があります。がデフォルトで有効でしょ！と思い追記しなかった…】 と思いながら、Debug構成内に追加した所動くじゃありませんか…orz すげーやられた感が漂います。 修正後内容はこれ <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'"> <DefineConstants>DEBUG;TRACE;</DefineConstants> <XamlDebuggingInformation>true</XamlDebuggingInformation> </PropertyGroup> <Propert...
続きを読む