Meinung über ein Kommentarscript

[Simon]

Hi,
bin ansich noch Anfänger in PHP.
Wollte jetzt eine Möglichkeit, damit Benutzer einen Kommentar hinterlassen können. Da ich kein fertiges Script gefunden habe, was mit gefallen hat, hab ich selbst eins geschrieben.

Das Script funktioniert ansich perfekt, so wie ich mir es vorstelle.
Ich wollte nur fragen was daran vielleicht falsch ist oder was ich noch ändern sollte.

Hier mal das Script:

<?php  
	//Anzahl der Kommentare auslesen  
	$hostUrl = 'localhost';  
	$userName = 'xxx';  
	$password = 'xxx';  
	$connectID = mysql_connect($hostUrl, $userName, $password)  
		or die ("Tut mir leid, kann keine Verbindung zur Datenbank aufbauen");  
	mysql_select_db("xxx", $connectID)  
		or die ("Auswahl der Datenbank nich Möglich");  
	$query = "SELECT count(*) FROM comments";  
	$result = mysql_query($query)  
		or die ("MySQL-Fehler: " . mysql_error());  
	$anzahl = mysql_result($result,0);  
		  
	  
	if(isset($_POST['comm_send']))  
	{  
		$name = $_POST['name'];  
		$email = $_POST['email'];  
		$website = $_POST['website'];  
		$kommentar = $_POST['kommentar'];  
		//Validierung für die Eingaben einfügen  
		include("komm_vali.php");  
	  
		//Kommentare schreiben  
		function writeComment ($id, $name, $datum, $uhrzeit, $website, $email, $kommentar)  
		{  
			$datum = date('j. M Y');  
			$uhrzeit = date('H:i');  
			$connectID = mysql_connect($hostUrl, $userName, $password)  
				or die ("Tut mir leid, kann keine Verbindung zur Datenbank aufbauen");  
			mysql_select_db("xxx", $connectID)  
				or die ("Auswahl der Datenbank nich Möglich");  
			mysql_query ("INSERT INTO comments (id, autor, datum, uhrzeit, website, email, kommentar)  
			VALUES ('$id', '$name', '$datum', '$uhrzeit', '$website', '$email', '$kommentar')", $connectID)  
				or die ("Schreiben des Eintrags in die Datenbank nicht möglich");  
		}  
		if (!$error_msg_1 &&!$error_msg_2 &&!$error_msg_3 &&!$error_msg_4 &&!$error_msg_5 &&!$error_msg_6 &&!$error_msg_7)  
		{  
			writeComment($id, $name, $datum, $uhrzeit, $website, $email, $kommentar);  
			header ("Location: komm.php");  
		}  
	}  
  
	//Formular Ausgabe  
	echo "<div id=\"comments\">";  
	echo "<h2>$anzahl Kommentare bis jetzt</h2>";  
	//Kommetare auslesen  
	$connectID = mysql_connect($hostUrl, $userName, $password)  
		or die ("Tut mir leid, kann keine Verbindung zur Datenbank aufbauen");  
	mysql_select_db("xxx", $connectID)  
		or die ("Auswahl der Datenbank nich Möglich");  
	$query = "SELECT * FROM comments ";  
	$out = mysql_query($query,$connectID);  
	while ($row_out = mysql_fetch_array($out))  
	{  
		$ausgabe_Autor = $row_out['autor'];  
		$ausgabe_Datum = $row_out['datum'];  
		$ausgabe_Uhrzeit = $row_out['uhrzeit'];  
		$ausgabe_Website = $row_out['website'];  
		$ausgabe_Email = $row_out['email'];  
		$ausgabe_Kommentar = nl2br($row_out['kommentar']);  
		if($ausgabe_Website == "http://")  
		{  
			echo "<p>$ausgabe_Autor am $ausgabe_Datum um $ausgabe_Uhrzeit</p>";  
		}  
		else  
		{  
			echo "<p><a href=\"$ausgabe_Website\">$ausgabe_Autor</a> am $ausgabe_Datum um $ausgabe_Uhrzeit</p>";  
		}  
		echo "<div class=\"comm_cont\">";  
		echo "<p class=\"comment\"><br />";  
		echo "$ausgabe_Kommentar";  
		echo "</p>";  
		echo "</div>";  
	}  
  
	echo "<h2>Hinterlasse einen Kommentar</h2>";  
	echo "<form action=\"";$_SERVER['PHP_SELF'];echo "\" method=\"post\">";  
	  echo "<input type=\"text\" size=\"20\" id=\"name\" name=\"name\" value=\"$name\" />";  
	  echo "<label for=\"name\">Name</label><br />";  
	  echo "<input type=\"text\" size=\"20\" id=\"email\" name=\"email\" value=\"$email \" />";  
	  echo "<label for=\"email\">E-Mail</label><br />";  
	  echo "<input type=\"text\" size=\"20\" id=\"website\" name=\"website\" value=\""; if(isset($website)){echo $website;} else echo "http://\" />";  
	  echo "<label for=\"website\">Website</label><br />";  
	  echo "<textarea cols=\"45\" rows=\"10\" id=\"kommentar\" name=\"kommentar\" >$kommentar</textarea><br />";  
	  echo "<input type=\"submit\" value=\"Kommentar senden\" id=\"comm_send\" name=\"comm_send\" />";  
	echo "</form>  
  </div>";  
  ?>

Mfg
Simon