[PHP] Regexp parameters uit string.

[Optix]

Hi,

Ik heb de volgende string: "test[did=regexp,work=false]".
Dan wil ik mbv een regexp met de volgende array eindigen:

PHP:

    array('did' => 'regexp', 'work' => 'false');

Ben (nog steeds) lerende regexp, maar dit is wat ik nu heb:

PHP:

    $subject = "test[did=regexp,work=false]";       
    $result = preg_match("/\[(.*)=(.*)\]/", $subject, $matches);        
    print_r($matches);

Wat het volgende uitpoept:

code:

array
  0 => '[did=regexp,work=false]'
  1 => 'did=regexp,work'
  2 => 'false'

Ik moet zoiezo nog iets met die komma doen maar ik kom er niet helemaal uit.

[NMe]

PHP:

$result = preg_match_all("/\[([^=]*)=([^,\]]*)\]/iU", $subject, $matches);

edit:

Scratch that, dat werkt niet. Ik zou dit ook niet met regexps doen, gewoon alles splitten op komma's en daarbinnen alles splitten op = werkt ook prima en is waarschijnlijk zelfs sneller.

[ Voor 54% gewijzigd door NMe op 14-10-2008 19:21 ]

[Optix]

-NMe- schreef op dinsdag 14 oktober 2008 @ 19:20:

PHP:

$result = preg_match_all("/\[([^=]*)=([^,\]]*)\]/iU", $subject, $matches);

Resulteert:

code:

array
  0 => 
    array
  1 => 
    array
  2 => 
    array

[NMe]

Zie mijn edit.

[Optix]

-NMe- schreef op dinsdag 14 oktober 2008 @ 19:21:
Zie mijn edit.

Ha, mja zo heb ik het dus ook opgelost, maar vind toch iets netter als het met 1 regexp KAN.

[NMe]

Het kan wel, maar het is, denk ik, trager en bovendien een stuk minder leesbaar.

PHP:

$result = preg_match_all("/\[(([^=]*)=([^,\]]*),?)\]*/iU", $subject, $matches);

Die werkt wel, denk ik. Maar nogmaals, het zou niet mijn voorkeur hebben.

[Optix]

-NMe- schreef op dinsdag 14 oktober 2008 @ 19:29:
Het kan wel, maar het is, denk ik, trager en bovendien een stuk minder leesbaar.

PHP:

$result = preg_match_all("/\[(([^=]*)=([^,\]]*),?)\]*/iU", $subject, $matches);

Die werkt wel, denk ik. Maar nogmaals, het zou niet mijn voorkeur hebben.

Helaas, deze werkt ook niet

code:

array
  0 => 
    array
      0 => '[did='
  1 => 
    array
      0 => 'did='
  2 => 
    array
      0 => 'did'
  3 => 
    array
      0 => ''

[Optix]

Overigens, mijn huidige oplossing

PHP:

    $options = array();
    $result = preg_match("/\[(.*)\]/", $subject, $matches);
                            
    foreach (explode(',', $matches[1]) as $option) 
    {
        $option = explode('=', $option);
        $options[$option[0]] = $option[1];
    }

is 0.4 microseconde langzamer

[pedorus]

Je hebt het probleem niet helemaal exact gedefinieerd, dus wat je precies voor eisen stelt weet ik niet.
Het volgende werkt met maar 1 regexp:

PHP:

$subject = 'test[did=regexp,work=false]';
preg_match_all('/([^[,=\]]*)=([^[,=\]]*)/', $subject, $matches);
$options = array_combine($matches[1],$matches[2]);
print_r($options);

Maar checkt gewoon de syntax niet...
Anders worden het toch al snel meerdere regexpen, of een andere oplossing.

[Optix]

Klopt, ik had mijn eisen dan ook net wat aangepast mbt de sepperator zoals je ziet

PHP:

if (preg_match("/\[(.*)\]/", $subject, $matches)) 
{
    parse_str($matches[1], $options);
}

[Verwijderd]

Optix schreef op dinsdag 14 oktober 2008 @ 19:23:
Ha, mja zo heb ik het dus ook opgelost, maar vind toch iets netter als het met 1 regexp KAN.

Tja, m.i. kunnen "netjes" en "regexp" in 1 regel alleen maar als er een ontkenning tussen staat.
Tikkie gechargeerd natuurlijk, maar vaak wanneer een string geparsed moet worden zie je allerlei ingewikkelde, onleesbare 'bend over backwards' regexen, terwijl meestal een simpel parsertje (al dan niet stack based) 't ook prima doet en vaak beter performt.
Voordeel van zo'n parsertje is dat jij en jouw collega's na een paar maand nog direct kunnen zien wat 't ding doet, terwijl je bij een uitgebreide regex je al na een week afvraagt wat 'ie doet en waarom. (goed commenten helpt dan wel, maar je comment heeft dan waarschijnlijk evenveel regels als die parser...)

Regexps gebruik ik niet of nauwelijks in m'n code, maar wel veelvuldig in grep, awk, sed etc. scripts, en daar is 't hardstikke handig. Maar (alweer m.i.) is in een applicatie een regex die niet snel te vatten is niet handig. Een parsertje is dan prettiger, ook voor je collega's.

"you are not supposed to understand this" (uit Janoz' signoff geloof ik) is leuk voor 't ego van de maker, maar staat onderhoud van de applicatie gigantisch in de weg.

[pedorus]

pedorus schreef op dinsdag 14 oktober 2008 @ 20:00:
Anders worden het toch al snel meerdere regexpen, of een andere oplossing.

Wat een onzin, het kan prima met een:

PHP:

$subject = "test[did=regexp,work=false]";
if (preg_match_all('/(?:^[^[]*\[|(?<!^)\G,)([^=]*)=([^,]*)(?=(?:,[^=]*=[^,]*)*]$)/', 
                   $subject, $matches)) {
    $options = array_combine($matches[1],$matches[2]);
    print_r($options);
}

Comments zijn ook niet nodig, want iedereen snapt de details toch?

Some people, when confronted with a problem, think
“I know, I'll use regular expressions.” Now they have two problems.

(source)

[Verwijderd]

dit is trouwens een goed boek over regular expressions

[Verwijderd]

pedorus, I rest my case.
Dank voor je hilarische voorbeeld.

[chris]

Het is nog iets simpeler te doen volgens mij:

1. letterlijke "[": \[
2. Vervolgens komt er een "key=value": ([^=]*)=([^,]*)
3. Daarna komt een lijst van ",key=value": (,([^=]*)=([^,]*))*
4. En vervolgens een vierkante haak sluiten: \]

Dus de volgende preg-expression

code:

/\[([^=]*)=([^,]*)(,([^=]*)=([^,]*))*\]/

Je moet alleen het derde element uit de array gooien:

Parenthesized substring matches:

    * substring 1: did
    * substring 2: regexp
    * substring 3: ,work=false
    * substring 4: work
    * substring 5: false

Maar eigenlijk is het natuurlijk gewoon veel mooier met een "normale" functie zoals hierboven.

[OxiMoron]

chris schreef op donderdag 16 oktober 2008 @ 01:16:
Het is nog iets simpeler te doen volgens mij:

1. letterlijke "[": \[
2. Vervolgens komt er een "key=value": ([^=]*)=([^,]*)
3. Daarna komt een lijst van ",key=value": (,([^=]*)=([^,]*))*
4. En vervolgens een vierkante haak sluiten: \]

Dus de volgende preg-expression

code:

/\[([^=]*)=([^,]*)(,([^=]*)=([^,]*))*\]/

Je moet alleen het derde element uit de array gooien:

Parenthesized substring matches:

    * substring 1: did
    * substring 2: regexp
    * substring 3: ,work=false
    * substring 4: work
    * substring 5: false

Maar eigenlijk is het natuurlijk gewoon veel mooier met een "normale" functie zoals hierboven.

code:

/\[([^=]*)=([^,]*)(?:,([^=]*)=([^,]*))*\]/

Toegevoegd: ?: Als je dat na een open haakje zet worden deze non-capturing.

[pedorus]

chris schreef op donderdag 16 oktober 2008 @ 01:16:
Het is nog iets simpeler te doen volgens mij:

1. letterlijke "[": \[

Je hebt gelijk, bij mij kan "^[^[]*" weg, omdat we alles toestaan nu. Ik kan me enkel voorstellen dat je later dingen als "foutbegin[,,][a=b]" wil uitsluiten, door komma's in de keys niet toe te staan.

Dus de volgende preg-expression

code:

/\[([^=]*)=([^,]*)(,([^=]*)=([^,]*))*\]/

Je moet alleen het derde element uit de array gooien:

Parenthesized substring matches:

    * substring 1: did
    * substring 2: regexp
    * substring 3: ,work=false
    * substring 4: work
    * substring 5: false

Dat derde element is simpel weg te halen (zie boven). Het probleem is enkel dat deze methode niet werkt als je meer dan 2 key-value-pairs hebt. Dan worden match 3&4 steeds overschreven...
En ik neem aan dat je tot het einde wil matchen, voor de syntax-check. Voorbeeld van mogelijke fout: "a[a=b][c=d,]"