値を返し、それを悪い習慣とみなされる変数に格納していませんか？次の関数を考える

Question

：

function lamePathValidator(path) { 
    if (typeof path !== 'string') { 
     throw new TypeError(`Invalid path "${path}": should be a string.`); 
    } 

    return path; 
    } 

は時折、それが変数に戻り値を格納することなく使われている場合、それは悪い習慣と考えられますか？

... 
lamePathValidator('./data'); 
... 

編集：私は戻り値を必要とする唯一のユースケースなど、コンストラクタでプロパティの割り当てのためのものです：

constructor(basepath) { 
    this._basepath = this._validatePath(basepath); 
} 

私はもちろん書き込みの可能性：

constructor(basepath) { 
    this._validatePath(basepath) 
    this._basepath = basepath; 
} 

前者のブロックはより簡潔です。

編集＃2： それらのどれが実際にどのような方法で変更されていないされているので、私はまた、_validateAll(params)関数にコンストラクタのパラメータを渡すことができます。結局のところ、「妥当性検査」ではなく、「サニタイズ」しません。 :)

Answer 1

いつも役に立つとは限りませんが、時にはデータを返す関数がたくさんあります。あなたがそれを必要としないときに出力を保存しないそれらのケースでは、それは理にかなっています。

ただし、戻り値が有用でない場合は、最初に返さないでください。あなたのサンプル関数では、パスは変更されていないので、それを返す理由はないので、呼び出しコードにはすでにそれがあります。

Answer 2

関数は、必要に応じて値を返すことができます。 値を返すことは必須ではありません。あなたの場合、値を返す必要はありません。

js関数のドキュメントを読むhere