reguläre Audrücke verbinden

[struppi79]

Hallo
schon wieder ich. Ich will einen String nach Regulären Ausdrücken filter, es dürfen nur Zahlen drinnen stehen, das habe ich gelöst mit

menge.matches("[^0-9])

nur leider stürzt mein Programm ab wenn der Benutzer die Zahl 1,2 eingibt .....

 

[tfa]

leider stürzt mein Programm ab

Is ja 'n Ding. Und ne Fehlermeldung kommt dabei nicht?

 

[XHelp]

1. Ist das von der Syntax falsch, da die Anführungszeichen nicht geschlossen werden
2. Damit prüfst du nicht, ob es aus Zahlen besteht, sondern nur ob es in dem String ein Zeichen steckt, welcher keine Ziffer ist.

 

[struppi79]

Entschuldigung
also: ja ich prüfe ob im String was anderes als eine Ziffer steht, dann gibts eine Fehlermeldung.
Da ich kurze Zeit später den String ein eine Int Variable umwandle sollte im String auch nur eine Zahl stehen. soweit werden alle Nicht Zahlen abgefangen, nur leider klappt das nicht wenn der User 1,2 oder 1.2 eingibt :-(
Besser?

 

[tfa]

leider klappt das nicht

Was heißt "es klappt nicht"?

 

[XHelp]

match überprüft den ganzen String. D.h. du prüfst ob dein String aus einem "nicht-Ziffer-Zeichen" besteht.
Und darüber hinaus: nehmen wir mal "1.2" als Beispiel. Welche Arten von Zeichen erkennst du da? 2 Ziffern und ein Punkt.
Edit: aber tfa hat recht: formuliere erstmal eine konkrete Fehlerbeschreibung.

 

[struppi79]

mei seid Ihr wieder Konstruktiv,
ist doch logisch, wenn er ne Eingabe von 1,2 nicht rausfiltert kann ich 1,2 auch nicht in eine Int Variable umwandeln, ...
Nur Was muss ich in die Regex schreiben dass auch 1,2 und 1.2 herausgefiltert werden ??

java.lang.NumberFormatException: For input string: "1,2"

 

[XHelp]

Ist auch logisch, dass wenn du prüfen willst, ob in einem String nur Ziffern vorhanden sind, dass du dann sowas wie

matches("\\d+")

nimmst. Aber wir sollen mitdenken und erraten was du meinst und die dein Code aussieht? :bahnhof:

 

[struppi79]

ok sorry
so schaut die betreffende Stelle aus.

for (int i=0; i<artikel2.length; i++){  
  if (menge[i].matches("[^0-9]")){
  out.print ("<h2>Sie haben ein ung&uuml;ltiges Zeichen eingegeben. <br/>Bitte korrigiern Sie die Eingabe</h2>");
  }else{
		for (int j=0; j<artikel2.length; j++){
			int test = Integer.parseInt(menge[j]);

 

[tfa]

Aha, dein Programm stürzt gar nicht ab, sondern es gibt eine NumberFormatException, sag das doch!
Ich glaube auch, deine Regex ist falsch. Du musst erstmal herausfinden, wie die für deine Zwecke aussehen muss.

		String s = "1,2";
		
		if (s.matches("[^0-9]")) {
			System.out.println("ja");
		}
		else {
			System.out.println("nein");
		}

 

[XHelp]

Du prüfst den i-ten String in einer Schleife, und wenn es gültig ist, dann wandest du in einer inneren Schleife alle String um? hÄ?

for (int i=0; i<artikel2.length; i++){  
  if (menge[i].matches("[^0-9]")){
    out.print ("<h2>Sie haben ein ung&uuml;ltiges Zeichen eingegeben. <br/>Bitte korrigiern Sie die Eingabe</h2>");
  }else{
    //mach was nur mit dem Zeichen, welches du überprüft hast
    int test = Integer.parseInt(menge[i]);

So macht das mehr Sinn.
Und warum eigentlich eine Schleife über artikel2-Länge, aber Arbeit mit menge-Array? :bahnhof:

 

[struppi79]

ja weil sonst , sollte jemand eine nicht ziffer eingeben
die Fehlermeldung sooft dasteht wie die schleife durchlaufen wird deswegen der ungünstige Aufbau

aber egal, ich wollte ja nur wissen wie ich die regex so aufbaue dass sie nur zahlen durchlässt
danke trotzdem

 

[tfa]

s.matches("\\d+")

Siehe Beitrag #8.

 

[XHelp]

Das ist aber falsch was du machst unabhängig davon was du willst. Wenn du es nur einmalig machen willst, dann mach sowas wie:

boolean isValid = true;
for (String currentAmount: menge) {
  if (!currentAmount.maches("\\d+")) {
    isValid = false;
    break;
  }
}
if (isValid) {
  //mach was immer du willst
} else  {
  //zeige die Fehlermeldung an
}

Oder du machst eine ähnliche geschichte mit dem boolean-Flag, nur dass du direkt die korrekten Werte verarbeitest:

boolean isValid = true;
for (String currentAmount: menge) {
  if (currentAmount.maches("\\d+")) {
    //mach was damit
  } else {
    isValid = false;
  }
}
if (!isValid) {
  //zeige die Fehlermeldung an
}

 

[struppi79]

Hallo
funktioniert leider nicht ....
das fängt noch weniger nicht Zahlen ab als [^0-9]

 

[XHelp]

Doch tut es. Aber da wären wir wieder bei: was genau willst du denn haben?
Und du hast hoffentlich bedacht, dass

\\d+

Zahlen sucht, wo hingegen

[^0-9]

auf eine komische Art und Weise versucht Fehler zu suchen.

 

[struppi79]

hm ok
ich würde es jetzt so machen, nur spinnt der tomcat jetzt wegen meiner Klammerung :-(

for (int i=0; i<artikel2.length; i++){
	if (menge[i].matches("\\d+")){
	int test = Integer.parseInt(menge[i]);
		if (test>0 && test<151){ 
		String bestellung = station2 +"  " +test  +" x " +artikel2[i] +"\r\n"; 
		out.print("<tr> <td>" +station2 +"</td><td>" +test  +"</td><td>" +artikel2[i] +"</td></tr>");
			
		final String DATEI="/var/www/intern/jspfiles/Bestellung_Aigner.txt";   //in csv-Datei schreiben
     
		FileWriter fw=new FileWriter(DATEI,true);
		BufferedWriter bw=new BufferedWriter(fw);
     
		bw.write(bestellung);
		bw.newLine();
     
		bw.close();
		fw.close();}
			else{			
			out.print("<h3>Bitte korrigieren Sie die Bestellmenge!</h3>");}
				else{ 
				out.print ("<h2>Sie haben ein ung&uuml;ltiges Zeichen eingegeben. <br/>Bitte korrigiern Sie die Eingabe</h2>");
				out.print ("<h2>Hier haben Sie ein ungültiges Zeichen eingegeben:"+station2+"  "+artikel2+"  "+menge);}
		}
	}
}

 

[XHelp]

Das hat rein gar nichts mit tomcat zu tun. Du hast schlicht eine geschweifte Klammer zu viel. Wo diese zu viel ist erkennst du, wenn du deinen Code sinnvoll einrückst und auch die schließenden Klammer in eine extra Zeile machst

 

[struppi79]

Hallo danke
ich habe den fehler gestern schon gefunden und behoben,
ja Einrückung .... da sollte ich mal daran arbeiten .
Danke und grüße