スカラループを1行に簡略化

Question

このループをforeachやmapなどの関数やScalaで単純化するにはどうすればよいですか？私はそのフィルターshipList.filterの中にhitsArrayを入れたい。あなたはどこでもそれを使用しないようごshipSize = shipList.lengthを呼び出している理由

val hitsArray: Array[String] = T.split(" "); 
for (hit <- hitsArray) { 
    shipSize = shipList.length 
    shipList = shipList.filter(!_.equalsIgnoreCase(hit)) 
} 
if (shipList.length == 0) { 
    shipSunk = shipSunk + 1 
} else if (shipList.length < shipSize) { 
    shipHit = shipHit + 1 
} 

Answer 1

公平を期すために、私は理解していません。

T.split(" ").foreach{ hit => 
    shipList = shipList.filter(!_.equalsIgnoreCase(hit)) 
} 

あなたが行きたい場所に移動します。私はそれが副作用を介して働いていることを強調したいので、それを3行にしました。foreach。それは、私はそれを1ライナーにすることに利点がないと言いました。あなたが以前に持っていたものは完全に読みやすいものでした。

Answer 2

これはおそらく何か？

shipList.filter(ship => T.split(" ").forall(!_.equalsIgnoreCase(ship))) 

クリーナーが、shipListはすでに、すべて小文字である場合：

shipList.filterNot(T.split(" ").map(_.toLowerCase) contains _) 

またはあなたのTは、ループの外に移動し、大きい場合：

val hits = T.split(" ").map(_.toLowerCase) 
shipList.filterNot(hits contains _)