Skrypt dodawanie newsów Opcje

[radziowie]

Chcę stworzyć prosty skrypt newsów. Napisałem pewien kod i chodzi o to, żebym chciał aby formularz dokonywał wpisu dopiero po wciśnięciu przycisku "wyslij". Jak można zauważyć tutaj mamy dodawanie od razu po wczytaniu strony. Co możecie zaproponować? Co można tu przerobić/dodać?

[PHP] pobierz, plaintext 
<?
include ('../dodatki/bonus.php');
 
polacz_baza();
 
echo "<h1>NEWSY - DODAWANIE I WYŚWIETLANIE</H1>";
echo '<form action="wiadomosci.php" method="post">
Tytuł wiadomości: <input type="text" name="tytul" /><br />
Treść wiadomości: <textarea></textarea><br />
Autor: <input type="text" name="autor" /><br />
<input type="submit" value="wyślij" />
</form>';
 
$tytul = $_POST['tytul'];
$text = $_POST['text'];
$autor = $_POST['autor'];
$data = date('Y-m-j');
 
//dodawanie danych do bazy
$zapytanie = "INSERT INTO newsy VALUES (NULL, '".$tytul."', '".$text."', '".$autor."', '".$data."')";
$result = mysql_query($zapytanie);
 
 
 
//wyświetlenie danych z bazy
$wynik = mysql_query("SELECT * FROM newsy");
	echo '<table border="1" width="700"><tr><td width="30">ID</td><td>Tytuł</td><td width="170">text</td><td>autor</td><td>data</td></tr>';
WHILE ($rekord = mysql_fetch_assoc ($wynik)) {
$nrid = $rekord['id'];
$tytul = $rekord['tytul'];
$text = $rekord['text'];
$autor = $rekord['autor'];
$data = $rekord['data'];
 
	echo '<tr><td width="30">'.$nrid.'</td><td>'.$tytul.'</td><td>'.$text.'<td>'.$autor.'</td><td>'.$data.'</td></tr>';
	}
echo '</table>';
 
?>
 
</body>
</html>
[PHP] pobierz, plaintext 

[radziowie]

Dopiero zaczynam, ze tak powiem tworzyć. O błędach sql injection słyszałem a co to e-notice?

[markonix]

Dopiero zaczynam, ze tak powiem tworzyć. O błędach sql injection słyszałem a co to e-notice?

[PHP] pobierz, plaintext 
    ini_set( 'display_errors', 'On' ); 
    error_reporting( E_ALL );
[PHP] pobierz, plaintext 

To błędy mniejszej wagi (nie krytyczne) ale warto tak pisać aby nie było i takich błędów.
Włączysz je powyższym kodem.

Koniecznie dodaj walidacje lub dodaj np. addslashes bo skrypt może Ci się rozlecieć gdy dodasz w treści cudzysłowie.
Pomijam SQL INJECTION, o którym już kolega wyżej wspomniał (newsy domyślam się, że i tak dodajesz Ty więc zabezpieczenia mogą iść na drugi plan).

Ten post edytował markonix 1.11.2010, 17:35:51