Lodash、ループ内で関数を呼び出し、最初に一致する結果を返します。

Question

オブジェクトの配列を反復処理し、メソッドを呼び出したいとします。そのメソッドの結果が何らかの条件を満たす場合。私はすぐにその結果を返したい。私はこれを書いた：

public getFirstMatch(value: string, allValues: string[]): Result { 
    let foundResult = undefined; 
    _.find(this.myArrayofMatchers, matcher => { 
     let result = matcher.isMatch(value, allValues); 
     if(result.isMatch){ 
      foundResult = result; 
      return true; 
     } 
    }) 

    return foundResult || new Result(false); 
} 

それは動作しますが、それはclunkyと不明なようです。 _.find、imhoは実際のmatcherを気にしないので、私がやろうとしていることの明確な指標ではありません。事実foundResultが存在する必要があるのは、私がかなり醜いと感じるものです。そして、それは長くなる必要があるようです。 ここでもっとうまくいくものはありますか？これにはより良いlodash関数がありますか？

はなお、ここであなたが_.foreachよう_.findを使用しているループ

public isMatch(value: string, allValues: string[]): Result { 
    for (let i = 0; i < this.myArrayofMatchers.length; i++){ 
     let result = this.myArrayofMatchers[i].isMatch(value, allValues); 
     if (result.isMatch) { 
      return result; 
     } 
    } 
    return new Result(false); 
} 

Answer 1

のために使用して、私が念頭に置いていたものです。これは悪いです。 Lodashは戻り値を見つけるので、それを活用する必要があります。

あなたの方法は次のようになります。

public getFirstMatch(value: string, allValues: string[]): Result { 
    const foundResult = _.find(
     this.myArrayofMatchers, 
     matcher => matcher.isMatch(value, allValues).isMatch 
    ); 

    return foundResult || new Result(false); 
}