[PHP] eregi durch preg_match ersetzen

[tobias1985]

Hi,
ich steh zurzeit wohl auf dem Schlauch. Ich nutze in einem Script zurzeit noch folgendes:

if(eregi("Opera[ |/]{0,1}([1-9])", $user_agent, $regs)) return 'Opera '.$regs[1];

Da preg_match() schneller wäre, dachte ich, ich ersetze die Funktion mal durch:

if(preg_match("Opera[ |/]{0,1}([1-9])", $user_agent, $regs)) return 'Opera '.$regs[1];

Laut php.net nutzt preg_match auch normale reguläre Ausdrücke. Also sollte doch das einfache ersetzen der Funktion schon ausreichen?
Ich bekomm dann aber folgende Fehlermeldung:

Warning: preg_match() [function.preg-match]: Delimiter must not be alphanumeric or backslash in /pfad/datei.php on line 92

Ich habs leider nicht hinbekommen und keine Lösung gefunden. Entweder hab ich irgendwas nicht verstanden oder ich steh gerade voll auf dem Schlauch. Hoffe ihr könnt mir helfen.

Danke schonmal.

Gruß
Tobias

 

[jabba]

Moin,

spontan würde ich sagen, es fehlen die "Begrenzungszeichen" für den Regulärenausdruck.

versuchs mal z.B. so:

if(preg_match("~Opera[ |/]{0,1}([1-9])~", $user_agent, $regs)) return 'Opera '.$regs[1];

Vor und hinter deinem regulären Ausdruck noch ein Begrenzungszeichen (das kannst du "fast" frei wählen).

 

[theHacker]

Die Mitte stimmt wahrscheinlich auch ned. In der Charaktergruppe gelten alle Zeichen, also auch der vertikale Strich. Ich nehm ned an, dass du den erlauben willst.
Statt {0,1} kannst du auch einfach ? schrieben.

Ich würde dann auf

[FONT=Courier New]~Opera[ /]?([1-9])~[/FONT]

kommen.

 

[tobias1985]

versuchs mal z.B. so:

if(preg_match("~Opera[ |/]{0,1}([1-9])~", $user_agent, $regs)) return 'Opera '.$regs[1];

Danke damit gehts. Lag also nur am Begrenzungszeichen...