Optimierungsvorschläge?

12.01.2005, 11:51

hallo forum,

bei folgendem codeschnipsel dreht es sich um die navigation eines preisbereichs. die unterkategorie wird in $_GET['ss'] übergeben, inkludierung des gewählten bereiches erfolgt an späterer stelle.
es funktioniert alles wunderbar, nur frage ich mich ob es nicht optimiert werden kann oder schwachstellen hat...

PHP-Code:

  <?

    if ((!empty($_GET['ss'])) && (!(substr($_GET['ss'],0,-6)==$p_secs[0])))

        echo "<a class=\"kleiner\" href=\"?s=preise&amp;ss=".$p_secs[(array_search(substr($_GET['ss'],0,-6),$p_secs))-1]."preise&amp;flavor=".$flavor."\">&lt; </a>";

 
    echo ":: ";

 
    for ($i=0;$i<sizeof($p_secs);$i++) {

        if (substr($_GET['ss'],0,-6)==$p_secs[$i])

            echo "[b]".$p_descs[$i]."[/b] :: ";

        else

            echo "<a href=\"?s=preise&amp;ss=".$p_secs[$i]."preise&amp;flavor=".$flavor."\">".$p_descs[$i]."</a> :: ";

    }

    

    if ((!empty($_GET['ss'])) && (!(substr($_GET['ss'],0,-6)==$p_secs[sizeof($p_secs)-1])))

        echo "<a class=\"kleiner\" href=\"?s=preise&amp;ss=".$p_secs[(array_search(substr($_GET['ss'],0,-6),$p_secs))+1]."preise&amp;flavor=".$flavor."\">&gt; </a>";

 
    echo "</div>";

 
?>

mfg

edit:
für die ampersands kann ich nichts

scheint ja ein bug oder so zu sein

mepeisen · 12.01.2005, 11:55

Wie kommt denn das mit den & zustande? Sieht merkwürdig bzw. falsch aus, aber ich gehe mal von Copy&Paste- Problemen aus.

12.01.2005, 12:02

naja wenn ich zum beispiel if ((bedingung) && (bedingung)) als php-code einfüge,
erhalte ich ((bedingung) && (bedingung))...

wenn ich meinen beitrag editiere, sehe ich es wieder in der form ((bedingung) && (bedingung))...

probiers mal aus..

robo47 · 12.01.2005, 12:05

Zitat:

Zitat von mepeisen

Wie kommt denn das mit den & zustande? Sieht merkwürdig bzw. falsch aus, aber ich gehe mal von Copy&Paste- Problemen aus.

ist ein Bug im Codehighlighter, den Göke irgendwann mal fixen will, stand irgendwo im offtiopic schon

mfg
robo47

mepeisen · 12.01.2005, 12:09

achso. na dann.

12.01.2005, 12:11

und sonst?

irgendwelche ideen/vorschläge zu meinem code?

imported_Ben · 12.01.2005, 12:33

ja. mach ihn übersichtlicher.

12.01.2005, 14:12

super kommentar, hilft mir super weiter danke :wink:

imported_Ben · 12.01.2005, 14:26

Zitat:

Zitat von mistemef

super kommentar, hilft mir super weiter danke :wink:

sollte er.

nutze keine aliases.
warum veranstaltest du "escape-orgien" bei deinen 'echos'?
setze mal klammern bei if und else.
wieso <? anstatt <?php ?

short_open_tags lässt grüßen ^^

grüße ben.

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

12.01.2005, 11:51
Gast Beiträge: n/a	Optimierungsvorschläge? hallo forum, bei folgendem codeschnipsel dreht es sich um die navigation eines preisbereichs. die unterkategorie wird in $_GET['ss'] übergeben, inkludierung des gewählten bereiches erfolgt an späterer stelle. es funktioniert alles wunderbar, nur frage ich mich ob es nicht optimiert werden kann oder schwachstellen hat... PHP-Code: <? if ((!empty($_GET['ss'])) && (!(substr($_GET['ss'],0,-6)==$p_secs[0]))) echo "<a class=\"kleiner\" href=\"?s=preise&ss=".$p_secs[(array_search(substr($_GET['ss'],0,-6),$p_secs))-1]."preise&flavor=".$flavor."\">< </a>"; echo ":: "; for ($i=0;$i<sizeof($p_secs);$i++) { if (substr($_GET['ss'],0,-6)==$p_secs[$i]) echo "[b]".$p_descs[$i]."[/b] :: "; else echo "<a href=\"?s=preise&ss=".$p_secs[$i]."preise&flavor=".$flavor."\">".$p_descs[$i]."</a> :: "; } if ((!empty($_GET['ss'])) && (!(substr($_GET['ss'],0,-6)==$p_secs[sizeof($p_secs)-1]))) echo "<a class=\"kleiner\" href=\"?s=preise&ss=".$p_secs[(array_search(substr($_GET['ss'],0,-6),$p_secs))+1]."preise&flavor=".$flavor."\">> </a>"; echo "</div>"; ?> mfg edit: für die ampersands kann ich nichts scheint ja ein bug oder so zu sein


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

12.01.2005, 11:55
mepeisen Erfahrener Benutzer Registriert seit: 21.12.2004 Beiträge: 5.234 PHP-Kenntnisse: Fortgeschritten	Wie kommt denn das mit den & zustande? Sieht merkwürdig bzw. falsch aus, aber ich gehe mal von Copy&Paste- Problemen aus. __________________ www.php-maven.org PHP und Maven vereint: Build/Deploy/Produktion/Konfiguration, Projekt Management, CI, PHPUnit, zahlreiche Frameworks Twitter @ https://twitter.com/#!/mepeisen und Facebook @ http://t.co/DZnKSUih

12.01.2005, 12:02
Gast Beiträge: n/a	naja wenn ich zum beispiel if ((bedingung) && (bedingung)) als php-code einfüge, erhalte ich ((bedingung) && (bedingung))... wenn ich meinen beitrag editiere, sehe ich es wieder in der form ((bedingung) && (bedingung))... probiers mal aus..

12.01.2005, 12:09
mepeisen Erfahrener Benutzer Registriert seit: 21.12.2004 Beiträge: 5.234 PHP-Kenntnisse: Fortgeschritten	achso. na dann. __________________ www.php-maven.org PHP und Maven vereint: Build/Deploy/Produktion/Konfiguration, Projekt Management, CI, PHPUnit, zahlreiche Frameworks Twitter @ https://twitter.com/#!/mepeisen und Facebook @ http://t.co/DZnKSUih

12.01.2005, 12:11
Gast Beiträge: n/a	und sonst? irgendwelche ideen/vorschläge zu meinem code?

12.01.2005, 12:33
imported_Ben Erfahrener Benutzer Registriert seit: 18.09.2003 Beiträge: 13.598 PHP-Kenntnisse: Fortgeschritten	ja. mach ihn übersichtlicher. __________________ Webentwickler-Blog - Buchbesprechung PHP Design Patterns - SQL-Antipatterns

12.01.2005, 14:12
Gast Beiträge: n/a	super kommentar, hilft mir super weiter danke :wink: