这种实现方法的问题在哪里？

[初晓暮年]

A,B,C,D四个文本框 报错msg是”请在A和B或者C和D中输入内容” 我的check实现代码如下 if (StringUtils.isEmpty(A) != StringUtils.isEmpty(B) || StringUtils.isEmpty(C) != StringUtils.isEmpty(D)) ||(StringUtils.isEmpty(A) && StringUtils.isEmpty(B) && StringUtils.isEmpty(C) && StringUtils.isEmpty(D)) { showmsg; } 这种写法存在什么问题？比如测试覆盖率会比较高，不容易理解等等？

[greadworld]

不懂为什么要写那么长，下面这个你把0当作空，然后把.equals("0")换成StringUtils.isEmpty 就是你要的了 public static void main(String[] args) { List<String[]> strings = new ArrayList<>(); strings.add("0,0,0,0".split(",")); strings.add("1,0,0,0".split(",")); strings.add("0,1,0,0".split(",")); strings.add("0,0,1,0".split(",")); strings.add("0,0,0,1".split(",")); strings.add("1,1,0,0".split(",")); strings.add("1,0,1,0".split(",")); strings.add("1,0,0,1".split(",")); strings.add("0,1,1,0".split(",")); strings.add("0,1,0,1".split(",")); strings.add("0,0,1,1".split(",")); strings.add("1,1,1,0".split(",")); strings.add("1,1,0,1".split(",")); strings.add("1,0,1,1".split(",")); strings.add("0,1,1,1".split(",")); strings.add("1,1,1,1".split(",")); for(String[] s : strings ){ String a = s[0]; String b = s[1]; String c = s[2]; String d = s[3]; if ((a.equals("0") || b.equals("0")) && (c.equals("0") || d.equals("0"))) { System.out.println("错误消息"); } else { System.out.println("正确消息"); } } }

[初晓暮年]

引用 2 楼 greadworld的回复:

不懂为什么要写那么长，下面这个你把0当作空，然后把.equals("0")换成StringUtils.isEmpty 就是你要的了 public static void main(String[] args) { List<String[]> strings = new ArrayList<>(); strings.add("0,0,0,0".split(",")); strings.add("1,0,0,0".split(",")); strings.add("0,1,0,0".split(",")); strings.add("0,0,1,0".split(",")); strings.add("0,0,0,1".split(",")); strings.add("1,1,0,0".split(",")); strings.add("1,0,1,0".split(",")); strings.add("1,0,0,1".split(",")); strings.add("0,1,1,0".split(",")); strings.add("0,1,0,1".split(",")); strings.add("0,0,1,1".split(",")); strings.add("1,1,1,0".split(",")); strings.add("1,1,0,1".split(",")); strings.add("1,0,1,1".split(",")); strings.add("0,1,1,1".split(",")); strings.add("1,1,1,1".split(",")); for(String[] s : strings ){ String a = s[0]; String b = s[1]; String c = s[2]; String d = s[3]; if ((a.equals("0") || b.equals("0")) && (c.equals("0") || d.equals("0"))) { System.out.println("错误消息"); } else { System.out.println("正确消息"); } } }

结果并不对， AB，CD两两输入状态必须要一样的

[maradona1984]

如果你的StringUtils是commons-lang3的工具类,你可以尝试用isAllEmpty这个方法 如果是自己写的,那就自己封装一个isAllEmpty的方法

[greadworld]

引用 3 楼 qq_28392343 的回复:

[quote=引用 2 楼 greadworld的回复:]不懂为什么要写那么长，下面这个你把0当作空，然后把.equals("0")换成StringUtils.isEmpty 就是你要的了 public static void main(String[] args) { List<String[]> strings = new ArrayList<>(); strings.add("0,0,0,0".split(",")); strings.add("1,0,0,0".split(",")); strings.add("0,1,0,0".split(",")); strings.add("0,0,1,0".split(",")); strings.add("0,0,0,1".split(",")); strings.add("1,1,0,0".split(",")); strings.add("1,0,1,0".split(",")); strings.add("1,0,0,1".split(",")); strings.add("0,1,1,0".split(",")); strings.add("0,1,0,1".split(",")); strings.add("0,0,1,1".split(",")); strings.add("1,1,1,0".split(",")); strings.add("1,1,0,1".split(",")); strings.add("1,0,1,1".split(",")); strings.add("0,1,1,1".split(",")); strings.add("1,1,1,1".split(",")); for(String[] s : strings ){ String a = s[0]; String b = s[1]; String c = s[2]; String d = s[3]; if ((a.equals("0") || b.equals("0")) && (c.equals("0") || d.equals("0"))) { System.out.println("错误消息"); } else { System.out.println("正确消息"); } } }

结果并不对， AB，CD两两输入状态必须要一样的 [/quote] 并没有看到你有这需求

[greadworld]

if ((a.equals("0") || b.equals("0") || !a.equal(b)) && (c.equals("0") || d.equals("0") || !c.equals(d)))