[PHP][MYSQL]Problem z dodawaniem wartosci do ksiazki telefonicznej Opcje

[Drokal]

Witam
Chce stworzyć ksiazkę telefoniczną, w której bedzie można dodawać i usuwać wartości. Problem w tym ze nie wszystko działa tak jak trzeba - nic nie dodaje, nic nie usuwa

[PHP] pobierz, plaintext 
<?PHP
ini_set( 'display_errors', 'On' );
error_reporting( E_ALL );
?>
<?php
mysql_connect ("localhost", "root", "aaa") or die ("Fak ju");
mysql_select_db ("ksiazka") or die ("fakju2");
if (isset($_POST['action']) && isset($_POST[$action]) =='add')
{
	$imie_f=$_POST['imie'];
	$nazw_f=$_POST['nazwisko'];
	$tele_f=$_POST['telefon'];
		if ( $imie_f && $nazw_f && $tele_f )
		{
		$query="INSERT INTO 'ksiazkatelefoniczna' (imie,nazwisko,telefon) VALUES ('$imie_f', '$nazw_f', '$tele_f')";
		$wynik=mysql_query($query) or die(mysql_error());
			if ($wynik)
			{
			print ("Dodales poprawnie dane");
			}
			else
			{ print ("nie dodales danych cioto"); }
			}
			else
			{print ("Wypełnij wszystko cioto!");
			}
			}
elseif (isset($_GET['action']) == 'skasuj')
{
	$wynik = mysql_query ("DELETE FROM ksiazkatelefoniczna WHERE nr='".$_GET['id']."';");
}
$wynik = mysql_query ("SELECT * FROM ksiazkatelefoniczna;") or die ("blad");
print ("<TABLE CELLPADDING=5 BORDER=1>");
print ("<TR><TD>Imię</TD><TD>Nazwisko</TD>");
print ("<TD>telefon</TD><TD>Akcja</TD></TR>");
while ($rekord = mysql_fetch_assoc($wynik))
{
$nr = $rekord['nr'];	
$imie = $rekord['imie'];
$nazwisko = $rekord['nazwisko'];
$telefon = $rekord['telefon'];
print ("<TR><TD>$imie</TD><TD>$nazwisko</TD>");
print ("<TD>$telefon</TD>");
print ("<TD><a href=\"dodawanie.php?action=skasuj&id=nr\" TITLE=\"Skasuj wpis!\">usuń</a>");
print ("<TD><TR>");
 
}
print ("</TABLE>");
print '<FORM METHOD="POST"><Nowy rekord:
<INPUT TYPE="hidden" NAME="action" VALUE="add">
<TABLE>
<TR><TD>Imie:</TD><TD><INPUT TYPE="text" NAME="imie"></TD></TR>
<TR><TD>Nazwisko:</TD><TD><INPUT TYPE="text" NAME="nazwisko"></TD></TR>
<TR><TD>Telefon:</TD><TD><INPUT TYPE="text" NAME="telefon"></TD></TR>
</TABLE>
<INPUT TYPE="submit" VALUE="Dodaj">
</FORM>';
?>
[PHP] pobierz, plaintext 

Nie rozumiem czemu nie da sie usuwac wartości mimo ze we wczesniejszej fazie kodu nie było z tym problemu, oto próbka

[PHP] pobierz, plaintext 
<?php
mysql_connect ("localhost", "root", "aaa") or die ("Fak ju");
mysql_select_db ("ksiazka") or die ("fakju2");
if(isset($_GET['action'])&& $_GET['action']=='skasuj')
{
$wynik=mysql_query("DELETE FROM ksiazkatelefoniczna WHERE nr='".$_GET['id']."';");
}
$wynik = mysql_query ("SELECT * FROM ksiazkatelefoniczna;") or die ("blad");
print ("<TABLE CELLPADDING=5 BORDER=1>");
print ("<TR><TD>Imię</TD><TD>Nazwisko</TD>");
print ("<TD>Telefon</TD><TD>Akcja</TD></TR>");
while ($rekord = mysql_fetch_assoc($wynik))
{
$nr = $rekord['nr'];	
$imie = $rekord['imie'];
$nazwisko = $rekord['nazwisko'];
$telefon = $rekord['telefon'];
print ("<TR><TD>$imie</TD><TD>$nazwisko</TD>");
print ("<TD>$telefon</TD>");
print ("<TD><a href=wyswietlanie.php?action=skasuj&id=$nr\" TITLE=\"Skasuj wpis!\">Wypiernicz</a>");
print ("<TD><TR>");
}
print ("</TABLE");
?>
[PHP] pobierz, plaintext 

Nie wiem czy to ja nie dostrzegam jakiegoś małego szczegółu czy co, wiem ze kombinuje z tym juz sporo, wiele tematów naczytalem i dalej nic

[b4rt3kk]

[PHP] pobierz, plaintext 
<?PHP
ini_set( 'display_errors', 'On' );
error_reporting( E_ALL );
?>
<?php
mysql_connect ("localhost", "root", "aaa") or die ("Fak ju");
mysql_select_db ("ksiazka") or die ("fakju2");
if (isset($_POST['action']) && isset($_POST[$action]) =='add') // ów warunek nigdy nie zostanie spełniony, ponieważ funkcja isset zwraca wartość true / false, nigdy nie będzie to add, cały kod zawarty w if nie zadziała
{
	$imie_f=$_POST['imie'];
	$nazw_f=$_POST['nazwisko'];
	$tele_f=$_POST['telefon'];
		if ( $imie_f && $nazw_f && $tele_f ) // ten warunek również nie ma sensu, ponieważ pod te zmienne przypisujesz wartości (zakładam, że są to string), które również nie zwracają true/false
		{
		$query="INSERT INTO 'ksiazkatelefoniczna' (imie,nazwisko,telefon) VALUES ('$imie_f', '$nazw_f', '$tele_f')"; // błędne zapytanie, niepotrzebne apostrofy przy nazwie tabeli
		$wynik=mysql_query($query) or die(mysql_error());
			if ($wynik)
			{
			print ("Dodales poprawnie dane");
			}
			else
			{ print ("nie dodales danych cioto"); }
			}
			else
			{print ("Wypełnij wszystko cioto!");
			}
			}
elseif (isset($_GET['action']) == 'skasuj') // jak wyżej, warunek nigdy nie jest spełniony, gdyż zwracane wartości isset to true/false, nigdy 'skasuj'
{
	$wynik = mysql_query ("DELETE FROM ksiazkatelefoniczna WHERE nr='".$_GET['id']."';"); // zupełnie niepotrzebnie nawydziwiałeś z łączeniem string-ów
}
$wynik = mysql_query ("SELECT * FROM ksiazkatelefoniczna;") or die ("blad");
print ("<TABLE CELLPADDING=5 BORDER=1>");
print ("<TR><TD>Imię</TD><TD>Nazwisko</TD>");
print ("<TD>telefon</TD><TD>Akcja</TD></TR>");
while ($rekord = mysql_fetch_assoc($wynik))
{
$nr = $rekord['nr'];	
$imie = $rekord['imie'];
$nazwisko = $rekord['nazwisko'];
$telefon = $rekord['telefon'];
print ("<TR><TD>$imie</TD><TD>$nazwisko</TD>");
print ("<TD>$telefon</TD>");
print ("<TD><a href=\"dodawanie.php?action=skasuj&id=nr\" TITLE=\"Skasuj wpis!\">usuń</a>"); // bardzo niebezpieczna praktyka w ten sposób przekazywać id rekordu do skasowania, nie przekazujesz i tak zmiennej w GET, a napis 'nr'
print ("<TD><TR>");
 
}
print ("</TABLE>");
print '<FORM METHOD="POST"><Nowy rekord:
<INPUT TYPE="hidden" NAME="action" VALUE="add">
<TABLE>
<TR><TD>Imie:</TD><TD><INPUT TYPE="text" NAME="imie"></TD></TR>
<TR><TD>Nazwisko:</TD><TD><INPUT TYPE="text" NAME="nazwisko"></TD></TR>
<TR><TD>Telefon:</TD><TD><INPUT TYPE="text" NAME="telefon"></TD></TR>
</TABLE>
<INPUT TYPE="submit" VALUE="Dodaj">
</FORM>';
?>
[PHP] pobierz, plaintext 

Poniższy kod będzie działać, choć zalecałbym Ci optymalizację.

[PHP] pobierz, plaintext 
<?PHP
ini_set( 'display_errors', 'On' );
error_reporting( E_ALL );
?>
<?php
mysql_connect ("localhost", "root", "aaa") or die ("Fak ju");
mysql_select_db ("ksiazka") or die ("fakju2");
if (isset($_POST['action']) && $_POST['action'] =='add') 
{
	$imie_f=$_POST['imie'];
	$nazw_f=$_POST['nazwisko'];
	$tele_f=$_POST['telefon'];
		if ( $imie_f != '' && $nazw_f != '' && $tele_f != '' ) 
		{
		$query="INSERT INTO ksiazkatelefoniczna (imie,nazwisko,telefon) VALUES ('$imie_f', '$nazw_f', '$tele_f')";
		$wynik=mysql_query($query) or die(mysql_error());
			if ($wynik)
			{
			print ("Dodales poprawnie dane");
			}
			else
			{ print ("nie dodales danych cioto"); }
			}
			else
			{print ("Wypełnij wszystko cioto!");
			}
			}
elseif (isset($_GET['action']) AND $_GET['action'] == 'skasuj')
{
	$wynik = mysql_query ("DELETE FROM ksiazkatelefoniczna WHERE nr='{$_GET['id']}'");
}
$wynik = mysql_query ("SELECT * FROM ksiazkatelefoniczna") or die ("blad");
print ("<TABLE CELLPADDING=5 BORDER=1>");
print ("<TR><TD>Imię</TD><TD>Nazwisko</TD>");
print ("<TD>telefon</TD><TD>Akcja</TD></TR>");
while ($rekord = mysql_fetch_assoc($wynik))
{
$nr = $rekord['nr'];	
$imie = $rekord['imie'];
$nazwisko = $rekord['nazwisko'];
$telefon = $rekord['telefon'];
print ("<TR><TD>$imie</TD><TD>$nazwisko</TD>");
print ("<TD>$telefon</TD>");
print ("<TD><a href=\"dodawanie.php?action=skasuj&id=$nr\" TITLE=\"Skasuj wpis!\">usuń</a>");
print ("<TD><TR>");
 
}
print ("</TABLE>");
print '<FORM METHOD="POST"><Nowy rekord:
<INPUT TYPE="hidden" NAME="action" VALUE="add">
<TABLE>
<TR><TD>Imie:</TD><TD><INPUT TYPE="text" NAME="imie"></TD></TR>
<TR><TD>Nazwisko:</TD><TD><INPUT TYPE="text" NAME="nazwisko"></TD></TR>
<TR><TD>Telefon:</TD><TD><INPUT TYPE="text" NAME="telefon"></TD></TR>
</TABLE>
<INPUT TYPE="submit" VALUE="Dodaj">
</FORM>';
?>
[PHP] pobierz, plaintext