problem z skryptem, wygryzanko Opcje

[Szymon80]

Witam.

Mam taki oto kod skryptu wygryzanko

[PHP] pobierz, plaintext 
<?php
// plik $file chmod 666
$file="wpis.txt";
// przypisujemy nazwe pliku
// gdzie bedą przechowywane dane
$fd = fopen ("$file", "r");
$dane = fread ($fd, filesize ("$file"));
$tmp=explode("|",$dane);
// Wyświetlamy wpis z wygryzanka
echo "<a href=\"$tmp[0]\">$tmp[1]</a> &nbsp;( $tmp[2] )";
// Wyświetlamy formualrz wygryzanka
echo "
<form method=\"post\" action=\"\">
<fieldset style=\"border:0px;\">
<input type=\"hidden\" name=\"wygr\" value=\"tak\" />
<input type=\"text\" name=\"link\" style=\"width: 100px\" value=\"http://\" />&nbsp;
<input type=\"text\" name=\"opis\" style=\"width: 100px\" maxlength=\"50\" value=\"opis\" />
&nbsp;<input type=\"submit\" name=\"submit\" value=\"dodaj\" />
</fieldset>
</form>";
// Zapisujemy dane do pliku:
if ($wygr == 'tak') {
if ($opis != 'opis') { 
$ip = $REMOTE_ADDR;
// przypisanie zmniennej $file nazwy pliku
$file="wpis.txt";
// uchwyt pliku, otwarcie do dopisania 
$fp = fopen($file, "w"); 
// blokada pliku do zapisu 
flock($fp, 2); 
$k = '|';
// zapisanie danych do pliku 
fwrite($fp, "$link $k $opis $k $ip");
// odblokowanie pliku 
flock($fp, 3); 
// zamknięcie pliku 
fclose($fp); 
 }
 // w przypadku nie wpisania opisu
 else echo "Musisz wpisać opis dla strony!";
 }
?>
[PHP] pobierz, plaintext 

ale wywala mi taki oto błąd.

[PHP] pobierz, plaintext 
Warning: fread() [function.fread]: Length parameter must be greater than 0 in /home/symeon/public_html/seokatalog/test/wygr.php on line 7
 ( )
[PHP] pobierz, plaintext 

Z tego rozumiem ze parametr "fread()" musi byc dłuższy niż 0. Ale jak to naprawić ?

[krowal]

[PHP] pobierz, plaintext 
<?php #
$dane = fread ($fd, filesize ("$file")); ?>
[PHP] pobierz, plaintext 

W tej linii funkcja filesize() zwraca Ci 0 lub false (i jeśli masz wyłączone pokazywanie błędów typu E_WARNING to tego nie zauważysz). Po drugie jaki jest sens pisania czegoś takiego: "$file" (mam na myśli pakowanie pojedynczej zmiennej w zbędne cudzysłowy)

[Szymon80]

To nie jest skrypt mojego autorstwa, znalzłem go w sieci. Jak powinien wyglądać ten skrypt aby działał poprawnie ?

[Makciek]

1. echo "<a href='.$tmp[0].'>'.$tmp[1].'</a> &nbsp;( '.$tmp[2].' )";

gdy dajesz w echo PHP musisz dodać kropeczki, i lepiej dodać pojedynczy apostrof niż \"

[thek]

Lepiej użyj funkcji pobierających cały plik jak file_get_contents czy podobnych. Masz przykłady działających podobnie w manualu przy tej funkcji. Oczywiście sens stosowania innych jest tylko w przypadku gdy masz serwer gdzie wspomnianej przeze mnie funkcji nie ma