Witam, jest to mój pierwszy większy kod wykonany w całości przeze mnie, postanowiłem zrobić księgę gości opartą na plikach tekstowych, mógłby ktoś przeanalizować kod i powiedzieć co zrobiłem źle?
Jak już podpatrzę jak to zrobić to jeszcze dodam tam, że gdy liczba komentarzy jest większa niż 5 niech tworzy drugą stronę i na drugiej stronie pokazuje kolejne 5, na 3 kolejne 5 itd

Dodawanie nowych wpisów

[PHP] pobierz, plaintext 
<?php
	// Utworzenie krótkich nazw zmiennych
	$nick = trim($_POST['nick']);
	$komentarz = trim($_POST['komentarz']);
	$DOCUMENT_ROOT = $_SERVER['DOCUMENT_ROOT'];
	$ip = $_SERVER['REMOTE_ADDR'];
	$data = date('H:i, j F Y');
	$komentarz = htmlspecialchars($komentarz);
	$nick = htmlspecialchars($nick);
 
		if($nick == '')
		{
			$nick = 'Gość';
		}
 
		if($komentarz == '')
	{
		echo "Proszę o wpisanie komentarza";
		exit;
	}
	echo '<hr/>';
	echo 'Nick: '.$nick.'<br />';
	echo 'IP: '.$ip.'<br />';
	echo 'Komentarz: '.$komentarz.'<br />';
 
	$ciagwyjsciowy = $data."\t".$nick."\t".$ip."\t".$komentarz."\n";
 
		//otwarcie pliku w celu dopisywania
	$wp = fopen("DOCUMENT_ROOT/../ksiega/ksiega.txt", 'ab');
	if(!wp)
	{
		echo 'Nie można teraz dopisać się do księgi gości.';
		exit;
	}
 
	fwrite($wp, $ciagwyjsciowy, strlen($ciagwyjsciowy));
	flock($wp, LOCK_UN);
	fclose($wp);
	echo '<br />Twój wpis został dodany!<br />';
 
?>
[PHP] pobierz, plaintext 

Wyświetlanie wpisów

[PHP] pobierz, plaintext 
<?php
 $wpisy = file("DOCUMENT_ROOT/../ksiega/ksiega.txt");
  if(!wpisy)
  {
  	echo 'Nie można wczytać wpisów!';
  }
  else
  {
  	$ilosc_wpisow = count($wpisy);
  	if($ilosc_wpisow == 0)
  	{
  		echo 'Brak wpisów! Bądź pierwszy!';
  		exit;
  	}
  	echo ' Wszystkich wpisów '.$ilosc_wpisow.'';
  		for ($i=0; $i < $ilosc_wpisow; $i++) {
	//Rozbicie każdej linii
	$linia = explode( "\t", $wpisy[$i]);
 
 
  	echo "<table width=\"600px\" border=\"1\" cellpadding=\"5px\" cellspacing=\"0px\">
	<tr style=\"background-color: #CCCCFF;\" height=\"30px\"><td width=\"100px\">Nick: $linia[1]  </td><td width=\"100px\">Data: $linia[0]</td><td width=\"100px\">IP: $linia[2]</td></tr>
	<tr><td colspan=\"3\" style=\"text-align:left;\">$linia[3] </td></tr>
</table>";
 
 
 
  }
}
 
 
?>
[PHP] pobierz, plaintext 

Edit:Jeszce teraz wpadłem na pomysł, aby w przyszłości wprowadzić tam możliwość kasowania pojedynczych wpisów.

Nie ma za bardzo co oceniać. ale dam Ci kilka sugestii. Przede wszystkim:

[PHP] pobierz, plaintext 
	if($nick == '')
{
$nick = 'Gość';
}
 
if($komentarz == '')
{
echo "Proszę o wpisanie komentarza";
exit;
}
[PHP] pobierz, plaintext 

Użyj empty.


Zamiast date lepiej zapisać time
Zamiast fopen, fwrite itd użyj file_get_contents, file_put_contents;

[PHP] pobierz, plaintext 
	$nick = trim($_POST['nick']);
$nick = htmlspecialchars($nick);
[PHP] pobierz, plaintext 

Można zrobić w 1 linii:

[PHP] pobierz, plaintext 
$nick = trim(htmlspecialchars($_POST['nick']))
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$ciagwyjsciowy = $data."\t".$nick."\t".$ip."\t".$komentarz."\n";
[PHP] pobierz, plaintext 

Źle łączysz. Jeśli separatorem ma być tab, to musisz wykluczyć go z zawartości którą wprowadza użytkownik albo użyć innego separatora.

[PHP] pobierz, plaintext 
 $wpisy = file("DOCUMENT_ROOT/../ksiega/ksiega.txt");
if(!wpisy)
[PHP] pobierz, plaintext 

1. if(!$wpisy)
2. file wczytuje Ci do tablicy po nowej linii, co jeśli komentarz będzie takie znaki zawierał?

Brakuje jakiejkolwiek kontroli nad spamowaniem.

Jak na pierwszy w życiu skrypt, nie ma tragedii ;-p