prefer_contains

Always 'false' because 'indexOf' is always greater than or equal to -1.

Always 'true' because 'indexOf' is always greater than or equal to -1.

Unnecessary use of 'indexOf' to test for containment.

O analisador emite este diagnóstico quando o método indexOf é usado e o resultado é comparado apenas com -1 ou 0, de um modo em que a semântica é equivalente a usar contains.

O código a seguir produz este diagnóstico porque a condição da instrução if verifica se a lista contém a string:

void f(List<String> l, String s) {
  if (l.indexOf(s) < 0) {
    // ...
  }
}

Use contains no lugar, negando a condição quando necessário:

void f(List<String> l, String s) {
  if (l.contains(s)) {
    // ...
  }
}