スコープがあるがアクセスしたい場合わかりやすい方法はないでしょうか？

下記のようにスコープの影響でlet mathRandom;の部分をわざと無名関数の外で宣言して、中で代入だけを行って、
他のスコープからアクセスできるようにすることが非常に多いのですが、
他人が見ると非常にわかりにくいです。
このような場合どのように対処しますか？
スコープがあるがアクセスしたい場合わかりやすい方法はないでしょうか？

        let Position = () => {
            createElement.style.top = (mathRandom * 100) + '%';
        }


        let mathRandom;
        let setClickItem1 = setInterval(() => {
            mathRandom = Math.random();
            Position();
        }, speed);

行動規範の内容に同意します

回答1件

ベストアンサー

コードが意味のある形で抜き出されていないのでそもそもわかりにくいですが、引数を使えばよいのでは？

JavaScript
1let Position = mathRandom => {
2    createElement.style.top = (mathRandom * 100) + '%';
3}
4let setClickItem1 = setInterval( () => {
5    Position( Math.random() );
6}, speed );

投稿2017/05/04 06:53

kei344

総合スコア69407

spablock

2017/05/04 07:01

let Position = () => { var mathRandom = Math.random(); createElement.style.top = (mathRandom * 100) + '%'; } じょうきのようになるという事でしょうか？引数を使えば参照したい先に仮引数を書けばその中の式からアクセスできるようになるので、スコープの問題がなくなるという事でしょうか？

spablock

2017/05/04 07:02

let Position = mathRandom => { createElement.style.top = (mathRandom * 100) + '%'; } はES6のアローファンクションで仮引数をmathRandomにしたという事でよろしいでしょうか？ function(mathRandom)　｛｝という事ですね。

spablock

2017/05/04 07:10

ありがとうございました。確かにできました。引数は実質下記と同じと聞いていたので、なんのために存在するのかがわからなかったのですが、別スコープから参照したい時にローカル変数を作るのではなく、引数を利用すると別スコープからも参照できるので、このような場合にローカル変数ではなく引数にする意味があるのですね。 let Position = () => { var mathRandom = Math.random(); createElement.style.top = (mathRandom * 100) + '%'; }