Warum läst mein replace die Klammern drin?

[Thallius]

Hi

ich habe einen String im Format

(0008,0002)

ich möchte nun nur die beiden Hex-Zahlen haben. Das probiere ich so:

            // first row contains tag in format (0008,0001)
            String[] tags = tagBuffer.get(row).split(",");
            if(tags.length != 2)
            {
                System.out.println("First row of tag is mail formatted.");
                return null;
            }
            // remove all non hex digits
            String groupString = tags[0].replace("/[^A-Fa-f0-9x]/g", "");
            String tagString = tags[1].replace("/[^A-Fa-f0-9x]/g", "");

            System.out.println(groupString+"-"+tagString);

Die Ausgabe ist aber
(0008-0002)

Die Klammern werden also nicht entfernt. Wenn ich den Regex auf einem Tester im Web laufen lasse, werden dieKlammern aber entfernt. Macht ja auch Sinn, da sie nicht in meinem definierten Charset enthalten sind.

Was hat Java hier wieder für eine Besonderheit?

Gruß

Claus

 

[mrBrown]

Du nimmst die falsche Methode, für Regex nutzt man replaceAll oder replaceFirst, replace ersetzt alle "echten" Vorkommen von "/[^A-Fa-f0-9x]/g"

Was sollen denn "/" und "/g"?

 

[Thallius]

Sorry kenne das nur von PHP und da kann ich mit /ausdruck/g einen match all definieren.

DAs man in Java da nicht mit arbeiten kann wußte ich nicht. Mit replaceAll funktioniert es dann auch.

Danke

Claus

 

[Tobse]

Wenn du schon Regex benutzt, kannst du doch einfach die beiden Zahlen in je eine Gruppe matchen und mit einem javax.pattern.Matcher extrahieren...

 

[dzim]

die Slashes und so, sind glaube ich RegExp in Perl-Stil.
Am Besten mal in Java anschauen: https://docs.oracle.com/javase/8/docs/api/java/util/regex/Pattern.html
Hier teste ich immer: http://www.regexplanet.com/advanced/java/index.html

Um einen Match herzustellen, könnte man folgenden RegExp nutzen: \(([0-9a-fA-F]{2,}),([0-9a-fA-F]{2,})\)
dann Pattern#compile("<regexp>") -> pattern#matcher("<string-to-test>") -> und auf dem Matcher die Gruppen (0 ist der Match, also musst du Gruppe 1 und 2 holen) auswerten.

Oder einfacher einen Matcher mit dem Pattern ([0-9a-fA-F]{2,}) erstellen und dann mit einem while-loop über die #finds iterieren - korrekt wäre dann wieder Gruppe 1 auszulesen, aber in dem Fall ist nur eine Gruppe drin, also ginge auch Gruppe 0.

 

[Tobse]

Oder einfacher einen Matcher mit dem Pattern ([0-9a-fA-F]{2,}) erstellen und dann mit einem while-loop über die #finds iterieren - korrekt wäre dann wieder Gruppe 1 auszulesen, aber in dem Fall ist nur eine Gruppe drin, also ginge auch Gruppe 0.

Das wird nicht funktionieren, weil der Bindestrich zwischendrin ist.

 

[dzim]

Gib den String Mal auf der Test-Webseite ein. Und den Text von @Thallius . Hab das schon getestet, bevor ich es gepostet hab.

 

[Thallius]

Da es immer nur zwei Zahlen sind ist der Matcher wohl ein wenig mit Kanonen auf Spatzen geschossen. Eigentlich ist es das Regex ja schon. Ich könnte genauso gut mit substr() arbeiten da die Zahlen auch immer 4 Digits lang sind.

Gruß

Claus

 

[Viktim]

So würds auch gehen

    String groupString = tags[0].replace("/[^A-Fa-f0-9x]/g", "");
    groupString = entferneKlammern(groupString);

    String tagString = tags[1].replace("/[^A-Fa-f0-9x]/g", "");
    tagString = entferneKlammern(tagString);

private String entferneKlammern(String string) {
    string = string.replaceAll("\\(", "");
    string = string.replaceAll("\\)", "");
    return string;
  }

Ist aber nicht so schön, weil du ja schon deinen Regex hast
an sonsten müsstest du das mit Regex-Gruppen machen

 

[dzim]

Ah... Ich hab es falsch rum verstanden...
Dann nimm einfach diesen RegExp [0-9a-fA-F,]+ und Gruppe 0.
Oder nutze dein String#replace und füttere es mit dem RegExp [()]

String s = "(0008,0001)".replace("[()]", "");
// s sollte nun "0008,0001" sein

#edit: das letzte ist allerdings ungetested

 

[mrBrown]

Ah... Ich hab es falsch rum verstanden...
Dann nimm einfach diesen RegExp [0-9a-fA-F,]+ und Gruppe 0.
Oder nutze dein String#replace und füttere es mit dem RegExp [()]

String s = "(0008,0001)".replace("[()]", "");
// s sollte nun "0008,0001" sein

#edit: das letzte ist allerdings ungetested

replaceAll wäre richtig, um RegEx zu nutzen