変数代入によるコードの可読性について

Question

かなり個人の好みが出てしまうことかと思いますが、どう感じる方が多いのかを知りたくて質問させていただきます。
使い回さない値を一旦変数に渡して、その後別メソッドに渡すという行為に関してです。

const exampleMethodA = (arg) => {
  // 何かしら処理

  return result
}

const exampleMethodB = (arg1, arg2) => {
  // 何かしら処理
}

という関数があり、exampleMethodAで処理した後にexampleMethodBに渡したい場合、

// A.
const varAA = exampleMethodA(varA)
const varBB = exampleMethodA(varB)
const result = exampleMethodB(varAA, varBB)
// 以後varAA, varBBは利用しない

// B.
const result = exampleMethodB(
  exampleMethodA(varA),
  exampleMethodA(varB),
)

みなさんならどちらで記述しますか？
Aは無駄な変数代入ですが、見栄えはいいような気がします。
好きにしろというレベルの話ですが、好みを聞かせていただければ幸いです。

Comments

[Zuishin]

アンケートですか？

[Lhankor_Mhy]

一応念のため。

https://teratail.com/help/avoid-asking#:~:text=%E6%BC%A0%E7%84%B6%E3%81%A8%E3%81%97%E3%81%9F%E8%88%88%E5%91%B3%E3%81%8B%E3%82%89%E7%A5%A8%E3%82%92%E5%8B%9F%E3%82%8B%E3%82%88%E3%81%86%E3%81%AA%E8%B3%AA%E5%95%8F

[p19ljk]

失礼いたしました。削除します。

Answer 1

変数に代入するというのは、「値に名前を付ける」ということなので、
その値に名前を付けることにより、プログラムが読みやすくなるのであれば、代入する（名前を付ける）し、
そうでもない場合は、代入しません（無名のまま）。

あとは、１つの式があまりに長すぎる場合でしょうか。

Answer 2

推奨されない質問でしたので自己解決とさせていただきます。
(削除リクエストはできませんでした。)

Answer 3

まさに、

好きにしろというレベルの話です

が、デバッグなどをするときに、関数の戻り値がどうなるのかを調べる場合に、変数を介してたほうがやりやすいってのはありますな。

まあ、見てわかりやすい、ってのが絶対正義になりますんで、その時々で変わっていくんもんですわな。

Comments

[p19ljk]

回答ありがとうございます！
確かにみやすさっていうのは一概に言えませんねー。