これを簡潔に書く方法はありますか？

質問

今、なるべくコードを簡潔に書けるように勉強中です。
下記のコードなのですが、
これを簡潔に書く方法はありますか？
よろしくお願い致します。

$(window).load(function() {
adjStyleW($(this).width());
adjStyleH($(this).height());
});

回答No.3

b0a0a · ベストアンサー

adjStyleW($(this).width());
adjStyleH($(this).height());
を
adjStyle($(this).width(),$(this).height());
にすれば簡にはなるんじゃないでしょうか
でも潔ではないですね

ライブラリを使っているせいもありますがわかりにくいです
最低自分が１年後とかに見てもパットわかるコードだといいですね
例えばですけど

$(window).load(function() {
//formの大きさを調整
var winWidth = $(this).width();
var winHeight = $(this).height();
adjFormSize(winWidth,winHeight);
});

コード量が増えてもこんな感じをより簡潔と言うのではないでしょうか
もしくはスペースを沢山入れてわかりやすくするとか。
エディタと構成によってはadjustFormSizeByWindowSize()くらいやっても平気かもしれません

まあでも
$(window).load(function(){t=$(this),adjStyleW($(t.width()),adjStyleH(t.height())})
が一番簡潔だと思う人もいるかもしれませんからね
個人的に書く分には好みでいいと思います

お礼

最高です。
とてもとても参考になります。
少しでもレベルが上がるよう日々勉強致します。

ありがとうございました。

回答No.2

CyberCypher

いや関数だろうなぁってのは理解していましたが
どういう処理をさせたいのか書かれていないと
縦横サイズ渡してどうすんの？って回答が付けられません

単純に縮小したいのなら
関数一つにして引数を二つ渡してやれば良いんじゃないですかね？

回答No.1

CyberCypher

adjStyleW
これはなに？

補足

説明不足でスイマセン。

function adjStyleW(width) {
/*内容*/
}

function adjStyleH(height) {
/*内容*/
}

$(window).load(function() {
adjStyleW($(this).width());
adjStyleH($(this).height());
});

という風にしています。