[DasMööp]

Hallo,

habe hier einen Codefetzen der mich auf die Palme bringt, es geht darum
einen String zu ersetzen ... aber in diesem ist eine url angegeben

z.B. home.php?1=2

PHP-Code:

$string = str_replace('?','\?',$string);

        while (preg_match("/<a#(.*)#a>/isU", $string, $pregMatch)) {

            $arrayString = explode('#',$pregMatch[1]);

            $content = '<img src="./html/system/img/link.gif" alt="link.gif"> <a href="'. str_replace('\?','?',$arrayString[0]) .'" title="'.$arrayString[1].'" name="'.$arrayString[1].'" accesskey="'.$arrayString[2].'">'.$arrayString[1].'</a><br>';

            $string = preg_replace("/<a#".$pregMatch[1]."#a>/isU",$content,$string);

        } 


Wenn man ?1=2 weglässt geht es ... ich verstehe nur nicht warum da
ich ja ? ausklammere!?

Wo liegt der Wurm begraben sieht ihn jemand ...

[MrNiceGuy]

Das Problem sollte in dieser Zeile liegen:

PHP-Code:

            $string = preg_replace("/<a#".$pregMatch[1]."#a>/isU",$content,$string); 


Du hast zwar am Anfang das ? Mit einem Backslash versehen, jedoch bin ich mir nicht sicher, dass das auch Bestand hat. Überprüfe mal das Suchergebnis $pregMatch[1]. Ansonsten rate ich dir aber auch noch folgende Funktion an: preg_quote()

[DasMööp]

Hi,

PHP-Code:

$string = str_replace('?','\?',$string); 

        while (preg_match("/<a#(.*)#a>/isU", $string, $pregMatch)) { 

            $arrayString = explode('#',$pregMatch[1]); 

            $content = '<img src="./html/system/img/link.gif" alt="link.gif"> <a href="'. str_replace('\?','?',$arrayString[0]) .'" title="'.$arrayString[1].'" name="'.$arrayString[1].'" accesskey="'.$arrayString[2].'">'.$arrayString[1].'</a><br>'; 

echo $pregMatch[1].'<br>';

            $string = preg_replace("/<a#".$pregMatch[1]."#a>/isU",$content,$string); 

        } 


gibt folgendes aus

homepage.php5#Homepage#h
index.php5#Test#t
homepage.php5\?do=newPage#Neue Page#1
homepage.php5\?do=newPage#Neue Page#1
homepage.php5\?do=newPage#Neue Page#1
homepage.php5\?do=newPage#Neue Page#1
homepage.php5\?do=newPage#Neue Page#1
homepage.php5\?do=newPage#Neue Page#1
... (wiederholung unendlich da er ja nicht ersetzt...)

Ich schau mit mal preg_quote() an ...

Edit: das (=) könnte noch das Problem Darstellen ...

Spezielle Zeichen regulärer Ausdrücke sind: . \ + * ? [ ^ ] $ ( ) { } = ! < > | :

[DasMööp]

Serh schön ... die Function ist mal sehr sehr Hilfreich


Also wie folgt Funktioniert es:

PHP-Code:

        while (preg_match("/<a#(.*)#a>/isU", $string, $pregMatch)) {

            $arrayString = explode('#',$pregMatch[1]);

            $content = '<img src="./html/system/img/link.gif" alt="link.gif"> <a href="'. str_replace('\?','?',$arrayString[0]) .'" title="'.$arrayString[1].'" name="'.$arrayString[1].'" accesskey="'.$arrayString[2].'">'.$arrayString[1].'</a><br>';

            $string = preg_replace("/<a#".preg_quote($pregMatch[1])."#a>/isU",$content,$string);

        }

        return $string; 


Ich danke für den Tip !

[MrNiceGuy]

Keine Ursache, nur wo ich mir das jetzt alles so anschaue, fällt mir auf, dass folgender Code sehr wahrscheinlich deutlich schneller ist:

PHP-Code:

$string = str_replace ($pregMatch[0], $content, $string); 


Das pregMatch[0] steht dabei für den kompletten gefundenen String, wodurch bei einer Änderung der Codes (z.B. wenn du anstatt <a#....#a> <link#...#link> nutzen willst) nurnoch der RegEx geändert werden muss. Jedenfalls sollte die Funktion str_replace gegenüber der preg_replace schon deutlich schneller sein und zusätzlich fällt auch noch das preg_quote() weg.
Du kannst übrigens mal versuchen, ob es auch ohne die Ersetzung von ? in \? funktioniert, würde dir zusätzlich 2 Funktionsaufrufe sparen und ich gehe mal davon aus, dass du das probiert hast, um den preg_replace() zum Laufen zu bekommen!? *kratzamkopf*

[DasMööp]

Mit str_replace hast Du vollkommen recht habe das mal eben zum Bastelln so hingeklatscht...

PHP-Code:

class test {

    private $content = null;
    private $string = null;
    private $array = null;

    public function __construct() {
        $this->test_preg_Match();
    }
    
    public function test_preg_Match() {

        $this->string = "[§cms_boxen.php5?do=newBox#Neue Box#1]";
        $this->string .= "[§cms_boxen.php5?do=newBox#Neue Box#1]";

        while (preg_match("/\[§(.*)\]/isU", $this->string, $pregMatch)) {

            $this->array = explode('#',$pregMatch[1]);
            
            $this->content = '<a href="'.$this->array[0].'" title="'.$this->array[1].'" name="'.$this->array[1].'" accesskey="'.$this->array[2].'">'.$this->array[1].'</a><br>';

            $this->string = str_replace($pregMatch[0],$this->content,$this->string);

        }

        echo $this->string;

    }
    
}

$test = new test(); 


Ich denke so ist das minimum erreicht ... ausser man würde in dem preg
gleich die inhalte zwischen den # abfragen. Sehe das aber als Problem wenn
es mehr oder weniger Inhalte im String sind.

[§cms_boxen.php5?do=newBox#Neue Box#1]
[§cms_boxen.php5?do=newBox#Neue Box]

Meine so ist es schon gut oder fällt Dir noch was ein !?

[MrNiceGuy]

Uff, sicher fallen mir da einige Möglichkeiten ein, aber du musst doch wissen, wie du es für am sinnvollsten hälst, schließlich musst du mit dem Funktionsumfang zufrieden sein und wenn es alles passt, dann lass es doch einfach so, ich sehe in der Verarbeitungsweise jedenfalls keine nennenswerten Nachteile

Einziger Punkt, über den du nachdenken solltest ist, dass wenn du mal Links mit Ankern benötigst, dass du ein anderes Trennzeichen nehmen solltest, denn dann brauchst du selbst in der URL die Raute. Prozent sind aber auch wieder nicht sinnig, da diese zum maskieren von Sonderzeichen usw. genutzt werden, vielleicht solltest du lieber auf die PIPE (|) umsteigen als Trennzeichen!? Ist aber auch nur dann notwendig wie gesagt, wenn du mal Anker benötigst...

Jetzt ist mir doch noch ein Punkt eingefallen

[DasMööp]

Wenn man das genau nimmt absolut richtig habe mich für Pipe entschieden.