Eingabe wird immer als "false" ausgegeben

[NicoRyl]

Hallo zusammen.

In meinem Programm möchte ich einen kleinen SQL-Validator mit Hilfe von Regex erstellen. Der Regex stimmt so wie er ist und funktioniert auch wie er es soll, das habe ich schon getestet. Das Problem ist, wenn ich die Main-Methode starte und einen SQL-Befehl (z.B.: SELECT * FROM tabelle1; ) eingebe, kommt immer die Antwort, dass es falsch ist, egal was ich eingebe. Ich suche seit Stunden nach dem Fehler, finde ihn aber nicht. Kann mir jemand bitte helfen?
Vielen Dank schonmal im Voraus!

Liebe Grüße,

Nico.


Anbei noch der Code:

 

[kneitzel]

Ich würde Dich bitten, Code nicht als Anhang sondern über die Code Funktion (</> Icon über der Eingabe) zu posten.

Dann wäre bei sowas mein Tipp: Gehe es doch einmal im Debugger durch oder gib Zwischenergebnisse aus. Also was gibt die Eingabe Methode von Dir zurück?

Scanner ist eine Token-Basierte Eingabe und next() liefert nur das nächste Token zurück. Daher gibst Du zwar
SELECT * FROM ....
ein, aber aus dem Eingabestream liest er nur das "SELECT" und gibt es zurück. Und SELECT alleine ist natürlich kein Match.

 

[NicoRyl]

Vielen Dank für deine Antwort. Wegen dem Anhang: das wusste ich nicht, bin neu auf der Seite. Ich werde es in Zukunft so machen. Gibt es denn anstatt dem next() eine Alternative?

 

[NicoRyl]

Ich würde Dich bitten, Code nicht als Anhang sondern über die Code Funktion (</> Icon über der Eingabe) zu posten.

Dann wäre bei sowas mein Tipp: Gehe es doch einmal im Debugger durch oder gib Zwischenergebnisse aus. Also was gibt die Eingabe Methode von Dir zurück?

Scanner ist eine Token-Basierte Eingabe und next() liefert nur das nächste Token zurück. Daher gibst Du zwar
SELECT * FROM ....
ein, aber aus dem Eingabestream liest er nur das "SELECT" und gibt es zurück. Und SELECT alleine ist natürlich kein Match.

Ich hab´s! Mit nextLine() hat es funktioniert. Vielen Dank nochmal für den Tipp!

 

[NicoRyl]

Ich würde Dich bitten, Code nicht als Anhang sondern über die Code Funktion (</> Icon über der Eingabe) zu posten.

Dann wäre bei sowas mein Tipp: Gehe es doch einmal im Debugger durch oder gib Zwischenergebnisse aus. Also was gibt die Eingabe Methode von Dir zurück?

Scanner ist eine Token-Basierte Eingabe und next() liefert nur das nächste Token zurück. Daher gibst Du zwar
SELECT * FROM ....
ein, aber aus dem Eingabestream liest er nur das "SELECT" und gibt es zurück. Und SELECT alleine ist natürlich kein Match.

Noch eine Frage: Wenn ich jetzt noch eine optionale "WHERE" - Anweisung hinten dran hängen möchte, wie muss ich dann den Regex erweitern? Ich habe bis jetzt folgendes: "^SELECT\s+(\*?[A-Za-z0-9,\s]*)\s+FROM\s+[A-Za-z0-9]+(\s+WHERE\s+[A-Za-z0-9]=[A-Za-z0-9])?;$"
Das scheint aber nicht zu funktionieren.

 

[kneitzel]

Erst einmal schön, dass Du mit dem Hinweis eine Lösung gefunden hast. Hier evtl. noch ein wichtiger Ratschlag: Wenn Du ein next, nextInt oder sonst eine Token basierte Methode nutzt, dann wird das Trennzeichen nicht mitgenommen und bleibt noch im Stream. Das ist wichtig und man will da ggf. nach einem nextInt noch ein nextLine aufrufen um die Eingabe etwas zu bereinigen... Das nur als zusätzlichen Hinweis. Die gefundene Lösung ist so richtig.

Zur neuen Thematik:
Bisher hast Du nur Bedingungen mit 1 Zeichen = 1 Zeichen. Also sowas wie a=1 oder so. Wenn da mehr sein dürfen, dann musst Du da jeweils ein + hinter die ] setzen.
^SELECT\s+(\*?[A-Za-z0-9,\s]*)\s+FROM\s+[A-Za-z0-9]+(\s+WHERE\s+[A-Za-z0-9]+=[A-Za-z0-9]+)?;$

Das erlaubt schon etwas mehr. Aber das ist halt alles noch sehr grob. Quote Zeichen für Strings sind noch nicht erlaubt um ein Beispiel zu nennen.

 

[NicoRyl]

Erst einmal schön, dass Du mit dem Hinweis eine Lösung gefunden hast. Hier evtl. noch ein wichtiger Ratschlag: Wenn Du ein next, nextInt oder sonst eine Token basierte Methode nutzt, dann wird das Trennzeichen nicht mitgenommen und bleibt noch im Stream. Das ist wichtig und man will da ggf. nach einem nextInt noch ein nextLine aufrufen um die Eingabe etwas zu bereinigen... Das nur als zusätzlichen Hinweis. Die gefundene Lösung ist so richtig.

Zur neuen Thematik:
Bisher hast Du nur Bedingungen mit 1 Zeichen = 1 Zeichen. Also sowas wie a=1 oder so. Wenn da mehr sein dürfen, dann musst Du da jeweils ein + hinter die ] setzen.
^SELECT\s+(\*?[A-Za-z0-9,\s]*)\s+FROM\s+[A-Za-z0-9]+(\s+WHERE\s+[A-Za-z0-9]+=[A-Za-z0-9]+)?;$

Das erlaubt schon etwas mehr. Aber das ist halt alles noch sehr grob. Quote Zeichen für Strings sind noch nicht erlaubt um ein Beispiel zu nennen.

Alles klar. Vielen Dank für die Tipps!