eingegebene Strings nache Eigenschaften durchsuchen.

[Licht]

Hallo Leute,

für eine Aufgabe muss ich ein eingegebenes Wort nach von mir bestimmten Inhalten durchsuchen lassen.
Zum Beispiel soll nachgeguckt werden, ob die Zahlen 1 - 9 enthalten sind bzw Buchstaben von a-z.

Bin inzwischen so weit gekommen:

import javax.swing.JOptionPane;

public class ContentSearch {

public static void main(String[] args) {
        String Entry = JOptionPane.showInputDialog("Bitte Eingabe machen");

        String Wert = "";

           if (Entry.contains("1,2,3,4,5,6,7,8,9")){
           Wert = "Alle Zahlen von 1 bis 9 enthalten";
       }
         
        
        JOptionPane.showMessageDialog(null, Wert);

    }

Die Frage ist nun, wie der Inhalt in den Klammern denn überhaupt geschrieben werden sollen ???:L :bahnhof: Ich glaube ich mache das dort irgendwie Falsch. Wenn ich alle Zahlen eingebe gibt er mir kein Wert.

Vielen Dank im Voraus :toll:

 

[eRaaaa]

Du könntest anstelle contains -> matches verwenden mit einem entsprechenden Regex z.B. [1-9] oder [a-zA-Z] (evtl mit .* davor + dahinter, je nachdem wie der eigentliche String auszusehen hat..)

 

[hdi]

Entry.contains("1,2,3,4,5,6,7,8,9")

Das liefert nur true, wenn der String die gesamte Zeichenkette "1,2,3,4,5,6,7,8,9" enthält. Inklusive Kommas, und auch in genau dieser Reihenfolge. Also offensichtlich nicht das, was du brauchst. Du musst jede Zahl einzeln prüfen. Mein Vorschlag:

String zahlen = "123456789";
boolean alleZahlenVorhanden = true;
for(char c : zahlen.toCharArray()){
    if(!zuPruefenderString.contains(c)){
       alleZahlenVorhanden = false;
       break;
    }
}
if(alleZahlenVorhanden){
  // ...
}

Analog kannst du das noch für die Buchstaben machen. Hier kannst du aber gleich ne for-Schleife über chars machen, siehe ASCII Tabelle für Zuordnung der Buchstaben zu int-Werten). Kleines 'a' ist zB 97 soweit ich weiß.

PS: evtl geht's noch viel einfacher mit regulären Ausdrücken, aber damit bin ich persönlich nicht so bewandert..

 

[Licht]

@hdi: ja mit Arrays kenne ich mich gar nicht aus und hätte Schwierigkeiten dies zu erklären. Trotzdem vielen Dank für deinen Tipp.

@eRaaaa:

 if (Entry.matches([a-zA-Z])){
    Wert = "alle Buchstaben sind enthalten.";
}

So sagt er mir aber

"illegal start of expression

cannot find symbol
symbol: variable a
location: class uebung.ContentSearch

cannot find symbol
symbol: variable zA
location: class uebung.ContentSearch

cannot find symbol
symbol: variable Z
location: class uebung.ContentSearch

illegal start of expression"

 

[hdi]

Der Ausdruck muss schon ein String sein. Also in Anführungsstriche packen:

Entry.matches("[a-zA-Z]")

Allerdings heißt das nicht, dass alle Buchstaben vorhanden sind, sondern dass genau ein einziger Buchstabe vorhanden ist. Mach dich erstmal mit der Syntax von solchen Ausdrücken vertraut:
Pattern (Java Platform SE 6)

 

[eRaaaa]

Naja, da fehlen die Anführungszeichen vom String, wusste nicht dass ich das extra noch erwähnen muss
Und ich denke auch, dass der Regex so nicht passen wird, wie sieht denn der eingegebene String aus? Kann der irgendwie aussehen und da soll nur geguckt werden ob da irgendwo eine Zahl oder Buchstabe enthalten ist oder wie? Denn da würde da so einfach nicht gehen

 

[hdi]

@era Ich hab das so verstanden dass der übergebene String alle Ziffern 1-9 enthalten muss (Reihenfolge und andere Inhalte wohl egal). Ich glaub fast da ist meine Version noch leichter als mit Regex. Aber ich lass mich gern eines besseren belehren, ich hab mit RegEx wie gesagt nich so die Übung.

PS: bzw. alle Buchstaben, irgendwie nimmt der TO mal ein Beispiel mit Ziffern, mal eins mit Buchstaben..

 

[eRaaaa]

@era Ich hab das so verstanden dass der übergebene String alle Ziffern 1-9 enthalten muss (Reihenfolge und andere Inhalte wohl egal). Ich glaub fast da ist meine Version noch leichter als mit Regex. Aber ich lass mich gern eines besseren belehren, ich hab mit RegEx wie gesagt nich so die Übung.

PS: bzw. alle Buchstaben, irgendwie nimmt der TO mal ein Beispiel mit Ziffern, mal eins mit Buchstaben..

Mhm achso, ok dann habe ich es in der Tat falsch verstanden
/edit: Jepp stimmt
Naja, oder man machts eben doch mit ner Schleife selbst *g*

 

[hdi]

Aber damit prüfst du jetzt doch nur ob 9 Ziffern zwischen 1 und 9 vorkommen, aber nicht welche. Er will prüfen, ob der String tatsächlich jede der Ziffern von 1 bis 9 enthält. Deine Methode würd doch auch true liefern wenn der String "111222333" enthält. Oder nicht? Kann sein dass ich den replaceAll-Aufrufe falsch verstehe.

 

[eRaaaa]

Ja, ok, gebe mich um die Uhrzeit auch geschlagen, auch wenns da bestimmt was gibt

 

[Licht]

@eRaaaa: ja bin sehr neu in dieser Java-Materie. Bin grad dabei mich da durchzuschlagen. Viele Kleinigkeiten vergisst man schnell. Aber durch Fehler lernt man ja bekanntlich .
Und ja, es soll nachgeguckt werden ob z.B. alle Buchstaben enthalten sind.
Bsp.: "Franz jagt im komplett verwahrlosten Taxi quer durch Bayern" hier sind alle Buchstaben a bis z enthalten ob groß oder klein.

 

[hdi]

Bsp.: "Franz jagt im komplett verwahrlosten Taxi quer durch Bayern" hier sind alle Buchstaben a bis z enthalten ob groß oder klein.

Würde dann analog zu meinem Code mit den Ziffern so aussehen:

boolean alleBuchstabenVorhanden = true;
for(int i = 97; i<97+26; i++){ // siehe ASCII-Tabelle: 97 = a, 98 = b, ..., 
    if(!zuPruefenderString.contains((char)i)){
       alleBuchstabenVorhanden = false;
       break;
    }
}
if(alleBuchstabenVorhanden ){
  // ...
}

Um nicht zwischen Groß- und Kleinschreibung zu unterscheiden müsstest du vor der Schleife dann noch den zu prüfenden String auf lower-case only runtertrimmen:

zuPruefenderString = zuPruefenderString.toLowerCase();

ja mit Arrays kenne ich mich gar nicht aus und hätte Schwierigkeiten dies zu erklären

Hier ist jetzt kein Array im Spiel. Und den Code zu den Ziffern kannst du auch ohne Array lösen:

String zahlen = "123456789";
boolean alleZahlenVorhanden = true;
for(int i = 0; i<zahlen.length; i++){
    if(!zuPruefenderString.contains(zahlen.charAt(i)){
       alleZahlenVorhanden = false;
       break;
    }
}
if(alleZahlenVorhanden){
  // ...
}

PS: Natürlich könntest du dir auch überlegen, dich halt mal mit Arrays zu beschäftigen

 

[eRaaaa]

So, da ich eben mit meinem Code auf die Nase gefallen bin, muss ich jetzt nochmal Klugschei***
Man kann btw auch char in einer for Schleife direkt benutzen, dann muss man a) nicht die ASCII Tabelle auswendig können, b) somit keine magic numbers , c) kein extra cast notwendig !

Aber ein Regex oder anderer Einzeiler ist mir auch noch nicht eingefallen :noe:

 

[hdi]

Man kann btw auch char in einer for Schleife direkt benutzen, dann muss man a) nicht die ASCII Tabelle auswendig können, b) somit keine magic numbers , c) kein extra cast notwendig !

Wie kannst du gleich 3-fach über eine einzige Zeile Code meckern! Und damit auch noch völlig Recht haben

 

[Licht]

@hdi: bin grad am Bauklötze staunen... :applaus: die ASCII-Tabelle haben wir durchgenommen. Könnte ich also ruhigen Herzens versuchen zu erklären.
Dennoch habe ich eine Fehlermeldung von Netbeans (im // Kommentar):

String Wertb = "";

        boolean alleBuchstabenVorhanden = true;
        for(int i=97; i<97+26; i++) {
            if(!alleBuchstabenVorhanden.contains((char)i)){  //boolean cannot be dereferenced
             alleBuchstabenVorhanden = false;
            break;
            }
        }
if(Pangramm){
    Wertb = "alleBuchstabenVorhanden";
}else {
    Wertb = "NichtAlleBuchstabenSindVorhanden";
}

        JOptionPane.showMessageDialog(null, Wertb);

@eRaaaa: meinst du eine for schleife wäre dann einfacher zu erklären? ich kann mit der for schleife sowas anstellen:

public class Schleife
{
  public static void main(String[] args)
  {
    for(int i=0; i<10; i++)
      System.out.print(i);  
  }
}

ich weiß es ist eher ernüchternd. dennoch kann ichs :rtfm:
aber wie befehle ich, dass es nachguckt ob von mir bestimmte inhalte (buchstaben a bis z) vorhanden sind? wie ginge das denn für buchstaben?... ???:L

 

[hdi]

!alleBuchstabenVorhanden.contains

Das hast du falsch abgetippt: Da sollte !zuPruefenderString.contains stehen.

 

[eRaaaa]

Naja, eig. genau so (da jetzt eh schon viel zu viel Code da ist)

		String s = "Franz jagt im komplett verwahrlosten Taxi quer durch Bayern".toLowerCase();
		for (int i = 'a'; i <= 'z'; i++) {
			if(s.indexOf(i) == -1){
				System.out.println("Buchstabe "+(char)i+" ist nicht im String enthalten");
				break;
			}
		}

@hdi: contains geht trotzdem (ausser du machst daraus nen String z.B. ) nicht und length-> length()

 

[Licht]

@eRaaaa: oh klar das ist ja natürlich sehr einleuchtend.

bloß will ich kein printoutln benutzen...

public static void main(String[] args) {
String Entry = JOptionPane.showInputDialog("Bitte Eintrag machen").toLowerCase();

        for (int i = 'a'; i <= 'z'; i++) {
            if(Entry.indexOf(i) == -1){
JOptionPane.showMessageDialog(null,"Buchstabe "+(char)i+" ist nicht im String enthalten");
                break;
            }
        }

so gehts denke ich auch

 

[Gossi]

public static void main(String[] args) {
String Entry = JOptionPane.showInputDialog("Bitte Eintrag machen").toLowerCase();

        for (int i = 'a'; i <= 'z'; i++) {
            if(Entry.indexOf(i) == -1){
JOptionPane.showMessageDialog(null,"Buchstabe "+(char)i+" ist nicht im String enthalten");
                break;
            }
        }

Führ das mal aus und als Zeichenkette nimmste 133766642 oder so, dann bekommst du 26 Fenster die dir sagen, dass die Buchstaben von a - z fehlen....

public static void main(String[] args) {
String Entry = JOptionPane.showInputDialog("Bitte Eintrag machen").toLowerCase();
String fehlendeBuchstaben = "";
for (int i = 'a'; i <= 'z'; i++) {
        if(Entry.indexOf(i) == -1){
                fehlendeBuchstaben += (char) i + ",";
                break;
        }
}

if(fehlendeBuchstaben.length > 0 {
        fehlendeBuchstaben = fehlendeBuchstaben.substring(0, fehlendeBuchstaben.length()-1);
        JOptionPane.showMessageDialog(null,"Die Buchstaben "+ fehlendeBuchstaben +" sind nicht im String enthalten");
}

[edit]
Vergiss meinen Post, du hast ja nen Break drinne, stand aufm Schlauch xD
[/edit]

 

[Licht]

@Gossi: danke trotzdem

@alle: java-forum rockt! einfach nur hammer wie schnell hier einem etwas erklärt wird. und auch noch so, dass man es verstehen und anwenden kann.

 

[Licht]

folgendes Problem:

public static void main(String[] args) {
String Entry = JOptionPane.showInputDialog("Bitte Eintrag machen").toLowerCase();
 
      for (int i = 'a'; i <= 'z'; i++) {
            if(Entry.indexOf(i) == -1){
JOptionPane.showMessageDialog(null,"Buchstabe "+(char)i+" ist nicht im String enthalten");
                
            }else JOptionPane.showMessageDialog(null, "alle Buchstaben enthalten");
       break; 
        }

am Anfang klappt das ja gut aber nach einigen Run´s mit dem Eintrag "a" sagt er mir, dass alle Buchtsaben enthalten sind.
Also sobald ein Eintrag gemacht wird, der den Buchstaben a enthält, kommt eine falsche Lösung raus.
Wobei das in den ersten Durchgängen funktioniert....

 

[hdi]

Das else ist falsch. Da kommst du ja hin sobald der erste Buchstabe erfolgreich gefunden wurde. Das heißt aber nicht das ALLE drin sind.

Außerdem glaube ich vermischt du hier char und int. Bin mir nicht sicher ob das das Gewünschte tut, da der Wert je nach deklariertem Datentyp unterschiedlich interpretiert wird. Und ich weiß nicht genau was indexOf(int) tut. Mach aus dem "int i" in der for-Schleife lieber ein "char i".

 

[Licht]

nee.. gleich lösung mit char...

 

[hdi]

Was? Unabhängig von dem int vs char Ding ist die Logik deiner Schleife einfach falsch, das hast du mitbekommen oder?

 

[Licht]

nur versteh ich nicht was da falsch ist. erst klappts dann wieder nicht.

 

[hdi]

Wie schon gesagt:

Das else ist falsch. Da kommst du ja hin sobald der erste Buchstabe erfolgreich gefunden wurde. Das heißt aber nicht das ALLE drin sind.

nur versteh ich nicht was da falsch ist. erst klappts dann wieder nicht.

Es gibt nur zwei Fälle, in denen deine Logik korrekt zu sein scheint:
1) Der String enthält alle Buchstaben, und genau diese Information wird angezeigt
2) Der String enthält KEINEN Buchstaben.