[php] Regular expression, gematchede tekst gebruiken

[r0bert]

Topic gaat eigenlijk alleen om het opstellen van een regular expression. Ik weet niet hoe ik moet beginnen, laat staan hoe het eruit moet komen te zien. Als voorbeeld gebruik ik een tekst met daarin ergens een "functie", tenminste het staat geschreven als een functie maar is gewoon tekst, die vervangen moet worden door een aantal andere teksten:

vb van tekst

code:

voorbeelden hiervan zijn opsomming("vbscript","javascript")

dan moet opsomming("vbscript","javascript") vervangen worden door:

HTML:

   <dt>vbscript</dt>
   <dt>javascript</dt>

Dus er moet tekst/html ingevoegd worden. De argumenten genoemd in opsomming moeten daarin gebruikt worden.

[ Voor 8% gewijzigd door r0bert op 12-05-2004 20:24 ]

[Gomez12]

Ikke nie snappe, vbscript, javascript zijn losse strings die in een array zitten??? Kijk dan eens naar $var= "<dt>" . $var . "</dt>"

Of zijn het strings die in een tekst zitten??? Kijk dan eens naar eregi.

[r0bert]

beschouw dit gewoon als 1 string

code:

$string = 'voorbeelden hiervan zijn opsomming("vbscript","javascript")';

zeg maar..

En het zal dus in de buurt van ereg/preg zitten.

[NMe]

opsomming(\"[^\"]+\",)*(\"[^\"]+\",)

Dat is je expressie denk ik. Kijk ook eens in de FAQ, en op PHP.net in de preg_replace manual.

[Domokoen]

Als het je alleen om de reguliere expressie gaat, zou je dan niet een tutorial zoeken?
Meestal kan dit met search/replace:
Search: vbscript|javascript
Replace with: <dt>\0</dt> (ookwel bekend als back-reference)

maar hoe je dat niet case sensitive doet mag je zelf uitzoeken, evenals welke functie in PHP dit is.

[r0bert]

Ik ben volgens mij echt dyslectisch voor reguliere expressies Ik klooi nog wel wat aan..

edit:
thnx TheDark

code:

opsomming(\"[^\"]+\",)*(\"[^\"]+\",)

Doet de truc hier niet.. nog niet, helaas..

[ Voor 193% gewijzigd door r0bert op 12-05-2004 20:47 ]

[MichelVH]

PHP:

preg_replace("#opsomming(\"[^\"]+\",)*(\"[^\"]+\",)#", $strSubject, $arrResult);

Of de expression verder goed is heb ik niet naar gekeken, maar je moet natuurlijk wel een geldige delimiter erom heen zetten

[ Voor 99% gewijzigd door MichelVH op 12-05-2004 20:46 ]

[Skaah]

preg_match("!\bopsomming\(\"[^)]*\")!i",$string,$matches);
$woorden = explode('","',$matches[1]);
for ($i = 0; $i < count($woorden); $i++)
{
if ($i == 1)
{
echo '<dl><dt>Opsomming</dt>'."\n";
}
echo '<dd>'.$woorden[$i].'</dd>'."\n";
if ($i == count($woorden))
{
echo '</dl>'."\n";
}
}

Wel ff een definition list <dl> item gebruiken hè! Goede code

[ Voor 130% gewijzigd door Skaah op 12-05-2004 21:04 ]

[r0bert]

Ik krijg deze:

code:

Warning: Compilation failed: unmatched parentheses at offset 20

Wat het betekend Maar het werkt niet.. En ja, DL wel gebruiken, maar die kan ik er al gewoon boven echo-en

dus

code:

echo dl
for-lus, dd's
echo /dl

edit:
zo krijg ik geen foutmelding:

code:

"!\bopsomming\(\"[^\)]*\"\)!i"

Maar ook geen resultaat

[ Voor 19% gewijzigd door r0bert op 12-05-2004 20:57 ]

[Skaah]

Pardon, dit is 'm.

PHP:

<pre><?php

$string = 'voorbeelden hiervan zijn opsomming("vbscript","javascript")';

preg_match("!\bopsomming\(\"([^)]*)\"\)!i",$string,$matches);

print_r($matches);

$woorden = explode('","',$matches[1]);
for ($i = 0; $i < count($woorden); $i++)
{
  if (!$i)
  {
    echo '<dl><dt>Opsomming</dt>'."\n";
  }
  echo '<dd>'.$woorden[$i].'</dd>'."\n";
  if ($i+1 == count($woorden))
  {
    echo '</dl>'."\n";
  }
}
?>

Output:

code:

<pre>Array
(
    [0] => opsomming("vbscript","javascript")
    [1] => vbscript","javascript
)
<dl><dt>Opsomming</dt>
<dd>vbscript</dd>
<dd>javascript</dd>
</dl>

[ Voor 90% gewijzigd door Skaah op 12-05-2004 21:03 ]

[r0bert]

Ik heb nu dit:

code:

$templResult = LoadFile($templFile);
$arrTemp     = preg_match("!\bopsomming\(\"([^)]*)\"\)!i", $templResult, $arrMatches);

Maar dit werkt niet! Als ik gewoon een string handmatig vul genaamd $templResult (zelfde waarde), maar als ik een bestand inlaad werkt het niet (de inhoud van het bestand wordt echt geladen in $templResult, want dat heb ik ge-echo-ed).. wat kan er nou fout zijn?

[ Voor 8% gewijzigd door r0bert op 15-05-2004 11:12 ]

[r0bert]

*kleine kick*

[NMe]

r0bert schreef op 15 mei 2004 @ 11:10:
Ik heb nu dit:

code:

$templResult = LoadFile($templFile);
$arrTemp     = preg_match("!\bopsomming\(\"([^)]*)\"\)!i", $templResult, $arrMatches);

Maar dit werkt niet! Als ik gewoon een string handmatig vul genaamd $templResult (zelfde waarde), maar als ik een bestand inlaad werkt het niet (de inhoud van het bestand wordt echt geladen in $templResult, want dat heb ik ge-echo-ed).. wat kan er nou fout zijn?

Ehm, LoadFile is geen standaardfunctie van PHP, dus dat werkt sowieso niet, tenzij het een zelfgeschreven functie is, wat weer betekent dat je ons die functie moet laten zien als we je moeten helpen.

Als je een standaardfunctie van PHP zoekt dan bedoel je waarschijnlijk readfile() maar die schrijft direct naar de output buffer. Probeer de volgende hoofdstukken van de manual maar eens:
file() voor het in één keer inlezen in een array

file_get_contents() voor het in één keer inlezen in een string

fopen()
fread() deze twee samen voor handmatig inlezen

[r0bert]

ok, mijn functie om de file in te lezen:

PHP:

/* Functie om bestand in te lezen .. */
function LoadFile($fileRef)
{
    $fileHandler    = @fopen($fileRef, 'r');
    $fileContent    = @fread($fileHandler, filesize($fileRef));
    @fclose($fileHandler);
    
    return $fileContent;
}

[NMe]

r0bert schreef op 16 mei 2004 @ 17:33:
ok, mijn functie om de file in te lezen:

PHP:

/* Functie om bestand in te lezen .. */
function LoadFile($fileRef)
{
    $fileHandler    = @fopen($fileRef, 'r');
    $fileContent    = @fread($fileHandler, filesize($fileRef));
    @fclose($fileHandler);
    
    return $fileContent;
}

Ok, dat is het probleem niet. Ik weet niet waar het dan wel zit.

Maar overal een @ voor zetten is wel gigantisch vies. Misschien is het een idee om fouten af te handelen in plaats van ze dmv @ te negeren?

[Verwijderd]

Je moet het @ teken gewoon compleet vermijden, maak een eigen class die de foutafhandeling regelt.

Zet de error_reporting op 0

Volgens mij is er ergens in de PHP documentatie zelfs nog informatie te vinden over het maken van een eigen foutafhandeling mechanisme.

[r0bert]

Ja, geloof dat dat waar is, maar is hier niet zo sterk van belang omdat het slechts om 1 paginaatje gaat.. Maar niemand weet dus een oplossing voor mijn probleem?

[NMe]

r0bert schreef op 16 mei 2004 @ 21:56:
Ja, geloof dat dat waar is, maar is hier niet zo sterk van belang omdat het slechts om 1 paginaatje gaat.. Maar niemand weet dus een oplossing voor mijn probleem?

Niet als hij bij de ene string wel werkt en bij de andere niet... Dat klinkt wel heel erg vaag namelijk.

[r0bert]

Ja, dat is dus ook het probleem Want als ik na LoadFile $templResult echo doet hij het gewoon prima..

[drm]

offtopic:
Je zou ook gewoon kunnen besluiten file_get_contents() te gebruiken

[NMe]

Ik betwijfel dat dit het oplost, maar probeer dit eens?

PHP:

$templResult = LoadFile($templFile);
$arrTemp     = preg_match("!\bopsomming\(\"([^\)]*)\"\)!i", $templResult, $arrMatches);

[Verwijderd]

NMe84 schreef op 17 mei 2004 @ 11:55:
Ik betwijfel dat dit het oplost, maar probeer dit eens?

PHP:

$templResult = LoadFile($templFile);
$arrTemp     = preg_match("!\bopsomming\(\"([^\)]*)\"\)!i", $templResult, $arrMatches);

uhm, volgens mij wil hij de het stukkie (opsomming(blabla)) vervangen hebben in de text, dus de matches eruit halen heeft dan vrijwel geen nut. Zover ik weet is het ook niet met een gewone preg_replace te doen omdat in een match weer meerdere results zitten (de dingen die tussen <dt> moeten komen).
daarom zou een preg_replace_callback de beste oplossing zijn

PHP:

function callback_preg($matches)
{
    $text="<dl><dt>Opsomming</dt>\n";
    $woorden = explode('","',$matches[1]);
    for ($i = 0; $i < count($woorden); $i++)
      $text.="<dd>$woorden[$i]</dd>\n";
    $text.= "</dl>\n";
    return $text;
}

$string='voorbeelden hiervan zijn opsomming("vbscript","javascript")';
$string=preg_replace_callback("!\bopsomming\(\"([^)]*)\"\)!i","callback_preg",$string);

hier is mijn versie (getest), een beetje geoptimaliseerd

[drm]

Mja, ik vind het eigenlijk een oplossing in de orde "Plakband"

Ik zou eerder zorgen dat de quotes ook daadwerkelijk functioneren als quotes en niet als iets "wat er gewoon leuk uitziet", zodat je ook gewoon haakjes en escape sequences kan gebruiken in de strings.

[NMe]

Verwijderd schreef op 17 mei 2004 @ 13:52:
uhm, volgens mij wil hij de het stukkie (opsomming(blabla)) vervangen hebben in de text, dus de matches eruit halen heeft dan vrijwel geen nut.

Ik heb gewoon zijn code gecopy-paste en daar een foutje uit gehaald. Hij had zijn code in principe namelijk al werkend, dus why change a winning team?

Verwijderd schreef op 17 mei 2004 @ 13:52:
Zover ik weet is het ook niet met een gewone preg_replace te doen omdat in een match weer meerdere results zitten (de dingen die tussen <dt> moeten komen).
daarom zou een preg_replace_callback de beste oplossing zijn

PHP:

function callback_preg($matches)
{
    $text="<dl><dt>Opsomming</dt>\n";
    $woorden = explode('","',$matches[1]);
    for ($i = 0; $i < count($woorden); $i++)
      $text.="<dd>$woorden[$i]</dd>\n";
    $text.= "</dl>\n";
    return $text;
}

$string='voorbeelden hiervan zijn opsomming("vbscript","javascript")';
$string=preg_replace_callback("!\bopsomming\(\"([^)]*)\"\)!i","callback_preg",$string);

hier is mijn versie (getest), een beetje geoptimaliseerd

Das idd een stuk beter.

[Verwijderd]

drm schreef op 17 mei 2004 @ 14:04:
Mja, ik vind het eigenlijk een oplossing in de orde "Plakband"

yep, das waar
maarja, het lijkt me niet dat dit het enige is wat er ooit veranderd zou moeten worden, en om nu ff in 5 minuten dynamische parser in elkaar te gaan fabrieken vind ik ook niets.
En dan ook nog.. de TS lijkt me nog niet al te bekend met php en om dan meteen een ingewikkelde oplossing voor te schotelen is naar mijn mening ook niet echt handig.