[PHP] Variabelen in string replacen

[Kaastosti]

Ik heb een lap tekst uit m'n database. In die tekst komen variabelen voor tussen de tekst.

Blahdiblah $variabele meerblahblah

Eerst include ik een bestand waarin alle mogelijke variabelen staan. Iets als:

PHP:

$kaas        = 'tosti';
$tosti      = 'worst';
$variabele  = 'kaas';

Dan wordt de tekst uit de database gehaald, waarna ik de variabelen daarin wil vervangen door de waarden uit het aparte bestand.

PHP:

$query    = "SELECT content FROM tabel WHERE id='".$this->id."'";
$result   = mysql_query($query);
$row      = mysql_fetch_assoc($result);

In $row zit dan de inhoud van de database in stringvorm "Blahdiblah $variabele meerblahblah"

Door wat zoekwerk kwam ik uit op de functie eval(), maar deze geeft geen restulaat. Of er komt een syntax error uit, of er staat de letter 'A' als enige output, of gewoon helemaal niets. Toch heb ik de indruk dat ik niet de eerste ben die zoiets wil doen en dat er dus ongetwijfeld een functie in php zal zitten waarmee ik dit effect snel kan bereiken. Help!

[killercow]

Ik denk dat je het beste je text uit de je database kunt replacen met de volgende setup:

PHP:

$text='Jouw text, met [variabelen] en nog meer [test] tags.';
$vars['variabelen']='een';
$vars['test']='twee';
$text=preg_replace("/\\[(\S+)\]/Ue","\$vars\\['\\1'\]",$text);

Zal geven: Jouw text, met een en nog meer twee tags

Eval is een vrij enge functie, aangezien je ook gevaarlijke stukken php code zou kunnen uitvoeren die in de database staan, of gezet zijn door derden. (daarnaast is alles zonder eval veiliger op te lossen)

[Kaastosti]

Ik heb inderdaad gelezen dat eval niet echt een populaire functie is, maar deze leek de enige te zijn voor een dergelijke actie. preg_replace zou inderdaad ook moeten werken... eens kijken.

[Michali]

killercow schreef op woensdag 13 april 2005 @ 16:41:
Ik denk dat je het beste je text uit de je database kunt replacen met de volgende setup:

PHP:

$text='Jouw text, met [variabelen] en nog meer [test] tags.';
$vars['variabelen']='een';
$vars['test']='twee';
$text=preg_replace("/\\[(\S+)\]/Ue","\$vars\\['\\1'\]",$text);

Zal geven: Jouw text, met een en nog meer twee tags

Eval is een vrij enge functie, aangezien je ook gevaarlijke stukken php code zou kunnen uitvoeren die in de database staan, of gezet zijn door derden. (daarnaast is alles zonder eval veiliger op te lossen)

Volgens mij is het resultaat dan: "Jouw text, met $vars['variabelen'] en nog meer $vars['test'] tags."

Beetje vreemd stukje code imo.

[Sybr_E-N]

eval() is zoals gezet eval, maar waarom sla een een variabele op in een database? Heeft dat een bepaalde reden, want het meestal niet de meest elegante manier van doen.

[Michali]

werkt dit niet:

PHP:

$string = "abc \$var ghi";
$var = "def";
$code = '$string = "'  . $string . '";';
eval($code);
print $string;

Ik blaat maar wat, het is niet echt netjes oid.

[Verwijderd]

Kijk eens goed naar het volgende voorbeeld:

PHP:

<?php

// INIT
$teststring = 'De grote hond was nooit bang.';

// SEARCH
// Vul een array met waarden die je wilt zoeken
$search[] = 'grote';
$search[] = 'hond';

// REPLACEMENTS
// Vul een array met waarden waarmee je wilt vervangen
$replacement[] = 'kleine';
$replacement[] = 'kat';

// EXECUTE
$string = str_replace($search, $replacement, $teststring);

// SHOW
echo $string;

?>

Als je gebruik wilt maken van een meer gecompliceerde manier van zoeken, voor bijvoorbeeld zinsdelen of woorden waarvan je de opbouw niet exact weet (etc.) maak dan gebruik van preg_replace(), een soort zelfde functie. Voor details verwijs ik je graag naar de php manual...

[ Voor 6% gewijzigd door Verwijderd op 13-04-2005 17:00 . Reden: typo ]

[Kaastosti]

Het nadeel van die manier van replacen, is dat de indices moeten kloppen. Dat werkt niet meer op het moment dat je er namen aan gaat geven, wat ik wel zou willen om de leesbaarheid te verhogen.

Ik geef direct toe dat het opslaan van variabelen in de database niet echt netjes is, maar daar ontkom ik niet aan. Niet alleen staat alles al in de database, het is de enige manier om met deze bestanden te werken. Het gaat om het genereren van bestanden waarin die variabelen voor kunnen komen. Elk bestand is weer anders en is dus niet generiek op te bouwen met scripting.. vandaar de variabelen in de database.

Het is me overigens met eval wel al gelukt om de variabelen te converteren...probleem zat in een array-index foutje mijnerzijds. Toch houd ik me aanbevolen voor de preg_replace manier, die ben ik nog aan het proberen

[ Voor 16% gewijzigd door Kaastosti op 13-04-2005 17:06 ]

[The Milkman]

Wel is str_replace sneller dacht ik...

[Verwijderd]

PHP:

<?php

// INIT
$teststring = 'De $grote $hond was nooit bang.';

// SEARCH
// Vul een array met waarden die je wilt zoeken
$search[] = '$grote';
$search[] = '$hond';

// REPLACEMENTS
// Vul een array met waarden waarmee je wilt vervangen
$replacement[] = 'kleine';
$replacement[] = 'kat';

// EXECUTE
$string = str_replace($search, $replacement, $teststring);

// SHOW
echo $string;

?>

?

_{Als dit niet is wat je zoekt, dan snap ik er helemaal niets van. Eet smakelijk, ga er vandoor.}

[ Voor 9% gewijzigd door Verwijderd op 13-04-2005 17:29 ]

[Kaastosti]

Het kan inderdaad met bovenstaande code, maar wat waarmee replaced wordt is dat niet erg duidelijk bedoel ik. Als ik een lijst heb van 30 variabelen is niet snel te zien wat de waardes gaan worden.

[drm]

't grootste nadeel van eval is dat als iemand zo leuk is om malicious code in de content te zetten (op wat voor manier dan ook) dat je de pineut bent.

Je kunt het beste gebruik maken van preg_replace_callback. En een mooie class eromheen doet 't altijd goed.

PHP:

<?
class Environment {
   function Environment ( &$env ) {
      $this->env =& $env;
   }

   function lookup ( $varName ) {
      if ( $varName [0] {0} == '\\' ) {
         return substr ( $varName[0], 1 );
      }
      if ( array_key_exists ( $varName[1], $this->env ) ) {
         return $this->env [ $varName[1] ];
      }
      return null;
   }

   function replaceAll ( $str ) {
      return preg_replace_callback ( '/\\\?\$(\w+)/', array ( &$this, 'lookup' ), $str );
   }
}

function localreplace ( $str ) {
   $var = 'some local var contents';
   
   // environment op basis van de lokale variabelen
   $localEnv = new Environment ( get_defined_vars () );
   return $localEnv->replaceAll ( $str );
}


// environment op basis van de globale variabelen
$globalEnv = new Environment ( $GLOBALS );

// environment op basis van een array (evt later te vullen)
$myMap = array ();
$myEnv = new Environment ( $myMap );

$str = '\$var needs to be replaced with &quot;$var&quot;';

$myMap [ 'var' ] = 'other environment var contents';
$var = 'global var contents';

printf ( 
   '<p>With an array environment: <strong>%s</strong></p>
   <p>With a local environment: <strong>%s</strong></p>
   <p>And with the global environmentt: <strong>%s</strong></p>
   ', 
   $myEnv->replaceAll ( $str ),
   localreplace ($str),
   $globalEnv->replaceAll ( $str )
);
?>

Ik raad je wel af gebruik te maken van die globals array, i.v.m. mogelijke security holes.

[Michali]

Kaastosti schreef op woensdag 13 april 2005 @ 17:40:
Het kan inderdaad met bovenstaande code, maar wat waarmee replaced wordt is dat niet erg duidelijk bedoel ik. Als ik een lijst heb van 30 variabelen is niet snel te zien wat de waardes gaan worden.

Dat is toch vrij simpel te omzeilen door er zelf een functie om heen te bouwen?

PHP:

<?php

function replaceKeysWithValues($keyValueArray, $target)
{
    return str_replace(
        array_keys($keyValueArray),
        array_values($keyValueArray),
        $target
    );
}

$variablesToReplace = array(
    '$var1' => 'value1',
    '$var2' => 'value2',
    '$var3' => 'value3'
);

$string = '$var1, $var2, $var3';

print replaceKeysWithValues($variablesToReplace, $string);

?>

[Verwijderd]

Michali schreef op woensdag 13 april 2005 @ 19:17:
[...]


Dat is toch vrij simpel te omzeilen door er zelf een functie om heen te bouwen?

PHP:

<?php

function replaceKeysWithValues($keyValueArray, $target)
{
    return str_replace(
        array_keys($keyValueArray),
        array_values($keyValueArray),
        $target
    );
}

$variablesToReplace = array(
    '$var1' => 'value1',
    '$var2' => 'value2',
    '$var3' => 'value3'
);

$string = '$var1, $var2, $var3';

print replaceKeysWithValues($variablesToReplace, $string);

?>

Exact. Ofwel, @TS, wees zelf ook creatief ;-) NOFI

[ Voor 10% gewijzigd door Verwijderd op 14-04-2005 09:54 ]

[Kaastosti]

Ik wil ook best creatief zijn, tuurlijk, leuk, maar ik was er van overtuigd dat er voor zoiets een oersimpele functie moest zijn Nu is het wel zo dat alle informatie in de database er wordt neergezet door mijzelf of een collega, dus malafide code zal niet voorkomen. Ik heb het er over gehad en ik laat nu heel even eval() staan, om me niet stuk te staren op dit probleem.

Als ik aan het einde van m'n stage tijd over houd ga ik dit soort schoonheidsfoutjes er uit filteren, want er zitten nog een aantal puntjes in waar ik nog niet helemaal tevreden over ben. Bovenstaande suggesties zijn in ieder geval zeer welkom, die ga ik dan ook proberen. Bedankt!

[Michali]

Kaastosti schreef op donderdag 14 april 2005 @ 11:36:
Ik wil ook best creatief zijn, tuurlijk, leuk, maar ik was er van overtuigd dat er voor zoiets een oersimpele functie moest zijn Nu is het wel zo dat alle informatie in de database er wordt neergezet door mijzelf of een collega, dus malafide code zal niet voorkomen. Ik heb het er over gehad en ik laat nu heel even eval() staan, om me niet stuk te staren op dit probleem.

Als ik aan het einde van m'n stage tijd over houd ga ik dit soort schoonheidsfoutjes er uit filteren, want er zitten nog een aantal puntjes in waar ik nog niet helemaal tevreden over ben. Bovenstaande suggesties zijn in ieder geval zeer welkom, die ga ik dan ook proberen. Bedankt!

Waarom niet gelijk goed doen? Smerige dingen voor later bewaren is gewoon verkeerd. Bovendien is dit een potentieel veiligheids probleem (waarvan ik overigens niet zo snel kan bedenken wat er fout kan gaan, maar een voorgevoel zegt bijn altijd wel wat). Probeer eval iig zo veel mogelijk te vermijden.