apache-commons-lang.StringUtils 的疑惑

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

这是一个创建于 3040 天前的主题，其中的信息可能已经有所发展或是发生改变。

我在官网下载最新的 lang3 3.4 版本的代码，游览它源代码的时候发现 ordinalIndexOf 这个方法有比较大的变动，我甚至怀疑它是错误的，我和 github 上最新的代码以及和 2.6 版本的同一个方法进行了比对，发现了只有 3.4 这个版本进行变化了，从而在使用这个方法的时候没法达到我预期想要的结果，问下大家知道为啥吗？还是我理解错了？

github 上最新的代码： http://7xlvme.com1.z0.glb.clouddn.com/tmp-github.png

3.4 版本最新的代码： http://7xlvme.com1.z0.glb.clouddn.com/tmp-3.4.4.png

最后问下，为啥我看新闻的时候，会把手放到 F5,F6 上面。。。

代码

GitHub

版本

最新

10 条回复 • 2016-07-15 22:44:09 +08:00

wmhx

2016-07-12 14:52:58 +08:00

其实你想放 F4,F5 的.

fwrq41251

2016-07-13 00:07:48 +08:00

直接在 github 上提 issue 好了啊

SoloCompany

2016-07-13 02:27:52 +08:00

为啥要贴图而不是直接帖代码 hash 。。。

SoloCompany

2016-07-13 02:32:02 +08:00

顺便说一下，我看新闻的时候（ ReadKit ），手是放在 J 和 V 上面的

thinkmore

2016-07-14 09:56:36 +08:00

@SoloCompany 贴代码我怕看着混乱

thinkmore

2016-07-14 09:57:06 +08:00

@fwrq41251 就是因为没提上，我给贡献者发了邮件也没人回我

SoloCompany

2016-07-14 15:52:27 +08:00

@thinkmore
https://github.com/apache/commons-lang/blob/LANG_3_4/src/main/java/org/apache/commons/lang3/StringUtils.java#L1015

https://github.com/apache/commons-lang/blob/7fd021d82ff431fb31f42bc6c5c44a3b979cb426/src/main/java/org/apache/commons/lang3/StringUtils.java#L1400

怎么就乱了呢？看代码只看个贴图，脱离上下文以及 commit 记录，是否有问题就很难判断了

thinkmore

2016-07-15 10:06:52 +08:00

@SoloCompany 我说的乱是因为两个方法在关键之处并不一致，同一个方法不同的版本输出的结果并不一致，我也不能理解为什么这样做。。

你能给我解析下吗？谢谢了

SoloCompany

2016-07-15 14:25:30 +08:00

@thinkmore 我没有花时间去看代码，但我看了 7fd021d82ff431fb31f42bc6c5c44a3b979cb426 的 commit log 不但说明的很清晰 “ LANG-1241 StringUtils.ordinalIndexOf broken" 而且同时提交的还有若干测试用例，因此我并不怀疑它有问题，更何况这只是一个私有函数。建议你多看看上下文再判断

thinkmore

2016-07-15 22:44:09 +08:00

@SoloCompany 好的，我认真去看看谢谢啦