在构造函数后使用安全导航运算符是否有合理理由?
关于在File构造函数后使用
?.运算符的合理性分析 你这个疑问抓得特别准!先直接给结论:在标准Java/Groovy的File构造函数后面使用?.完全没有必要,这本质上是对File构造函数行为的误解。咱们一步步拆解:
首先明确File构造函数的核心行为
标准JDK的File构造函数有个关键特性:无论传入的路径是否存在、是否有效,甚至传入null,它都会返回一个File实例,绝对不会返回null。它只会在后续操作(比如读取文件内容、判断文件存在性)时,才会因路径问题抛出异常或返回错误结果,但构造阶段本身永远不会返回null。
所以原代码里的:
def pass = new File("password_file.txt")?.text ?: "pass"
其实等价于:
def pass = new File("password_file.txt").text ?: "pass"
这里的?.完全是冗余的,起不到任何实际作用。
为什么会有人这么写?可能的误解场景
能想到的原因大概有两种:
- 混淆了构造函数和工厂方法:有些类的工厂方法(比如某些第三方库的工具类)可能会在参数无效时返回
null,但File的构造函数不属于这类情况。 - 想错误地防御异常:写代码的人可能以为
?.能处理文件不存在、无权限读取的情况,但实际上?.只能处理前面的对象为null的场景,根本捕获不了text属性读取时抛出的IOException。原代码里的?: "pass"也不会触发,因为异常会直接中断代码执行,走不到Elvis运算符的分支。
原代码的正确写法(实现“读不到文件就用默认密码”的需求)
如果想实现“能读到文件内容就用,否则用默认值pass”,正确的做法应该是捕获读取时的异常,或者先判断文件的可访问性:
方式一:捕获异常(更稳妥,覆盖所有读取失败的情况)
def pass = try { new File("password_file.txt").text.trim() // 可以加trim处理空白 } catch (IOException e) { "pass" }
方式二:先判断文件状态
def passwordFile = new File("password_file.txt") def pass = passwordFile.exists() && passwordFile.canRead() ? passwordFile.text : "pass"
总结
回到你的问题:在File构造函数后使用?.没有任何合理理由,属于对API行为不熟悉导致的冗余写法。
内容的提问来源于stack exchange,提问作者Jezor
相关产品推荐
相关产品推荐

