Comment tester si un caractère n'est pas dans une chaîne? (java, junit)

Question

Comme le dit le titre, j'ai des problèmes avec mes tests junit qui passent pour vérifier si un caractère n'est pas dans une chaîne et comment vérifier si une chaîne vide n'a pas de caractère. voici la méthode que j'ai:

 public static boolean isThere(String s, char value){ 
    for(int x = 0; x <= s.length(); x++){ 
    if(s.charAt(x) == value){ 
    return true; 
    } else if(s.length() == 0){ 
    return false; 
    } 
    } 
    return false; 

Et voici le test JUnit:

public void testIsThere() { 
    { 
    String sVal = "Jeff George"; 
    boolean hasA = StringMethods.isThere(sVal,'e'); 
    assertTrue(hasA); 
    boolean hasE = StringMethods.isThere(sVal, 'o'); 
    assertTrue(hasE); 
    boolean notIn = StringMethods.isThere(sVal,'b'); 
    assertTrue(notIn); 
    } 
    { 
    String sVal = ""; 
    boolean nothingIn = StringMethods.isThere(sVal,'a'); 
    assertFalse(nothingIn); 
    boolean notIn = StringMethods.isThere(sVal,'b'); 
    assertFalse(notIn); 
    } 
} 

Merci beaucoup, apprécié

Answer 1

Utilisation String.indexOf() à la place:

public static boolean contains(String s, char value){ 
    return s != null && s.indexOf(value) > -1; 
} 

String sVal = "Jeff George"; 
assertTrue(contains(sVal, 'e')); 
sVal = null; 
assertFalse(contains(sVal, 'e')); 

Answer 2

Pourquoi faites-vous cela? Votre fonction est déjà implémentée en tant que méthode sur String. Utilisez String.indexOf à la place:

s.indexOf('a') == -1 

Je pense que Carl Manaster avait raison dans les commentaires au sujet de votre problème - vous devez utiliser assertFalse pas assertTrue ici:

String sVal = "Jeff George"; 
boolean notIn = StringMethods.isThere(sVal, 'b'); 
assertFalse(notIn); // not assertTrue 

En aparté, notIn est un nom terrible pour cette variable - cela signifie exactement le contraire de ce qu'il dit. Peut-être que c'est pour ça que tu t'es embrouillé.

Answer 3

Quel problème rencontrez-vous?

Tout d'abord,

for(int x = 0; x <= s.length(); x++){ 

ne regarde pas droit. x va se terminer à la fin de votre chaîne (use x < s.length() à la place si vous voulez parcourir une chaîne). Mais les fonctions de niveau supérieur sont disponibles pour faire ce que vous voulez (voir les autres réponses ici).

Answer 4

Si String.indexOf (char) renvoie -1, hasA est faux. Sinon, c'est vrai.

Answer 5

Avec Java 6, vous pouvez simplement faire

final String s = "This is a test"; 
s.contains("x"); // False 
s.contains("t"); // True 

Answer 6

Ou, essayez StringUtils.contains() de biens communs apache - qui traitera le cas String null pour vous.

http://commons.apache.org/lang/api/org/apache/commons/lang/StringUtils.html#contains%28java.lang.String,%20char%29