Scalaでのマップとフィルタのヌルの組み合わせ

Question

マップとヌルスキップが1回の呼び出しで表現されるように、次のコードを表現できますか？

list.map(_.accept(this, arg).asInstanceOf[T]).filter(_ != null) 

Answer 1

list flatMap { i => Option(i.accept(this, arg).asInstanceOf[T]) } 

または代わりに、あなたが好きならば、（これはより多くのまたはあなたの元の式にはあまり変換されますが）

for { 
    item <- list 
    itemConverted = item.accept(this, arg).asInstanceOf[T] 
    itemNonNull = itemConverted if itemConverted != 0 
} yield itemNonNull 

collectを使用することも可能であろうが、それはおそらく二回acceptを呼ぶだろうほとんどの議論では、部分関数のテストのために：

list collect { 
    case i if i.accept(this, arg).asInstanceOf[T] != null => i.accept(this, arg).asInstanceOf[T] 
} 

これを避けるには、メモ帳（またはスマートエクストラクタ）を使う必要があります。

Answer 2

パフォーマンスを懸念している場合は、.view

list.view.map(_.accept(this, arg).asInstanceOf[T]).filter(_ != null) 

viewを追加することができますので、mapとfilterはリスト上の1回のパスではなく、二つの別々のパスで実行されます、トラバーサルは怠惰になるようにします。あなたはこのパターンを再利用する懸念がある場合

、独自のヘルパー関数を定義することができます：私は推測

def mapNN[A,B](list: List[A])(f: A => B) = { 
    list.view.map(f(_)).filter(_ != null) 
} 

mapNN(list)(_.accept(this, arg).asInstanceOf[T]) 

テスト...

> mapNN(List(1,2,3))(x => if (x%2==0) x else null).toList 
res7: List[Any] = List(2)