Scalaを使って漫画を読む

Question

今、次のコードで漫画本のファイル（.cbr &cbz）を開いていますが、もっとうまくいくと思います。それについてどうやって行きますか？

@throws(classOf[Exception]) 
def extract(filePath: String): File = { 
    val unZip = new runCommand(Seq("7z", "x", "-y", filePath, s"-o${tempLocation}")) 
    if (unZip.exitValue == 0){ 
     new File(tempLocation) 
    }else{ 
     throw new Exception(unZip.errValue) 
    } 
} 
@throws(classOf[Exception]) 
def preExtract(filePath: String): File = { 
    try { 
     filePath.split("\\.").last match { 
      case "cbz" => extract(filePath) 
      case "cbr" => extract(filePath) 
      case _ => 
       throw new FileNotFoundException("Incorrect file extension for file: " + filePath) 
     } 
    } catch { 
     case ex: Exception => 
      throw ex 
    } 
} 

Answer 1

私はあなたがscala.util.Tryを使用しての代わりに、例外をキャッチして再スローにルックアップするためにお勧めします。 Hereは、このトピックに関する素晴らしい記事です。

ここでは、使用しているrunCommand()機能に慣れていません。おそらく自分のものですか？その場合、エラーが発生しなければ、OptionまたはNoneにラップされたエラーコードを返すように変更することができます。もしそうなら、あなたのスニペットは2個のネストされたmatchブロックを持つ単一の機能として実装することができます。

def extract(filePath: String): Try[File] = filePath match { 
    case str if str.endsWith(".cbz") || str.endsWith(".cbr") => 
    runCommand(Seq(...)) match { 
     case None => new File(tempLocation) 
     case Some(errValue) => throw new UnzipFailedException(errValue) 
    } 
    case _ => throw new InvalidArgumentException(s"Incorrect file extension for file: $filePath") 
} 

私はこのコードをテストしていないが、多分それはいくつかの助けのです。