Semaphor: Schriftgröße verändern mit PHP

Beitrag lesen

Hi,

Die PHP-Lösung:

<?php
$schrift=$_GET[schrift];
da fehlen Quotes.

Stimmt, funktioniert aber auch so.

if(!isset($schrift)){
    $schrift=1;
}
$datei=$_SERVER[PHP_SELF];
hier auch - außerdem sollte man sicherheitshalber 'SCRIPT_NAME' verwenden.

SCRIPT_NAME war mir bisher nicht bekannt. Ich lern aber gern dazu. :)
Was ist an SCRIPT_NAME sicherer als an PHP_SELF ?

$groesser=$datei."?schrift=".$schrift*1.5;
$kleiner=$datei."?schrift=".$schrift/1.5;
Aua! Ungeprüft ... Der Link auf die Seite mit ?siehst_mich_nicht kommt bestimmt irgendwann. ;-)

Ja sicher - hier ging's aber um das Prinzip.
Lies, was Tim bei der Thread-Eröffnung geschrieben hat.
Meinetwegen packen wir halt noch if(is_numeric($schrift)){...}  hinein,
und wenn wir das ganze schön ausbauen, kommt der Text in eine
mysql-Datenbank, damit Du sagst: "was, wenn magic_quotes=off steht
und Du Deinen Query-String nicht richtig escaped hast?".
Schließlich bringt ein bißchen sql-injektion die größte Gaudi, usw.

<body style='font-size: <?php echo $schrift ?>em'>
warum inline-Style und kein style-Element?

auch hier: um nur das Prinzip zu erklären.

<p>Beispieltext. Bla bla ...</p>
<a href='<?php echo $groesser ?>'>groesser</a><br/>
<a href='<?php echo $kleiner ?>'>kleiner</a>
Auch hier sollte eine Begrenzung rein sowie ein Link zur Standardschriftgröße.

Sicher, wenn Du ihm die gesamte Anwendung für eine
Produktiv-Umgebung schreiben willst, kannste es gern machen.

</body>
Ferner sollte die Einstellung gespeichert werden (am besten in einem Cookie), damit sie auch auf den Folgeseiten zur Verfügung steht.

(Gleiche Antwort meinerseits, wie 3 Zeilen höher)

<a href="javascript: schrift('groesser')">groesser</a><br/>
    <a href="javascript: schrift('kleiner')">kleiner</a>
Funktionslos - ohne Javascript sollten die Links nicht präsentiert werden.

...

freundliche Grüße
Ingo

Um nicht nur zwischen Deinen Zeilen zu schreiben:
Sicher hast Du recht.

All das sollte beachtet werden, wenn man eine Anwendung schreibt,
die nicht nur "Tach, ich teste mal, ob's geht" auf'n Bildschirm
bringen soll und würde ich das als ein Tool (*hüstel*) ins Netz
packen, dann würd ich mich bei so einer Korrektur auch in Grund
und Boden schämen.

Deswegen: bitte achte auf den Kontext.

Willst Du Tim die Struktur von dynamischen Webseiten erklären,
indem Du ihm ne Din A4-Seite voll Code präsentierst?

Gruß,
Semaphor