[PHP] preg_replace kom er niet uit

[aex351]

Ik heb wat problemen met preg_replace waar ik niet uitkom. .
Onderstaande functie pakt de output van de buffer en zoekt naar alle href's die het dan moet gaan converteren naar "zoekmachine vriendelijke urls" . Het zoeken naar de href's is regexp is geen probleem, het veranderen van waardes daarintegen weer wel.

Bij array positie 2 begint het probleem. Regexp moet zoeken naar alle = (is) in de href en dat converteren naar / (slash) . Heb tientallen manieren in regexp geprobeerd maar kom er niet uit. Voor het gemak heb ik de regexp hieronder totaal gestript. Het veranderd nu alle = (is) in compleet broncode.

PHP:

    public function convertHref ( $value ) { 
        $arr = array(
            '%href="(.*)[!?](.*)"%',
            '%=%',
        );
        
        $replace = array(
            'href="\2"',
            '/"',
        );
        /*
        $value is : (directe input in output broncode)
         <a href="index?var=inhoud&var2=inhoud2&var3=inhoud3">Dit moet geconverteerd worden</a>
         <a href="?var=inhoud&var2=inhoud2">Dit moet geconverteerd worden</a>
        */
        $value = preg_replace($arr,$replace,$value);
        return $value ;
    }

[Room42]

Hier stond wat doms, zo laat op de avond...

Nah ik zal sportief zijn: Ik schreef:

Waarom doe je dat niet gewoon met str_replace? Die is voor dit soort simpele dingen toch veel efficienter?

Maar dat geintje gaat natuurlijk niet op, omdat je alles voor de ? ongemoeit wilt laten, natuurlijk.

[ Voor 50% gewijzigd door Room42 op 02-04-2006 06:27 ]

[brokenp]

gebruik de functie parseURL, http://www.php.net/function.parse-url, en dan een str replace, en dan alles omzetten uit het query deel met str_replace

Of zie ik iets over het hoofd?

[JHS]

Zoiets:

PHP:

$input = "<a href='http://bladiebla.com?a=b&dom=blaat' style=\"bla\">bla</a>";
echo convertHref($input); /* => <a href='http://bladiebla.com/b/blaat' style="bla">bla</a> */

function convertHref ($input) { 
  $arr = array(
    "/href=([\"|'])(.*)[!?](.*)([\"|'])/Usi",
    "/[\/|&][a-z][a-z0-9]*=([a-z0-9]*)/Usi"[s][/s]
  );        
    
  $replace = array(
    "href=$1$2/$3$4",
    "$3/"
  );

  return preg_replace($arr,$replace,$input);
}

[ Voor 3% gewijzigd door JHS op 02-04-2006 10:52 ]

[Michali]

{1,1} is toch gewoon hetzelfde als helemaal geen multipliciteit opgeven?

[JHS]

Hm, ja, je hebt gelijk . Het is vroeg, zullen we maar zeggen . Zie edit.

[crisp]

Ik zou dit zeker niet in 1 regexp willen oplossen, maar eigenlijk in 3 stappen. Eerst anchor-tags selecteren, daarna daaruit de href-attributen selecteren, en dan pas iets met het href attribuut doen. Je krijgt dan zoiets:

PHP:

$value='    <a href="index?var=inhoud&var2=inhoud2&var3=inhoud3">Dit moet geconverteerd worden</a>
        <a href="?var=inhoud&var2=inhoud2">Dit moet geconverteerd worden</a>';

$value = preg_replace_callback('~<a(\s([^</>]*|\'[^\']*\'|\"[^\"]*\")*)?[</>]~', 'parse_anchor', $value);

echo htmlspecialchars($value);

function parse_anchor($matches)
{
    return preg_replace_callback('~(\shref\s*=\s*)(\'[^\']*\'|\"[^\"]*\"|[^\s</>]*)~', 'parse_href', $matches[0]);
}

function parse_href($matches)
{
    // doe iets met $matches[2];
    return $matches[1] . $matches[2];
}

[aex351]

Ik heb het uiteindelijk op een andere manier opgelost. Alles via 1 regexp was niet te doen. Het is nu een combinatie geworden van preg_match_all str_replace en preg_replace.

Code is nog aan optimalisatie toe maar voor als iemand het zelfde probleem heeft, hieronder vind je de gestripte versie.

PHP:

    public function convertHref ( $value ) 
    { 
        $OUTPUT = $value ;
        preg_match_all ('/href="(.*)"/', $value, $value);
        $x = 0 ;
        $arr = array ();
        $value2 = array();
        while ( !empty ($value[1][$x] ) ) {
            $arr[$x] = '%'. str_replace('?','\?', $value[1][$x] ).'%' ;
            $value2[$x] = str_replace('&','/', $value[1][$x] );
            $value2[$x] = str_replace('/',$this->_separator, $value2[$x] );
            $x++;   
        }
        $value = preg_replace ($arr, $value2, $OUTPUT);
        return $value ;
    }

[JHS]

Ik zou van /href="(.*)"/ wel even /href="(.*)"/U maken, aangezien je anders lijkt me een probleem hebt als er twee a elementen in de $value staan . Maar wat vond je onelegant aan mijn i.c.m. / of crisp's oplossing?

[crisp]

Een greedy match lijkt me hier niet verstandig
Verder vind ik je oplossing echt verre van ideaal. Wie zegt dat je matches echt allemaal href-attributen van anchors zijn? En dat er ook overal netjes dubbele quotes zijn gebruikt?

En what about:

HTML:

<area nohref="nohref">

[ Voor 22% gewijzigd door crisp op 02-04-2006 18:02 ]