Лучший способ получения булевых значений в Java

Question

Я начинающий программист.

У меня возник вопрос о том, как вернуть значение boolean.

Какой вам нравится? По какой причине?

1.

public boolean methodA(List<String> targetList){ 
    boolean result = false;   
     for(String str : targetList){ 

      if(str.equals("word1")) { 
       result = true; 
       break; 
      } 
      if(str.equals("word2")) { 
       result = true; 
       break; 
      } 
     } 
    return result; 
} 

2.

public boolean methodB(List<String> targetList){ 
    boolean result = false;   
     for(String str : targetList){ 

      if(str.equals("word1")) { 
       return true; 
      } 
      if(str.equals("word2")) { 
       return true; 
      } 
     } 
    return result; 
} 

Answer 1

Это то, что вы в конечном итоге делает, как только ваш код становится абсурдно большой:

public boolean methodB(List<String> targetList) 
{ 
    for(String str : targetList) 
     if(str.equals("word1") || str.equals("word2")) 
      return true; 

    return false; 
} 

Answer 2

Второй метод лучше, потому что если он найдет поисковый запрос, он не будет тратить время на другие значения.

Лучший способ для этого будет:

public boolean methodC(List<String> list) { 
    for (String str : list) 
     if (str.equals("word1") || str.equals("word2")) return true; 
    return false; 
} 

или в Java 8/1.8: (Поправьте меня, если это не так!)

public boolean methodD(List<String> list) { 
    yes: { 
     list.forEach(str -> {if(str.equals("word1") || str.equals("word2")) break yes;}); 
    return false; 
    } 
    return true; 
} 

Answer 3

public boolean methodABC(List<String> list) { 
    return list.contains("word1") || list.contains("word2"); 
} 

Answer 4

Использование потоков, очень простой однострочный:

public boolean methodA(List<String> targetList){ 
    return targetList.stream().anyMatch(s -> s.equals("word1") || s.equals("word2")); 
} 

Answer 5

methodA() является хорошей причиной, поскольку он следует структурированному программированию с одним входом/с одним выходом и выполняет итерацию списка только один раз.

methodA() очищен:

public static boolean methodA(List<String> targetList) { 
    boolean result = false;   
    for(String str : targetList){ 
     if(str.equals("word1") || str.equals("word2")) { 
      result = true; 
      break; 
     } 
    } 
    return result; 
} 

methodB() хорошо, потому что это всего лишь несколько строк короче, итерация по списку раз, занимает меньше памяти (без использования результата boolean, ОЧЕНЬ МЕЛКАЯ память сохранить), возвращается сразу после успеха.

methodB() очищены:

public boolean methodB(List<String> targetList){ 
    for(String str : targetList){ 
     if(str.equals("word1") || str.equals("word2")) { 
      return true; 
     } 
    } 
    return false; 
} 

После изучения потоков, один вкладыш, который следует за однократные/одного выхода структурированного программирования и итерацию списка только один раз.

public static boolean methodX(List<String> targetList){ 
    return targetList 
      .stream() 
      .anyMatch(target -> target.equals("word1") || target.equals("word2")); 
} 

Использование:

public static void main(String[] args) throws Exception { 
    List<String> targets1 = new ArrayList() {{ 
     add("hello"); 
     add("goodbye"); 
    }}; 

    System.out.println(methodA(targets1)); 

    List<String> targets2 = new ArrayList() {{ 
     add("hello"); 
     add("goodbye"); 
     add("word1"); 
    }}; 

    System.out.println(methodA(targets2)); 

    List<String> targets3 = new ArrayList() {{ 
     add("hello"); 
     add("goodbye"); 
     add("word2"); 
    }}; 

    System.out.println(methodA(targets3)); 
} 

Результаты:

false 
true 
true 

Что касается производительности, все они перебирать список только один раз, так ИМО это просто сводится к личным предпочтениям, как, к которому лучше всего использовать. Для меня это было бы methodX()