[PHP] str_replace met variablen werkt niet

[Mac_Cain13]

Ik ben druk bezig met mijn projectje en wilde nu getypte smilies vervangen door plaatjes. Dus ik heb de str_replace erbij gepakt. Alleen zodra ik de search of replace string vervang door een variable gaat het helemaal mis.

Als ik doe:

PHP:

<? $reply = ":S";
$reply = str_replace(":S", "imgcode", $reply); ?>

Krijg ik netjes 'imgcode' terug. Dat is prima!

Maar als ik nou variablen ga gebruiken:

PHP:

<? $reply = ":S"; $tag = ":S"; $img = "imgcode";
$reply = str_replace($tag, $img, $reply); ?>

Krijg ik helemaal niets terug, zelfs geen spatie oid.

Als ik alleen "imgcode" vervang door en variable krijg ik ook niets terug, ook geen spatie. Bij alleen ":S" vervangen door een variable krijg ik ":S" weer terug, dus lijkt str_replace helemaal niets te doen.

Die variablen moeten later arrays worden, maar met arrays had ik dezelfde problemen. Nu zag ik op php.net staan dat er een bug is PHP in versies lager dan 4.3.3. (Ik gebruik nog 4.2.2 en kan pas over een poos upgraden naar een hogere versie )

quote: http://www.php.net/manual/en/function.str-replace.php

In PHP versions prior to 4.3.3 a bug existed when using arrays as both search and replace parameters which caused empty search indexes to be skipped without advancing the internal pointer on the replace array. This has been corrected in PHP 4.3.3, any scripts which relied on this bug should remove empty search values prior to calling this function in order to mimick the original behavior.

Als ik het goed begrijp gaat PHP de fout in als ik variablen gebruik die leeg zijn, of iig bij een array met een leeg veld. Maar dat lijkt mij niet van toepassing op mijn bovenstaande constructie. Want er worden (nog) geen arrays gebruikt en de variablen zijn zeker niet leeg.

Weet iemand waar ik hier de fout in ga? Of ligt het hem toch aan de bug in PHP? Alvast bedankt

[Mawlana]

Regular Expressions
str_replace() gebruiken voor dit doel is domweg ranzig.

[IceM]

Ik heb deze code even getest op php 4.3.3 (helaas, geen zin om te downgraden naar 4.2) en hij gaf wel gewoon een goede output

PHP:

<?PHP
$reply = ":S"; 
$tag = ":S"; 
$img = "imgcode"; 

$reply = str_replace($tag, $img, $reply); 

echo $reply;
?> 

[Mac_Cain13]

IceM schreef op 18 april 2004 @ 10:35:
Ik heb deze code even getest op php 4.3.3 (helaas, geen zin om te downgraden naar 4.2) en hij gaf wel gewoon een goede output

PHP:

<?PHP
$reply = ":S"; 
$tag = ":S"; 
$img = "imgcode"; 

$reply = str_replace($tag, $img, $reply); 

echo $reply;
?> 

Tnx voor het testen, ik begin nu ernstig te twijfelen aan mijn orginele code...
Ik pruts ff verder. Ik gebruik de str_replace namelijk in een functie en volgens mij komt er een variable niet helemaal goed over.

@sylvesterrr: Ik vind een hele expressie opzetten voor zo'n simpele replace ranzig. Op php.net zelf staat ook:

quote: http://www.php.net/manual/en/function.str-replace.php

[...] If you don't need fancy replacing rules (like regular expressions), you should always use this function [...]

En laat ik nou voor dit simpele klusje ff geen 'fancy replacing rules' nodig hebben.

Kortom, volgens mij is str_replace juist gemaakt voor dit soort klusjes. Regular expressions zijn juist weer supper handig om ingewikkelde replace dingen enz op te lossen. Dan voldoet str_replace niet meer.

edit:

Woei! Hij doet het
De fout zat idd bij men functie. Ik declareerde de variablen $img en $tag buiten mijn functie en dat blijkt niet te mogen. Daar worden namelijk nogal leeg van, vandaar die vreemde resultaten. Stomme fout, maarjah shit happens he.
Iig tnx voor de hulp, die mij mijn fout deed inzien

[ Voor 18% gewijzigd door Mac_Cain13 op 18-04-2004 10:54 ]

[djc]

sylvesterrr schreef op 18 april 2004 @ 10:35:
Regular Expressions
str_replace() gebruiken voor dit doel is domweg ranzig.

Dan heb je er toch weinig van begrepen. Regular expressions gebruiken voor een simpele replace zoals dit is overkill, en zal de performance van je scripts niet ten goede komen.

[Mawlana]

Manuzhai schreef op 18 april 2004 @ 10:56:
[...]
Regular expressions gebruiken voor een simpele replace zoals dit is overkill, en zal de performance van je scripts niet ten goede komen.

Hij is druk bezig met een projectje, en nu wil hij de smilies gaan parsen. Ik ging er klakkeloos van uit (sorry) dat die smilies in een lang stukje tekst zijn verwerkt (bijv. een nieuwsbericht/forumpost e.d.).

PHP:

<? 
$reply = "http://www.ditiseenlink.nl is bladiebla.... :/"; 
$tag = ":/"; 
$img = "<img src=\"/images/smilies/nosmile.gif\" alt=\"nosmile\"";
$reply = str_replace($tag, $img, $reply); 
echo $reply;
?> 

Gaat niet zo goed he?

[ Voor 7% gewijzigd door Mawlana op 18-04-2004 11:03 ]

[Mac_Cain13]

sylvesterrr schreef op 18 april 2004 @ 11:03:
[...]

Hij is druk bezig met een projectje, en nu wil hij de smilies gaan parsen. Ik ging er klakkeloos van uit (sorry) dat die smilies in een lang stukje tekst zijn verwerkt (bijv. een nieuwsbericht/forumpost e.d.).

PHP:

<? 
$reply = "http://www.ditiseenlink.nl is bladiebla.... :/"; 
$tag = ":/"; 
$img = "<img src=\"/images/smilies/nosmile.gif\" alt=\"nosmile\"";
$reply = str_replace($tag, $img, $reply); 
echo $reply;
?> 

Gaat niet zo goed he?

Hmm, ik ben nogal n00b met regular expressions dus ik vond str_replace ook gewoon makkelijker. Alleen staan de smilies idd in een stukje tekst. Alleen waarom zou die replace met een img-tag niet gaan. Hij doet het hier prima hoor

Ik heb van die variablen ondertussen een arraytje gemaakt. Alleen is het wel lastig dat ik niet

PHP:

str_replace($tag, "blaat".$img, $reply);

Mag doen ipv de inhoud van $img krijg ik dan 'Array' terug. Is hier een simpele oplossing voor? Of moet ik nu toch naar die enge regular expressions Zo jah zou een stukje (pseudo)code niet gek zijn.

[ Voor 5% gewijzigd door Mac_Cain13 op 18-04-2004 11:27 ]

[T-MOB]

PHP:

function smiley($string) 
    {

        $smile = array (
            'Z:)' => '[img]"/smily/s10.gif"[/img]',
            '~:)' => '[img]"/smily/s8.gif"[/img]',
            ':0)' => '[img]"/smily/s16.gif"[/img]',
            ':-)' => '[img]"/smily/s1.gif"[/img]',
            ';-)' => '[img]"/smily/s2.gif"[/img]',
            ':-D' => '[img]"/smily/s3.gif"[/img]',
            ':-p' => '[img]"/smily/s4.gif"[/img]',
            ':-P' => '[img]"/smily/s4.gif"[/img]',
            'db-' => '[img]"/smily/s5.gif"[/img]',
            '|%|' => '[img]"/smily/s6.gif"[/img]',
            ':?(' => '[img]"/smily/s7.gif"[/img]',
            ':-(' => '[img]"/smily/s9.gif"[/img]',
            '8)-' => '[img]"/smily/s11.gif"[/img]',
            ':#(' => '[img]"/smily/s12.gif"[/img]',
            ':-0' => '[img]"/smily/s13.gif"[/img]',
            '|~p' => '[img]"/smily/s14.gif"[/img]',
            '):X' => '[img]"/smily/s15.gif"[/img]'      
            );
        
        for(reset($smile); $key=key($smile); next($smile)) 
            {
            $string = str_replace("$key","$smile[$key]",$string);
            }
        return ($string);
    }

Dit is mijn smiley-script, met array...

[Mac_Cain13]

Tnx T-MOB!

Alleen vind ik het lastig dat ik steeds die hele img-tag in men array moet hebben. Ik wil graag alleen de naam van het gifje in de array en dan die img-tag statisch ernaast, maar dat is mss met een btje jou script modificeren wel te halen. Meer suggesties zijn altijd nog welkom natuurlijk.

[blizt]

Dit komt uit een projectje wat ik atm aan 't herschrijven ben, maar misschien heb je er wat aan:

PHP:

                $smilies[':arrow:'] = 'arrow.gif';
        $smilies[':evillaugh:'] = 'badgrin.gif';
        $smilies[':D'] = 'biggrin.gif';
        $smilies[':?'] = 'confused.gif';
        $smilies['8)'] = 'cool.gif';
        $smilies[':cry:'] = 'cry.gif';
        $smilies[':doubt:'] = 'doubt.gif';
        $smilies[':evil:'] = 'evil.gif';
        $smilies[':!:'] = 'exclaim.gif';
        $smilies[':idea:'] = 'idea.gif';
        $smilies[':lol:'] = 'lol.gif';
        $smilies[':x'] = 'mad.gif';
        $smilies[':?:'] = 'question.gif';
        $smilies[':P'] = 'razz.gif';
        $smilies[':oops:'] = 'redface.gif';
        $smilies[':roll:'] = 'rolleyes.gif';
        $smilies[':('] = 'sad.gif';
        $smilies[':shock:'] = 'shock.gif';
        $smilies[':)'] = 'smile.gif';
        $smilies[':o'] = 'surprised.gif';
        $smilies[';)'] = 'wink.gif';
        
        foreach($smilies as $key => $value)
        {
            
            $psOutput = str_replace($key,'[img]"'[/img]get_config('template') . '/' . $value . '" 
                                          alt="' . $key . '" />',$psOutput);
            
        }

[PrisonerOfPain]

T-MOB schreef op 18 april 2004 @ 11:36:
[....]
Dit is mijn smiley-script, met array...

waarom niet gewoon

PHP:

<?php
function smiley($string) 
{

    $smile = array (
        'Z:)' => '[img]"/smily/s10.gif"[/img]',
        '~:)' => '[img]"/smily/s8.gif"[/img]',
        ':0)' => '[img]"/smily/s16.gif"[/img]',
        ':-)' => '[img]"/smily/s1.gif"[/img]',
        ';-)' => '[img]"/smily/s2.gif"[/img]',
        ':-D' => '[img]"/smily/s3.gif"[/img]',
        ':-p' => '[img]"/smily/s4.gif"[/img]',
        ':-P' => '[img]"/smily/s4.gif"[/img]',
        'db-' => '[img]"/smily/s5.gif"[/img]',
        '|%|' => '[img]"/smily/s6.gif"[/img]',
        ':?(' => '[img]"/smily/s7.gif"[/img]',
        ':-(' => '[img]"/smily/s9.gif"[/img]',
        '8)-' => '[img]"/smily/s11.gif"[/img]',
        ':#(' => '[img]"/smily/s12.gif"[/img]',
        ':-0' => '[img]"/smily/s13.gif"[/img]',
        '|~p' => '[img]"/smily/s14.gif"[/img]',
        '):X' => '[img]"/smily/s15.gif"[/img]'      
        );
    return (str_replace (array_keys ($smile), $smile, $string));
}
echo smiley("Z:)");
?>

[ Voor 39% gewijzigd door PrisonerOfPain op 18-04-2004 12:11 ]

[Mac_Cain13]

Tnx iedereen! Het is gelukt
Met een beetje combineren en nadenken vond ik dit de beste constructie.

PHP:

function addsmiley($reply) {
    $smiley = array(
        ":)" => "smile.gif",
        ";)" => "knipoog.gif",
        ";(" => "huil.gif",
        ":P" => "tong.gif",
        ":(" => "boos.gif"
    );

    for(reset($smiley); $key=key($smiley); next($smiley)) {
     $reply = str_replace($key, "[img]\"img/smiley/".$smiley[$key]."\"[/img]", $reply);
    }

    return $reply;
}

Ik heb op deze manier slechts 1 array en alleen de naam van het gifje staat in die array. Niet de hele img-tag, das wel zo overzichtelijk en scheeld weer een btje RAM

[PrisonerOfPain]

Mac_Cain13 schreef op 18 april 2004 @ 13:39 een stukje code en:
Ik heb op deze manier slechts 1 array en alleen de naam van het gifje staat in die array. Niet de hele img-tag, das wel zo overzichtelijk en scheeld weer een btje RAM

Waarom pass je $reply dan by-value? en waarom niet gewoon een foreach ipv deze 'for-constructie' ?

[ Voor 48% gewijzigd door PrisonerOfPain op 18-04-2004 14:32 ]

[Thijsmans]

Mac_Cain13 schreef op 18 april 2004 @ 13:39:
...

Ik vind het een erg hippe constructie die ik nog nooit heb gezien, maar ik denk dat deze sneller is

PHP:

foreach( $smiley AS $charcode => $path )
{
    return str_replace($charcode, 'imgtag'.$path.'/img', $reply);
}

[ Voor 61% gewijzigd door Thijsmans op 18-04-2004 15:31 ]

[Blaise]

De fout zat idd bij men functie. Ik declareerde de variablen $img en $tag buiten mijn functie en dat blijkt niet te mogen. Daar worden namelijk nogal leeg van, vandaar die vreemde resultaten.

Waarschijnlijk handig om te weten: doe "global $varnaam" aan het begin van de definitie van je function. Op die manier is een variabele ook van kracht binnen de function.

Ik had daar ooit ook wat problemen mee, vandaar

[stekkel]

Prammenhanger schreef op 18 april 2004 @ 15:30:
[...]


Ik vind het een erg hippe constructie die ik nog nooit heb gezien, maar ik denk dat deze sneller is

PHP:

foreach( $smiley AS $charcode => $path )
{
    return str_replace($charcode, 'imgtag'.$path.'/img', $reply);
}

Die werkt niet omdat bij de eerste replacement de loop verlaten wordt vanwege de return statement.

deze is sneller:

PHP:

function addsmiley($reply) {
    $pre = '[img]"img/smiley/';
[/img]'
    // nog sneller is de complete image tag zonder gebruik van pre en end maar dat vond de 
    // topicstarter niet gewenst
    $smiley = array(
        ':)' => $pre . 'smile.gif' . $end,
        ';)' => $pre . 'knipoog.gif' . $end,
        ';(' => $pre . 'huil.gif' . $end,
        ':P' => $pre . 'tong.gif' . $end,
        ':(' => $pre . 'boos.gif' . $end
    ); 
    // vanaf php 4.0.6 kan je ook arrays als argument gebruiken bij str_replace

    // edit naar aanleiding comment prisonerofpain
    // return str_replace(array_keys($smiley),array_values($smiley),$reply);
    return str_replace(array_keys($smiley),$smiley,$reply);
}

[ Voor 61% gewijzigd door stekkel op 18-04-2004 15:53 ]

[PrisonerOfPain]

@stekkel: array_values is niet nodig str_replace kijkt namelijk alleen naar de values, en dan is het een beetje zonde, zie ook het stuk dat ik eerder poste.

[stekkel]

PrisonerOfPain schreef op 18 april 2004 @ 15:49:
@stekkel: array_values is niet nodig str_replace kijkt namelijk alleen naar de values, en dan is het een beetje zonde, zie ook het stuk dat ik eerder poste.

Daar heb je gelijk in

edit:

zie nu pas dat ik hetzelfde als jou deed. Ik had alleen je eerste post gezien, maar ja great minds think the same

[ Voor 19% gewijzigd door stekkel op 18-04-2004 22:14 ]

[PrisonerOfPain]

stekkel schreef op 18 april 2004 @ 15:51:
[...]


Daar heb je gelijk in

Ow, en foreach zit ook pas in php sinds versie 4, dus maakt dat niet zo heel veel uit.

[Mac_Cain13]

stekkel schreef op 18 april 2004 @ 15:44:
[...]
deze is sneller:

PHP:

function addsmiley($reply) {
    $pre = '[img]"img/smiley/';
[/img]'
    // nog sneller is de complete image tag zonder gebruik van pre en end maar dat vond de 
    // topicstarter niet gewenst
    $smiley = array(
        ':)' => $pre . 'smile.gif' . $end,
        ';)' => $pre . 'knipoog.gif' . $end,
        ';(' => $pre . 'huil.gif' . $end,
        ':P' => $pre . 'tong.gif' . $end,
        ':(' => $pre . 'boos.gif' . $end
    ); 
    // vanaf php 4.0.6 kan je ook arrays als argument gebruiken bij str_replace

    // edit naar aanleiding comment prisonerofpain
    // return str_replace(array_keys($smiley),array_values($smiley),$reply);
    return str_replace(array_keys($smiley),$smiley,$reply);
}

Hey die is cool
Die for-loop zat me al niet helemaal lekker. Dit met die variablen pre en end is een mooie constructie! Blijft het tenminste een btje overzichtelijk en snel. Tnx!

[T-MOB]

PrisonerOfPain schreef op 18 april 2004 @ 12:10:
waarom niet gewoon
[...]

Ach ja, die functie staat er al zo lang... En als zaken naar behoren werken verbeter je ze niet zo snel....