Überprüfen, ob eine Zeichenfolge nur eine Reihe von Buchstaben enthält

Question

Ich versuche zu überprüfen, ob ein Wort nur eine Reihe von Buchstaben wie I, O, S, H und X enthält Angenommen, der Benutzer gibt ein: SSHX, wird die Ausgabe ja, aber wenn der Benutzer SHEXX eintritt, wird der Ausgang NO

public static void main(String[] args) { 
    Scanner sc = new Scanner(System.in); 

    String word = sc.next(); 
    word = word.toUpperCase(); 

    int length = word.length(); 
    char letter = 0; 

    for (int counter = 0; counter < length; counter++) { 
     letter = word.charAt(counter); 
    } 
    if (letter != 'I' && letter != 'O' && letter != 'S' && letter != 'H' && letter != 'Z' && letter != 'X' && letter != 'N') { 
     System.out.print("NO"); 
    } else { 
     System.out.print("YES"); 
    } 
} 

Answer 1

Sie haben eine gute Möglichkeit, es zu lösen. Das Problem ist, dass Sie nicht wirklich jeden Buchstaben überprüfen, also müssen Sie die Prüfungen innerhalb der for Schleife durchführen, oder Sie überprüfen nur den letzten Buchstaben. Aber dann drucken Sie können nicht die „JA“, wie Sie es nur drucken möchten, wenn alle Buchstaben ja sind, so dass Sie einen boolean Wert verwenden können, das auch zu überprüfen, wie zum Beispiel:

boolean isMatch = true; 
    for (int counter = 0; counter < strLength && isMatch; counter++) { 
     letter = word.charAt(counter); 
     if (letter != 'I' && letter != 'O' && letter != 'S' && letter != 'H' && letter != 'Z' && letter != 'X' && letter != 'N') { 
      System.out.print("NO"); 
      isMatch = false; 
     } 
    } 
    if (isMatch) { 
     System.out.print("YES"); 
    } 

Aber, wie andere haben einen regulären Ausdruck wies darauf hin, effektiver ist (und dieser eine Arbeits regex hat für das, was Sie wollen .. Das Sternchen bedeutet null oder mehr von dem, was in den Klammern ist.):

if (word.matches("[HIOSX]*")) { 
     System.out.print("YES"); 
    } else { 
     System.out.print("NO"); 
    } 

Answer 2

einen regulären Ausdruck verwenden:

if (word.matches("[HIOSX]+")) 
    System.out.println("YES"); 
else 
    System.out.println("NO"); 

Answer 3

Verwenden Sie ein regular expression.

String regex = "[OSXHI]*"; 
String string = "SOMETHING"; 
Matcher matcher = Pattern.compile(regex).matcher(string); 
if (matcher.find()) 
{ 
    String match = matcher.group(1); 
    System.out.println(match); 
} 

Einige zusätzliche Ressourcen:

• http://www.regular-expressions.info/
• Regular Expression Language - Quick Reference
• Regular Expressions - User Guide

Answer 4

Abgesehen von den offensichtlichen Antwort von einem regulären Ausdruck verwenden, sollten Sie machen mit Googles Guava API zu Das ist ziemlich einfach le:

if(CharMatcher.anyOf("HIOSX").matchesAllOf(word)) { 

} ... 

Answer 5

allererst Sie sollte der Buchstabe wie folgt initialisiert werden: char letter = '0'; anstelle von 0 Sekunde alle Ihre for-Schleife ist schlecht verwendet, versuchen Sie diesen Code:

boolean isInSet; 
    for (int counter = 0; counter < strLength; counter++) 
    { 
     letter = word.charAt(counter); 
     if (letter != 'I' && letter != 'O' && letter != 'S' && letter != 'H' && letter != 'Z' && letter != 'X' && letter != 'N') 
     { 
      isInSet=false; 
      counter=strlength; //end loop 
     } 
     else 
     { 
      isInSet=true; 
     } 
    } 
    if(isInSet=true) 
    { 
     System.out.print("YES"); 
    } 
    else 
    { 
     System.out.print("NO"); 
    } 

Nun wird die Schleife Schleife über die Zeichenfolge und prüfen, ob jedes Zeichen in dem Satz ist, wenn es die Schleifenenden und die Boolesche ist nicht auf false gesetzt, die in dem NO Ausgabeergebnisse