スウィフト - ネストされたfor each closuresを避ける？

Question

私はすべてのUITouchで実行したいクロージャの配列を持っているとしましょう。ネストされたforeach文があります私のことを

touches.filter { touch in 
    return touch.phase == .Ended && touch.tapCount == 1 
}.forEach { touch in 
    actionsOnTap.forEach { action in 
     action(touch) 
    } 
} 

それのバグ、と私はそのような場合のために正確に適用することができ、いくつかのクリーンな方法はありだと思うが、私はそれを考えることはできません。 は、ここに私が使用するコードです。誰も私にヒントを与えることができますか？

Answer 1

ロジックからfilterを削除し、効率と簡潔さのために、代わりに最初のループの内側にguardを使用する必要があります。私はまた、forEachの代わりに、従来のforループを使用することの提案 - 少なくとも最初のループについては@Rob'sと@matt'sに同意します。

（多分クリーナー）の代替が直接where句を使用して、だけでなく、おそらく（あなたがより読みやすい見つける方）単一の行にあなたのforEach折りたたみてforループにタッチ条件ロジックを統合しているが。

私はこのようにそれを記述します

for touch in touches where touch.phase == .Ended && touch.tapCount == 1 { 
    actionsOnTap.forEach{$0(touch)} 
} 

Answer 2

2種類の異種アレイタイプがあります。 filterが行う余分な反復を避ける別の解決方法は、あなたがチェックしたいと思うタッチを自分自身でフィルタリングすることです。

個人的に

touches.forEach{ 
    guard $0.phase == .Ended && $0.tapCount == 1 else { return } 

    actions.forEach{ action in 
    action($0) 
    } 
} 

Answer 3

、私はの入れ子のような。私は次のように書いています：

for touch in touches { 
    if touch.phase == .Ended { 
     if touch.tapCount == 1 { 
      actionsOnTap.forEach {$0(touch)} 
     } 
    } 
} 

私には、それはきれいで（特に）明確です。

Answer 4

forEachがfor-inの普遍的な（または適切に共通する）代替品でない理由の良い例です。このコードは、単に伝統的なforループを使用して短い（140文字対186文字）と明確になる：

for touch in touches where touch.phase == .Ended && touch.tapCount == 1 {  
    for action in actionsOnTap { 
     action(touch) 
    } 
} 

また、余分な配列がfilterが行う方法をコピー作成されません。これは、filterを使用しない一般的な理由ではありません。 filterは非常に強力なツールですが、この場合はforを使用するとより効率的なとがより明確になります。

guardではなく、@ originaluser2の提案whereを使用するように編集されました。それはおそらくより良いスウィフトです。