Zapis danych z formularza do bazy mysql Opcje

[nevar]

Jestem raczej początkujący w tym.(IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Jak powinien wyglądać porawnie skrypt, który pobiera tekst z formularza i zapisuje w bazie mysql?

Mój wygląda tak:

[PHP] pobierz, plaintext 
<?
include ("baza.php");
 
$pyt = $_POST['pyt'];
$odpa = $_POST['odpa'];
$odpb = $_POST['odpb'];
$odpc = $_POST['odpc'];
$odppraw = $_POST['odppraw'];
 
mysql_connect($mysql_host, $mysql_login, $mysql_haslo);
mysql_select_db($mysql_baza);
$query = mysql_query("INSERT INTO quiz VALUES ('','$pyt', '$odpa', '$odpb','$odpc','$odppraw')");
$result = mysql_query($query);
if (mysql_affected_rows() == 1) {
echo '<p>Dodano do bazy</p>';
}
else
{
error_log(mysql_error());
echo '<p>Błąd! Spróbuj ponownie</p>';
}
 
print '
<form action="dodpyt.php" method="post">
<table>
<tr>
  <td align="right">Pytanie:&nbsp;</td>
  <td><textarea name="pyt" rows=5 cols=30 warp=phisical></textarea></td>
</tr>
<tr>
  <td align="right">Odpowiedź a:&nbsp;</td>
  <td><textarea name="odpa" rows=5 cols=30 warp=phisical></textarea></td>
</tr>
<tr>
  <td align="right">Odpowiedź b:&nbsp;</td>
  <td><textarea name="odpb" rows=5 cols=30 warp=phisical></textarea></td>
</tr>
<tr>
  <td align="right">Odpowiedź c:&nbsp;</td>
  <td><textarea name="odpc" rows=5 cols=30 warp=phisical></textarea></td>
</tr>
<tr>
  <td align="right">Poprawna odpowiedź to:&nbsp;</td>
  <td><input type="text" name="odppraw">&nbsp;- tu wpisz a,b lub c</td>
</tr>
<tr>
  <td colspan="2" align="right">
  <input type="submit" value="Zapisz" /></td>
</tr>
</table>
</form> ';
?>
[PHP] pobierz, plaintext 

Co powinienem tu zmienić??

[huntercs]

niepotrzebnie drukujesz kilogramy htmla jeżeli niema tam nic z php, i sprawdzanie czy dodano do bazy też nie najlepsze
ja bym to zrobił tak:

[PHP] pobierz, plaintext 
<?
include ("baza.php");
 
mysql_connect($mysql_host, $mysql_login, $mysql_haslo);
mysql_select_db($mysql_baza);
 
$pyt = $_POST['pyt'];
$odpa = $_POST['odpa'];
$odpb = $_POST['odpb'];
$odpc = $_POST['odpc'];
$odppraw = $_POST['odppraw'];
 
$add = mysql_query("INSERT INTO quiz(pyt, odpa, odpb, odpc, odppraw) VALUES('$pyt', '$odpa', '$odpb','$odpc','$odppraw')");
if ($add) echo '<p>Dodano do bazy</p>';
else { echo mysql_error(); echo '<p>Błąd! Spróbuj ponownie</p>'; }
?>
<form action="dodpyt.php" method="post">
<table>
<tr>
  <td align="right">Pytanie:&nbsp;</td>
  <td><textarea name="pyt" rows=5 cols=30 warp=phisical></textarea></td>
</tr>
<tr>
  <td align="right">Odpowiedź a:&nbsp;</td>
  <td><textarea name="odpa" rows=5 cols=30 warp=phisical></textarea></td>
</tr>
<tr>
  <td align="right">Odpowiedź b:&nbsp;</td>
  <td><textarea name="odpb" rows=5 cols=30 warp=phisical></textarea></td>
</tr>
<tr>
  <td align="right">Odpowiedź c:&nbsp;</td>
  <td><textarea name="odpc" rows=5 cols=30 warp=phisical></textarea></td>
</tr>
<tr>
  <td align="right">Poprawna odpowiedź to:&nbsp;</td>
  <td><input type="text" name="odppraw">&nbsp;- tu wpisz a,b lub c</td>
</tr>
<tr>
  <td colspan="2" align="right">
  <input type="submit" value="Zapisz" /></td>
</tr>
</table>
</form>
[PHP] pobierz, plaintext 

[revyag]

Od razu na starcie dostaniesz warningi o niezainicjalizowanych zmiennych.
Zrób tak:

[PHP] pobierz, plaintext 
<?php
if(isset($_POST["submit"])) {
	include ("baza.php");
	mysql_connect($mysql_host, $mysql_login, $mysql_haslo);
	mysql_select_db($mysql_baza);
	$pyt = $_POST['pyt'];
	$odpa = $_POST['odpa'];
	$odpb = $_POST['odpb'];
	$odpc = $_POST['odpc'];
	$odppraw = $_POST['odppraw'];
 
	$add = mysql_query("INSERT INTO quiz(pyt, odpa, odpb, odpc, odppraw) VALUES('$pyt', '$odpa', '$odpb','$odpc','$odppraw')");
	if ($add) {
		echo '<p>Dodano do bazy</p>';
	} else {
		echo mysql_error(); echo '<p>Błąd! Spróbuj ponownie</p>';
	}
} else {
?>
 <!--tutaj kod formularza-->
 <input type="submit" name="submit" />
<?php
}
?>
[PHP] pobierz, plaintext 

[nevar]

Dzięki revyag! Działa jak trzeba!! (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)