apache-commons-lang.StringUtils 的疑惑

[thinkmore]

我在官网下载最新的 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 上面。。。

[wmhx]

其实你想放 F4,F5 的.

[fwrq41251]

直接在 github 上提 issue 好了啊

[SoloCompany]

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

[SoloCompany]

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

[thinkmore]

@SoloCompany 贴代码我怕看着混乱

[thinkmore]

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

[SoloCompany]

@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]

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

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

[SoloCompany]

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

[thinkmore]

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