nicht angemeldet
Hallo,
Die Schreibfunktion der Guestbook.php sieht so aus :
function write_input($new_name, $new_input,$bookfile){
include ("BILDER/check.inc");
$new_input = check($new_input);
$fd = fopen($bookfile,"a");
$new_input = str_replace("\n","<br>",$new_input);
Das geht auch schon in check(), warum hier nochmal den ganzen Text durchkauen?
fwrite ($fd, "<b>".$new_name.": (" . strftime("%d.%m.%Y", time()) . ")</b><br>".$new_input."\r\n");
fclose ($fd);
}
Die Funktion Check in der check.inc sieht so aus :
<?PHP
function check($txt){
$bookfile = "check.txt";
$fd = fopen($bookfile,"r");
Datei öffnen, lesen, schließen kostet alles Zeit. Warum nicht hardcoden?
(Ich bin zwar auch gegen hardcoden, aber hier kann man's machen)
$boa = 0;
while (!feof($fd)) {
Dadurch würdest Du hier über ein simples Array loopen, statt über eine Datei.
$zeile = fgets($fd,1024);
Würde wegfallen.
$bob = 0;
$zeile = ereg_replace("\r\n","",$zeile);
Würde wegfallen.
if (strchr($zeile,"[")<>"") {$bob = 1;$boa = 1;}
if (strchr($zeile,"]")<>"") {$bob = 1;$boa = 0;}
if (($bob == 0)&&($boa == 0)) $dic = $zeile;
Würde wegfallen (Warum hast Du kein Switch genommen?)
if (($bob == 0)&&($boa == 1)) $txt = ereg_replace("#" . trim($zeile) . "#","<img src="http://mitglied.lycos.de/hermannguensti/Others/ROT/BILDER/" . $dic . "/" . $zeile . ".gif" border="0">",$txt);
}
Müßte dann etwas angepaßt werden. Würde ich dann über einen großen Switch() laufen lassen. Soviel ist es nicht, da die Zeit zum Parsen einen Einfluß nehmen könnte. (Falls nicht eh schon gecached wird)
fclose ($fd);
Würde wegfallen.
return $txt;
}
?>
Uns die die check.txt sieht in etwa so aus :
Würde dann auch wegfallen
Man könnte das ganze zwar extrem kürzen wenn man stadt #Name# gleich den <img src="..../Name"> angeben würde, jedoch wäre das nicht so nutzerfreundlich die #Name# methode.
Das kannst Du dann behalten ;-)
Warum nimmst Du eigentlich nicht direkt #name# ?
so short
Christoph Zurnieden