Pattern liefert false

[Raidri]

Hallo,

Ich weiss nicht warum mein Pattern immer false liefert. Ich übergebe folgenden Wert 00:30
und hier ist der code wo ich es kontrolliere.

public static boolean isValidBudget(String budget){
		Pattern p = Pattern.compile("[0-9]{1,}[:.,]?[0-9]?");
		Matcher m = p.matcher(budget);
		
		return m.matches();
	}

Habe ich irgendwas falsch im Pattern ? : und die Zahl dahinter soll Optinal sein.

Danke

 

[nrg]

"[0-9]{2}:[0-9]*"

so vielleicht?

edit: oder wenn die hintere Zahl optional zwischen leer und 2 Zahlen sein soll, vllt so:

"[0-9]{2}:([0-9]{2})?"

 

[SlaterB]

beschreibe doch in Worten, was die offensichtlich drei Bestandteile des Patterns je für sich leisten sollen,
dann wird eigentlich klar, was davon ganz einfach noch nicht stimmt

 

[Raidri]

dann bin ich aber gezwungen einen : zu nehmen oder nicht ? und der "." bzw das "," müssen auch optional möglich sein!

Also es müssen eingabe wie 20, 2000 oder 00:30 oder 20:20 oder 200.50 oder 200,50 möglich sein, alles geht bis auf 00:30. wo halt 2 nullen am Anfang stehen.

 

[SlaterB]

> alles geht bis auf 00:30. wo halt 2 nullen am Anfang stehen.

das stimmt ja mal gar nicht, zumindest für das Pattern deines ersten Postings

public class Test
{
    public static void main(String[] args)
    {
        System.out.println(isValidBudget("20:20"));
        System.out.println(isValidBudget("00:30"));
        System.out.println(isValidBudget("00:3"));
    }

    public static boolean isValidBudget(String budget)
    {
        Pattern p = Pattern.compile("[0-9]{1,}[:.,]?[0-9]?");
        Matcher m = p.matcher(budget);

        return m.matches();
    }
}

false
false
true

 

[faetzminator]

Ich behaupte, dass es daran liegt, dass hinter dem [c]:[/c] zwei Ziffern angegeben werden, du allerdings [c]?[/c] (0-1) verwendest.

Edit: zu spät

 

[nrg]

Habe ich irgendwas falsch im Pattern ? : und die Zahl dahinter soll Optinal sein.

hier hast du noch was anderes beschrieben...

[0-9]*[:.,]?[0-9]*

 

[faetzminator]

Ich wär für [c][0-9]+[:.,]?[0-9]*[/c] - btw, muss man in Gruppierungen den Punkt nicht escapen? Mach ich irgendwie aus Gewohnheit...

Edit: muss man anscheinend nicht. Aber für alle, welche nun sagen, dies würde keinen Sinn machen: doch, ein [c].[/c] matcht nämlich nicht zwingend alle Zeichen - als Beispiel von [c]\s[/c].

 

[nrg]

ja stimmt, das + ist an der stelle besser. mein fehler

 

[Raidri]

Ja sry, du hast recht die gingen echt nicht.

Aber die Lösung war tatsächlich das "?" am Ende daraus ein "*" zu machen.

Herzlichen dank

lg
Raidri