减少条件运算符的数量

时间:2017-01-19 07:15:21

标签: java

我有一个String arraylist。让我们说

private final List<String> fruits = new ArrayList<String>();

现在我必须将传入的行与arraylist中的项进行比较

while ((line = in.readLine()) != null) {
    if (!(line.equals(fruits.get(0)) || line.contains(fruits.get(1)) ||
        line.contains(fruits.get(2)) || line.contains(fruits.get(3)) ||
        line.contains(fruits.get(4)) || line.contains(fruits.get(5)) ||
        line.contains(fruits.get(6)) || line.equals(fruits.get(7)    ||  line.equals(fruits.get(8)))) {
          //                   "DO SOMETHING"
     }
}

我必须在某些情况下完全匹配字符串,并在某些情况下使用contains。但最后我的if子句中不应超过3个条件。

2 个答案:

答案 0 :(得分:1)

您的要求不明确。等同性检查是专门针对索引7还是仅针对8或者是什么。但无论如何这是我的建议。您可以使用一种简单的方法来检查line是否包含list

中的子集
public boolean isFound(List<String> f, String l){
    for(int i=0;i<f.size();i++){
        if(l.contains(f.get(i)){
            return true;
        }
    }
    return false;
}

然后你可以这样检查:

if(isFound(fruits, line) || fruits.contains(line)){
    //Do Something
}

答案 1 :(得分:1)

由于您希望在列表中的每个水果上使用equals()或contains(),并且您的成果不断增长,请考虑将列表转换为地图,然后通过水果存储所需的方法。

private enum Method {
    CONTAINS,
    EQUALS;
}

@Test
public void testFruits() throws IOException {
    Map<String, Method> methodByFruit = new HashMap<>();
    methodByFruit.put("apple", Method.CONTAINS);
    methodByFruit.put("pear", Method.CONTAINS);
    methodByFruit.put("grenade apple", Method.CONTAINS);
    methodByFruit.put("banana", Method.EQUALS);
    methodByFruit.put("kiwi", Method.EQUALS);

    BufferedReader in = new BufferedReader(new StringReader("kiwi2"));

    String line;
    while ((line = in.readLine()) != null) {
        boolean success = false;
        for (Entry<String, Method> entry : methodByFruit.entrySet()) {
            String fruit = entry.getKey();
            Method method = entry.getValue();
            if (method == Method.EQUALS) {
                success = line.equals(fruit);
            } else {
                success = line.contains(fruit);
            }
            if (success) {
                break;
            }
        }
        if (!success) {
            System.out.println("DO SOMETHING");
        }
    }
}