可読性

**デフォルトの名無しさん** · 2017/10/09(月) 12:21:58.39

ローカル変数には抽象的で短い名前をつけろ

× var foo_bar_baz_collection = FooBarBazCollection
○ var collection = FooBarBazCollection

ローカル変数に長い名前をつけるやつは
関数自体が長い可能性が高い。
長い関数を作るから、長いローカル変数名にしないと
何の変数か覚えられない

行き当たりばったりで名前を略するな。
略して良いのは、プロジェクトや業界で決まってる単語だけ

ただし変数の有効範囲が一行程度であれば略しても良い

○ values.map(v => v * v) //この行を見るだけでvがvaluesの要素であることがわかる
×
var ret = []
for(v in values) {
ret.push(v) // この行を見てもvがvaluesの要素であることがわからない
}

**デフォルトの名無しさん** · 2017/10/09(月) 12:35:50.57

うぇあ

**デフォルトの名無しさん** · 2017/10/09(月) 13:06:40.88

例に挙げた「collection」が特に抽象的でもない件

**デフォルトの名無しさん** · 2017/10/09(月) 16:40:26.04

valuesがすでに分かりにくいと思うんだよね
商品の個数なのか請求の金額なのかユーザの数なのかはっきりしようよ

retもresultなのかrestなのかretailなのかrestrictなのかはっきりしようよ

抽象的な値を扱うユーティリティ関数なら変数名なんて適当でいいけど
業務ロジック関数の一部を切り出してるならNGと言っていいでしょう