【日期】
2022/4/11
【问题】
今天，在和前端同事A对接接口的时候，发现前端传入空字符串，也能存入状态。（这里是把状态弄成一个公共方法，判断一个字符串不在这些字符串里）
例如：判断data是否有txt
/** * 判断data是否有txt * * @Method: isTxtInDate * @author: nanfangzhe * @date: 2022年4月11日 * @param txt * @param data * @return boolean */ public static boolean isTxtInDate(String txt, String data) { // return data.indexOf(txt) != -1; }
但是，这里是存在问题的，如果传入空的字符串：""，长度为0；返回结果也是true，就会导致出错了！
如图：
【原因】
原因是：indexOf的源码里，字符串长度是0，会直接返回规则的Data数据的长度，而不是会返回-1；进而返回false。
if (fromIndex >= sourceCount) { return (targetCount == 0 ? sourceCount : -1); }
【如何发现】
查看String.indexOf的源码
我们也来了解和分析一下indexOf()的原理，探索底层原理
这里就直接把理解和分析写在注释上了。（这里是直接复制最底层的indexOf的方法）
static int indexOf(char[] source, int sourceOffset, int sourceCount, char[] target, int targetOffset, int targetCount, int fromIndex) { // fromIndex从第几个开始对比；匹配是，默认为0； // sourceCount是data（大的字符串）的长度 if (fromIndex >= sourceCount) { // 这里data的长度如果是0，就直接被返回了 return (targetCount == 0 ? sourceCount : -1); // targetCount也是data的长度，所以直接返回了0，而不是返回-1 } if (fromIndex < 0) { fromIndex = 0; } if (targetCount == 0) { return fromIndex; } char first = target[targetOffset]; int max = sourceOffset + (sourceCount - targetCount); for (int i = sourceOffset + fromIndex; i <= max; i++) { /* Look for first character. */ if (source[i] != first) { while (++i <= max && source[i] != first); } /* Found first character, now look at the rest of v2 */ if (i <= max) { int j = i + 1; int end = j + targetCount - 1; for (int k = targetOffset + 1; j < end && source[j] == target[k]; j++, k++); if (j == end) { /* Found whole string. */ return i - sourceOffset; } } } return -1; }
【如何修复】
底层逻辑了解清楚了，肯定不能改动了，我们就在封装的方法里修改。
两种解决：① 控制先判断为空就返回false ② 再加一个 data.indexOf(txt) != 0
这里选择第一种方法。最终选择，多加一个判断commons.lang3的
StringUtils.isBlank(txt).
修改后的代码如下：
public static boolean isTxtInDate(String txt, String data) { return StringUtils.isBlank(txt) == true ? false : data.indexOf(txt) != -1; // 如果txt为空，直接返回false（因为indexOf判断传入长度为0的字符串也会返回true） }
StringUtils.isBlank()方法，需要引入的pom文件
<dependency> <groupId>org.apache.commons</groupId> <artifactId>commons-lang3</artifactId> <version>3.12.0</version> </dependency>
【完整代码分享】
package com.nanfangzhe.anpai; import org.apache.commons.lang3.StringUtils; public class Demo { public static void main(String[] args) { Integer status = 0; String textRule = "1,2,3"; System.out.println("状态必须符合：" + textRule); System.out.println("status = 0：" + isTxtInDate(status.toString(), textRule) + "\n不继续往下执行"); System.out.println("status = \"\"：" + isTxtInDate("", textRule) + "\n继续往下执行"); } /** * 判断data是否有txt * * @Method: isTxtInDate * @author: nanfangzhe * @date: 2022年4月11日 * @param txt * @param data * @return boolean */ public static boolean isTxtInDate(String txt, String data) { return StringUtils.isBlank(txt) == true ? false : data.indexOf(txt) != -1; // 如果txt为空，直接返回false（因为indexOf判断传入长度为0的字符串也会返回true） } }
【总结】
封装方法时要考察该底层方法，进行也要测试到位！
文章小尾巴
文章写作、模板、文章小尾巴可参考：《写作“小心思”》
感谢你看到最后，最后再说两点~
①如果你持有不同的看法，欢迎你在文章下方进行留言、评论。
②如果对你有帮助，或者你认可的话，欢迎给个小点赞，支持一下~
（文章内容仅供学习参考，如有侵权，非常抱歉，请立即联系作者删除。）
