私はこの機能を持っている:ほぼ同じことをしている複数のif文でこの関数を処理するより良い方法はありますか?
private addAdditionalResults(resultsToAdd: any) {
if(!isNaN(resultsToAdd.xx) && this.x.length > 0)
this.x.unshift(resultsToAdd.xx);
if(!isNaN(resultsToAdd.yy) && this.y.length > 0)
this.y.unshift(resultsToAdd.yy);
if(!isNaN(resultsToAdd.zz) && this.z.length > 0)
this.z.unshift(resultsToAdd.zz);
}
X、Y、Zは、数字の配列です。 resultsToAddは変数xx、yy、zzを持つオブジェクトで、NaNとは異なるものにすることができます。このコードをもっと美しくするにはどうしたらいいですか?これらのif文はほとんど同じことをしています。あなたが事前にキーを知っているし、彼らはパターンに従っている場合
もしあなたの配列 '自分の価値観とは無関係であるz' X '、' y'と '' resultsToAdd.'/'xx' /' yy'/'zz'は、ありません、反復処理する方法はありませんそれらの上に。あなたは 'if'文を' unshift'にする関数を作成することもできますが、それを3回呼び出さなければなりません – Ksyqo
resultsToAddは特定のインターフェースを持っていますか、それとも本当ですか? – toskv
これは私には良く見えますが、この種のプルリクエストに問題はありません。 –