斯卡拉尝试用最后的最佳实践

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)