ケースクラスから最大値を取得するノードのリスト、スカラを返すOption [List]

Question

私が書いたケースクラスNodeがあり、そのリストを作成して最大ディスクを持つノードを見つける必要があります。 私は以下のコードを書いた、それを行う良い方法はありますか？また、私の実際の生産コードでは、私の "nodeList"変数はOption[List[Node]]だけでなく、Future[Option[List[Node]]]になります。私はまだ答え/コードは、私が地図/ flatMapをやって、未来に行って同じことをすることを除いて、あまり変わらないと思う。 誰かがScalaの方法でコードを書くより良い提案があれば、あなたの考えを分けてください。あなたが書いたコードから判断する

scala> case class Node(disk: Integer, name: String) 

定義されたクラスノード

scala> val nodeList = Option(List(Node(40, "node1"), Node(200, "node3"),Node(60, "node2"))) 
nodeList: Option[List[Node]] = Some(List(Node(40,node1), Node(200,node3), Node(60,node2))) 

scala> val maxDisk = nodeList match { 
| case None => println("List is empty"); None 
| case Some(lst) => { 
| Some(lst.max(Ordering.by((_:Node).disk))) 
| } 
| }` 
maxDisk: Option[Node] = Some(Node(200,node3)) 

Answer 1

、私はあなたが本当にOptional[List[Node]]を使うべきかはわかりません。 Noneを空のListとして扱い、Someの場合は空のリストをチェックしていないようです。 List[Node]があなたの使用によく合っているかどうかを確認したい場合があります（はNilになり、Some(lst)はちょうどlstになります）、使用されていないSome(Nil)のケースは誰も混乱させません。

あなたがOptional[List[Node]]を維持した場合は、私はこのようにそれを行うだろう：

nodeList 
    .filterNot(_.isEmpty) // maxBy throws if the list is empty; check for it 
    .map(_.maxBy(_.disk)) // maxBy looks nicer than max(Ordering.by) 

あなたはList[Node]に切り替えると、それは少し醜いです：

Some(nodeList) 
    .filterNot(_.isEmpty) // We're using the filter utility of Option here, 
    .map(_.maxBy(_.disk)) // so I wrap with Some to get access to filterNot. 

Answer 2

あなたがリストのパターンマッチングで再帰を使用することができます。

case class Node(disk: Integer, name: String) 
    val nodeList = Option(List(Node(40, "node1"), Node(200, "node3"),Node(60, "node2"))) 

    def findMaxValue(list: List[Node]): Option[Node] = list match { 
    case Nil => None 
    case List(x) => Some(x) 
    case first :: second :: rest => if(first.disk > second.disk) findMaxValue(first::rest) else findMaxValue(second::rest) 
    } 

    val node:Option[Node] = findMaxValue(nodeList.getOrElse(Nil)) 
    println(node.get.disk) //print 200