Problem mit regex(-anwendung)

[StrikeTom]

Hallo Leute,
ich programmiere gerade einen Parser und nun möchte ich sachen in formeln wie:
(3)
3+(45)
ersetzen durch:
3
3+45

das versuchee ich mit regex, der ausdruck selbst funktioniert doch scheint da
noch ein fehler im ersetzen von den strings zu sein.
hier der code:

public void replaceNotUsingBreakets(String formel){
		Pattern p = Pattern.compile(".*[\\(]([0-9]+)[\\)].*");
        Matcher m = p.matcher(formel);
        while(m.find()){
            formel = formel.replace("(" + m.group() + ")", m.group()) ;
        }
	}

Danke

 

[eRaaaa]

group() = gesamte Ausdruck , group(1) = deine Gruppe

 

[StrikeTom]

Danke für die schnelle Antwort, aber mit

public void replaceNotUsingBreakets(String formel){
		Pattern p = Pattern.compile(".*[\\(]([0-9]+)[\\)].*");
        Matcher m = p.matcher(formel);
        while(m.find()){
            formel = formel.replace("(" + m.group(1) + ")", m.group(1)) ;
        }
	}

funktioniert es irgendwie auch nicht

 

[eRaaaa]

Dass du Strings nicht ändern kannst ist dir klar oder? Und so sowieso nicht...d.h. formel sollte dann nur in der Methode = 3+45 sein. Du musst also schon den String zurückgeben

mal ne andere Frage: wieso benutzt du nicht einfach replace/replaceAll ?

 

[Gast2]

Und warum willst Klammern entfernen?
Es ist doch zum Parsen viel einfacher wenn alles bereits geklammert ist.

 

[StrikeTom]

Dass du Strings nicht ändern kannst ist dir klar oder?

Ach du meine Güte, wie konnte ich das übersehen, Danke

mal ne andere Frage: wieso benutzt du nicht einfach replace/replaceAll ?

Was ist den der Unterschied zwischen replace und replaceall?

Und warum willst Klammern entfernen?
Es ist doch zum Parsen viel einfacher wenn alles bereits geklammert ist.

Weil meine methode parse eine Doublezahl zurückgibt, und wenn die zahl dann eingeklammert ist, dann ist da doch wohl was falsch. String zu double, da darf der String auch nur die zeichen enthalten, die eine Double auch enthalten darf.

Jetzt sieht mein code so aus, es funktioniert trotzdem nicht:

public String replaceNotUsingBreakets(String formel){
String a = null;
        Pattern p = Pattern.compile(".*[\\(]([0-9]+)[\\)].*");
        Matcher m = p.matcher(formel);
        while(m.find()){
            a = formel.replace("(" + m.group(1) + ")", m.group(1)) ;
        }
return a;
    }

Danke

 

[eRaaaa]

Was ist den der Unterschied zwischen replace und replaceall?

replaceAll erwartet ein Regex.

Vllt. solltest du mal genauer werden. Was heißt denn: "funktioniert trotzdem nicht"?

	public static void main(String[] args) throws Exception {
		System.out.println(replaceNotUsingBreakets("3+(45)"));
	}

liefert bei mir :

3+45

Meinst du, dass so etwas wie "3+(45)+(343)" nicht geht?
Probiere vllt:

	public String replaceNotUsingBreakets(String formel) {
		Pattern p = Pattern.compile("\\((.+?)\\)");
		Matcher m = p.matcher(formel);
		while (m.find()) {
			formel = formel.replace("(" + m.group(1) + ")", m.group(1));
		}
		return formel;
	}

wobei das irgendwie sehr schräg ist wie ich finde

Oder eben : [c]System.out.println("3+(45)".replaceAll("[()]",""));[/c] --> 3+45

Oder verstehe ich dich vllt grundsätzlich falsch? ;/

 

[Gast2]

Weil meine methode parse eine Doublezahl zurückgibt, und wenn die zahl dann eingeklammert ist, dann ist da doch wohl was falsch.

Ich weiß natürlich nicht was dein Parser genau macht oder machen soll.
Aber von einer Methode parse, die z.b. "(1.23)" übergeben bekommt, erwarte ich dann als Rückgabe 1.23. Oder bei "1+((1.23)*(2.3))" halt 3.829. Sind doch alles gültige Mathematische Ausdrücke.

 

[StrikeTom]

Vllt. solltest du mal genauer werden. Was heißt denn: "funktioniert trotzdem nicht"?

Tschuldingung
Also es funktioniert nicht heißt:
Ich habe folgenden code:

formel = formel.replace(lr[0] + e.getKey() + lr[1], calculate(lr[0] + e.getKey() + lr[1]).toString());
formel = replaceNotUsingBreakets(formel);

die erste zeile rechnet den einen teil der formel aus(bisher erst zwei zahlen mit einem operator dazwischen)funktioniert, denn er schafft mir diese exception, wenn ich z.B. 12-4 ausrechnen will:

Exception in thread "main" java.lang.NumberFormatException: For input string: "(8.0)"
	at sun.misc.FloatingDecimal.readJavaFormatString(Unknown Source)
	at java.lang.Double.parseDouble(Unknown Source)
	at javac.math.Parser.parse(Parser.java:261)
	at TestMain.Main.main(Main.java:11)

so, dafür möchte ich diese klammern wegschaffen. die metho die das machen soll rufe ich bei jedem "parsevorgang" neu auf.
Die zweite zeile ist ja klar was sie machen soll
jetzt kommt folgende exception:

Exception in thread "main" java.lang.NullPointerException
	at sun.misc.FloatingDecimal.readJavaFormatString(Unknown Source)
	at java.lang.Double.parseDouble(Unknown Source)
	at javac.math.Parser.parse(Parser.java:261)
	at TestMain.Main.main(Main.java:11)

wobei zeile 261 die return zeile ist:

		return Double.parseDouble(formel);

Danke

 

[eRaaaa]

Hast du mal gedebuggt? Hört sich für mich so an als wenn m.find() nichts findet, und du eben dann dein

String a = null;

zurück gibst
Aber irgendwie ist das Ganze für mich immer noch nicht ganz klar, was jetzt deine extra Methode bewirken soll

 

[StrikeTom]

Ich habs:
m.find()
findet nichts, weil mein string 13.0.
Und der ausdruck nur zahlen ohne komma findet.
Der regex muss so aussehen:

.*[\\(]([0-9]+\\.[0-9]+)[\\)].*

Danke