jQueryで、関数を作るときに$(function()外に書くか、内側に書くか

Question

例えば、以下のようなjsがあったとして、
どちらのパターンの方が良いのでしょうか？
※関数は、同function内でのみ使用予定

パターン①

$(function() {
　var $hoge = $('.hoge'),
      hogeClick = (fucntion() {
　　　　　$hoge.click();
　　　 }),
　　　hogeFocus = (fucntion() {
　　　　　$hoge.fucus();
　　　 });
　　hogeClick();
　　hogeFocus();
});

パターン②

var　hogeClick = (fucntion() {
　　　　$('.hoge).click();
　　　}),
　　　hogeFocus = (fucntion() {
　　　　$('.hoge).fucus();
　　　});
$(function() {
　　hogeClick();
　　hogeFocus();
});

よろしくお願いいたします。

Comments

[miyabi_pudding]

要件や実装したいことによって、どちらかがいいかまるで変わってきます。
なので、もう少し細かく要件定義すべきです。
パフォーマンスを重視したいのか、多人数で開発するときはどちらがいいかとか、
ファイルを分ける場合、とか（今回は同function内とのことなので、ファイル分けは発生しなさそうですが）

Answer 1

パターン①だと思います。

パターン②の場合、他のスクリプトでhogeClickというグローバル変数を使っていた場合、バグになるからです。「関数は、同function内でのみ使用予定」とのことですから、グローバルに置くメリットはないと思います。

Comments

[decteckaaaaaaaa]

ご意見、ご回答ありがとうございます！

Answer 2

スコープ内においたほうが競合が少ないですが
jQueryの場合、ユーザー関数を作って再利用する機会は非常に少ないと思います
所定の要素にonで振っておいてtriggerすればいいだけなので

Comments

[decteckaaaaaaaa]

すみません、初学者で理解が及ばず。。以下どういうことなのでしょうか？
'''
jQueryの場合、ユーザー関数を作って再利用する機会は非常に少ないと思います
所定の要素にonで振っておいてtriggerすればいいだけなので
'''

[yambejp]

jQueryはセレクタのイベントにコールバック関数を割り当てるからです
たとえばtype=buttonやid=hogeのクリック時になにか実行する
処理を割り当てたとき、別のHTML要素からそれを実行するには
id=hogeをクリックしてやればいいのです

[decteckaaaaaaaa]

ご回答ありがとうございます。

つまり、変数に格納して関数化せずに
例えば、$('.btn').on('click', function() {});
などとすれば良いではないのかということでしょうか？