コードの 39~40 行目の

 const columns = createColumns();
 renderBingo(columns);

の部分ですが、変数 columns を宣言せずに

renderBingo(createColumns());

のように引数として関数 createColumns() を渡すのは、よくない書き方なのでしょうか？

「変数のスコープを小さくするために、変数宣言した columns を引数としてわざわざ渡している」との説明を他の質問で読んだのですが、変数宣言をしないで（変数を作らないで）おけば、スコープのことを気にしなくてもよいのかな？と思ったのですが…。