斯卡拉嘗試用最後的最佳實踐

Question

，我有以下實施方案，其中我想在任何致命的異常處理合適的資源收盤：

private def loadPrivateKey(keyPath: String) = { 
    def tryReadCertificate(file: File): Try[BufferedReader] = Try { new BufferedReader(new FileReader(file)) } 

    def tryLoadPemParser(reader: BufferedReader): Try[PEMParser] = Try { new PEMParser(reader) } 

    def createXXX(buffReader: BufferedReader, pemParser: PEMParser) = try { 
    ... 
    } finally { 
     buffReader.close() 
     pemParser.close() 
    } 
    tryReadCertificate(new File(keyPath, "myKey.pem")) match { 
     case Success(buffReader) => tryLoadPemParser(buffReader) match { 
     case Success(pemParser) => createXXX(buffReader, pemParser) 
     case Failure(fail) => 
     } 
     case Failure(fail) => 
    } 
    } 

我已經看到我的巢式病例塊是一個爛攤子。有一個更好的方法嗎？最後，我只想確保關閉BufferedReader和PEMParser！

Answer 1

你可以調整你的代碼有點像這樣，使用換理解，以清理一些嵌套case語句：

def tryReadCertificate(file: File): Try[BufferedReader] = Try { new BufferedReader(new FileReader(file)) } 

def tryLoadPemParser(reader: BufferedReader): Try[PEMParser] = Try { new PEMParser(reader) } 

def createXXX(buffReader: BufferedReader, pemParser: PEMParser) = { 
    ... 
} 

val certReaderTry = tryReadCertificate(new File(keyPath, "myKey.pem")) 
val pemParserTry = for{ 
    certReader <- certReaderTry 
    pemParser <- tryLoadPemParser(certReader) 
} yield { 
    createXXX(certReader, pemParser) 
    pemParser 
} 

certReaderTry foreach(_.close) 
pemParserTry foreach (_.close) 

結構這樣，你將只落得調用close的事情你確定已成功打開。

更妙的是，如果你的PEMParser發生延長java.io.Closeable，這意味着Try小號都包裹Closeable對象，那麼你可以交換最後兩行的一行是這樣的：

(certReaderTry.toOption ++ pemParserTry.toOption) foreach (_.close) 

編輯

爲響應OP的評論：在第一個例子，如果tryreadCertificate成功，那麼certReaderTry將是一個Success[BufferedReader]因爲它是成功的，叫foreach就會產生BufferedReader，然後它會關閉它。如果certReaderTry是Success，那麼（通過for-comp），我們將調用tryLoadPemParser，如果這也成功，我們可以轉到createXXX並將tryLoadPemParser分配給pemParserTryval。然後，如果pemParserTry是Success，那麼foreach產生PEMParser也會發生同樣的情況，我們可以關閉它。根據這個例子，只要那些Trys是成功的，並且其他意想不到的事情（例如在createXXX中）將會一直拋出一個異常，那麼你可以保證在最後關閉相關代碼其工作和關閉這些資源。

EDIT2

如果你在一個單獨的Try想從createXXX值，那麼你可以做這樣的事情：

val certReaderTry = tryReadCertificate(new File(keyPath, "myKey.pem")) 
val pemParserTry = certReaderTry.flatMap(tryLoadPemParser) 
val resultTry = for{ 
    certReader <- certReaderTry 
    pemParser <- pemParserTry 
} yield createXXX(certReader, pemParser)