一个有意思的小细节

今天一个新来的小同事写下了这样一段Java代码:
代码很简单:

if (entryPath.contains("\\")) {
  entryPath.replaceAll("\\", '/');
}

仔细看一下这段代码,其实是很有意思的
这段代码的本意是要查找entryPath中的反斜杠字符,如果有的话就全部替换掉,可是这段代码至少犯了两个严重错误,还有一个不那么严重的错误。
1.replaceAll的参数使用的是正则表达式,而在正则表达式中反斜杠字符的匹配是字符串\\,那么看起来上面写的似乎没错,对吗?
错了,正确的写法应该是entryPath.replaceAll("\\\\", "/");因为这样在运行的时候表达式才会是\\。

2.entryPath.replaceAll("\\\\", "/");真正达到了效果吗?当然没有,Java中String是不可变的,所以这句话其实根本没有任何效果,替换完的字符串被丢掉了。

3.效率问题:contains要启动字符串的匹配,需要做很多工作,而replaceAll要启动正则表达式的编译,工作量更大,其实上面两句话可以简单的修改为:
String path = null;
if (entryPath.indexOf('\\') != -1) {
	path = entryPath.replace('\\', '/');
}


  • 1
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值