Java ,多个变量都只允许是 0 或 1，怎么写简单

[nutting]

if (!(open.equals("0") || open.equals("1")) && (w1.equals("0") || w1.equals("1"))
&& (w2.equals("0") || w2.equals("1")) && (w3.equals("0") || w3.equals("1"))
&& (w4.equals("0") || w4.equals("1")) && (w5.equals("0") || w5.equals("1"))
&& (w6.equals("0") || w6.equals("1")) && (w7.equals("0") || w7.equals("1"))) {
throw new Exception("设置项目参数非法！");
}

checkstyle 检测说表达式过于复杂，怎么简化，最好别用 java8 以上的什么新语法

[corningsun]

提醒下，open 前面有个 `!`，只对 open 有效。建议写好单元测试再重构


https://gist.github.com/corningsun/1afdd69da58d3ed7888ce022be76d4da

[Gwkang]

变参加递归

[akira]

既然你只是要过 chekcstyle 。。。

if (!(open.equals("0") || open.equals("1")) ) {
throw new Exception("设置项目参数非法！");
}
if (!(w1.equals("0") || w1.equals("1")) ) {
throw new Exception("设置项目参数非法！");
}
if (!(w2.equals("0") || w2.equals("1")) ) {
throw new Exception("设置项目参数非法！");
}

[stevefan1999]

真值表啊

[stevefan1999]

草那這樣好了
```js
const isAcceptable = str => /0|1/.matches(str)
if ([open, w1, w2, w3, w4, w5, w6, w7].map(isAcceptable).filter(x => !x).length > 0) {
throw new Exception("设置项目参数非法！");
}
```

[ojbkojbk]

你这个参数应该是从接口传入的吧，在接口的入参用注解正则校验
另外在 equals 的时候 ，常量放在前面，防止封装类型传入空报指针

[xx6412223]

这代码看着被人看了要被骂的

[weizhen199]

0000,0001,1101,0101

lz 你看看这个像啥

[TheF00L]

枚举多好

[daryl]

枚举不行么？

[checkzhzzzzz]

map.put(str,"");
map.size()

[SkyLine7]

枚举