Witam wszystkich to mój pierwszy post na forum więc proszę mnie oszczędzić

mam stronę na której są dwa formularze

1. dodawanie danych do bazy(cały czas widoczny)
2. edycja danych (pojawia sie dopiero po kliknieciu w link do edycji )

pierwszy formularz działa dobrze, drugi w połowie bo pojawiają się odpowiednie dane do edycji, jednak jak wprowadzę zmiany i klikne w przycisk popraw to i tak zadziała pierwszy submit i zamiast edytowac dane formularz dodaje nowy rekord

na pewno mógłbym to rozwiązać gdyby formularze odwoływały sie do osobnych skryptów, jednak podoba mi się jak działa to wszystko na jednej stronie i chciałbym aby tak zostało, jednak sam nie potrafię zmusić tego do działania.

To jest mój pierwszy kontakt z PHP (ok 5 dni z tym walcze) więc proszę o wyrozumiałość.

załączam fragment kodu.

[HTML] pobierz, plaintext 
 <form action="" method="post">
 
    url: <input type="text" name="url" />  słowo kluczowe:<input type="text" name="keyword" /> <input type="submit" value="Dodaj" name="submit"/><br /><br/>
 
    </form>
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
	$url = $_POST['url'];
	$keyword = $_POST['keyword'];
 
if($url and $keyword) {
 
    // dodajemy rekord do bazy
    $ins = @mysql_query("INSERT INTO static_data SET url='$url', keyword='$keyword', added_date='1234567890', active='1'");
 
    if($ins) echo "Rekord został dodany poprawnie";
    else echo "Błąd nie udało się dodać nowego rekordu";
 
}
 
?>
 
 <?php
    $a = trim($_GET['a']);
    $id = trim($_GET['id']);
 
    if($a == 'del' and !empty($id)) {
 
    /* usuwamy rekord */
    mysql_query("DELETE FROM static_data WHERE id='$id'")
    or die('Błąd zapytania: '.mysql_error());
 
    echo 'Rekord został usunęty z bazy';
} 
 
	if($a == 'edit' and !empty($id)) {
    /* zapytanie do tabeli */
    $wynik = mysql_query("SELECT * FROM static_data WHERE id='$id'")
    or die('Błąd zapytania');
    /* wyświetlamy wyniki, sprawdzamy, czy zapytanie zwróciło wartość większą od 0 */
    if(mysql_num_rows($wynik) > 0) {
         /* odczytujemy zawartość wiersza z tabeli */
        $lista = mysql_fetch_assoc($wynik);
        /* wczytujemy dane do formularza */
        /* w formularz znajdują się ukryte pola "a" z wartością "save" i pole "id" z wartością zmiennej id */
        echo '<form action="" method="post">
        <input type="hidden" name="a" value="save" />
        <input type="hidden" name="id" value="'.$id.'" />
        url:<input type="text" name="url" value="'.$lista['url'].'" />	keyword:<input type="text" name="keyword" value="'.$lista['keyword'].'" />	active:<input type="text" name="active" value="'.$lista['active'].'" /> <input type="submit" value="popraw" />
        </form>';
    }
}
		elseif($a == 'save') {
		/* odbieramy zmienne z formularza */
		$id = $_POST['id'];
		$url = trim($_POST['url']);
		$keyword = trim($_POST['keyword']);
		$active = trim($_POST['active']);
 
		/* uaktualniamy tabelę */
		mysql_query("UPDATE static_data SET url='$url', keyword='$keyword', active='$active' WHERE id='$id'")
		or die('Błąd zapytania');
		echo 'Dane zostały zaktualizowane';
}
 
// pobierabie wartości z tabeli
   $sql = "SELECT id, url, keyword FROM static_data WHERE active = 1 ORDER BY url ASC";
   $rezultat = mysql_query($sql) or die(mysql_error());
 
 
// wyświetlanie rezultatu na ekranie
	  echo "<table cellpadding=\"2\" border=1>";
	  echo "<tr>";
	  echo "<th>DOMENA</th>";
	  echo "<th>SŁOWO KLUCZOWE</th>";
	  echo "<th>SZCZEGÓŁY</th>";
	  echo "<th>EDYCJA</th>";
	  echo "<th>USUŃ</th>";
	  echo "</tr>";
   while ($lista = mysql_fetch_array($rezultat)) {
   	  echo "<tr>";
	  echo "<td>".$lista['url']."</td>";
	  echo "<td>".$lista['keyword']."</td>";
	  echo "<td align=\"center\"><a href=\"stats.php?s=".$lista['id']."\">info</a></td>";
	  echo "<td align=\"center\"><a href=\"stats.php?a=edit&amp;id=".$lista['id']."\">edytuj</a></td>";
	  echo "<td align=\"center\"><a href=\"stats.php?a=del&amp;id=".$lista['id']."\">usuń</a></td>";
	  echo "</tr>";
	  }
      echo "</table>";
 
 
?>
[PHP] pobierz, plaintext 

kod to zlepek z tutoriali i kursów które przerobiłem w ostatnim czasie więc prosze nie krytukować jakości kodu, chociaż wszystkie sugestie mile widziane.

Głowy uciąć nie dam (jestem zmęczony itd.) wydaje mi się, że chodzi o tą linijkę kodu
[

[PHP] pobierz, plaintext 
 
<?php
echo '<form action="" method="post">
?>
[PHP] pobierz, plaintext 

w pełnym kodzie jest to linijka nr 40. Powinno być tak: (głowy uciąć, że zadziała nie dam ;>)

[PHP] pobierz, plaintext 
<?php
echo '<form action="twoj_skrypt.php?a=save" method="post">
?>
[PHP] pobierz, plaintext 

Na stronie może być i 100 formularzy, wszystko zależy od organizacji kodu.

Najprościej jest je rozróżniać po submicie, a konkretnie po jego nazwie.

przykładowo

[HTML] pobierz, plaintext 
<form action="" method="post">
<input type="text" name="xyz" />
<input type="submit" value="Dodaj" name="dodaj"/>
</form>
 
<form action="" method="post">
<input type="text" name="xyz" />
<input type="submit" value="Dodaj" name="edytuj"/>
</form>
 
<form action="" method="post">
<input type="text" name="xyz" />
<input type="submit" value="Dodaj" name="usun"/>
</form>
 
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
if(isset($_POST['dodaj'])){
 
//... tutaj akcja gdy wysłano formularz dodawania
 
}
 
if(isset($_POST['edytuj'])){
 
//... tutaj akcja gdy wysłano formularz edycji
 
}
 
if(isset($_POST['usun'])){
 
//... tutaj akcja gdy wysłano formularz odp. za usunięcie
 
}
[PHP] pobierz, plaintext 

-oczywiście nazwy tych submitów muszą być unikalne! (nie mogą występować nigdzie indziej w formularzach na tej stronie)

@ZuyPan - niestety nie pomogło

@neverever - wygląda obiecującoale tam już mam sporo ifów i elseifów, także niestety musze przeczytać jeszcze kilka manuali, ale dzięki za nakierowanie

Każdemu submitowi nadaj inny name. Poza tym submit z value='popraw' nie ma u Ciebie name w ogole.

@sadistic_son - niestety nie dało to również żadnego efektu

a czy to Tobie pomoże?

[PHP] pobierz, plaintext 
<?php
if(!isset($_POST['sub1'])){$_SESSION['akcja_1']='celbar';}
else{$_SESSION['akcja_1']=$_POST['sub1'];}
if(!isset($_POST['sub2'])){$_SESSION['akcja_2']='celbar';}else {$_SESSION['akcja_2']=$_POST['sub2'];}
if(!isset($_POST['sub3'])){$_SESSION['akcja_3']='celbar';}else{$_SESSION['akcja_3']=$_POST['sub3'];}
 
 
echo '<form method="post" action="tenformularz.php">';
if($_SESSION['akcja_1']=='DALEJ'){echo 'tutaj róbta co chceta-------------------';}else{}
 echo '<br>';
 echo '<input type="text" name="okno1">';
 
 echo '<br>';
 echo '<input type="submit" name="sub1" value="DALEJ" >';
 echo '<br>';
 echo '</form>';
 //-----------------------------------------------------------
 echo '<form method="post" action="tenformularz.php">';
 if($_SESSION['akcja_2']=='EDYTUJ'){echo 'tutaj wpisz kod edycji';}else{}
 if($_SESSION['akcja_3']=='ZAPISZ'){echo 'tutaj wpisz kod zapisu danych';}else{}
 
echo '<br>';
 echo '<input type="text" name="okno2">';
 echo '<input type="submit" name="sub2" value="EDYTUJ" >';
 echo '<input type="submit" name="sub3" value="ZAPISZ" >';
 
 
 echo '</form>';
 
 ?>
 
 
 
 
 
 
 
[PHP] pobierz, plaintext 

daj znać

Przecież obie te akcje (dodanie i edycja) to jedno i to samo różniące się raptem tylko jedną daną! Robisz formularz z wszystkimi polami jak trzeba i dodajesz jedno pole typu hidden z wartością równą id. Jeśli podczas otwierania nie będzie go -> dajesz 0 i wiesz, że to coś nowego. Jeśli masz owe id, wiesz, że to edycja. Po stronie bazy jedynie sprawdzasz czy nie ma już identycznego podczas sprawdzania "na wszelki wypadek" (nigdy nie ufaj danym usera). Wtedy wystarczy JEDEN submit dla wszystkiego. Jeśli przy submit owo pole ukryte o name = "id" ma value = 0 -> INSERT, jeśli większe od 0 -> UPDATE. Oczywiście po drodze sprawdzasz czy nie ma w bazie już takiego dubla. Pozbywasz się jednak wiele niepotrzebnego kodu. A tak jak radzą poprzednicy, najprościej użyć sprawdzenia po nazwie.
Często spotykam się z błednymi sprawdzeniami typu if( isset( !$_POST ) ) co jest dziwne, bo zmienna ta jest zawsze ustawiona Dlatego sprawdzam tak if( !empty( $_POST ) ) a zaraz potem czy aby wcisnęli właściwy guzik if( array_key_exists( 'name_submita', $_POST ) ) to Ci ujawni, czy klepnięto w interesujący Cię. Nie można bowiem kliknąc dwóch jednocześnie.Do serwera i tak dojdzie tylko jeden (poza przypadkami z modyfikacją danych post, ale to inna bajka ). Wrzucanie kilku submitów do forma i oddanie tego userowi znającemu się na rzeczy to proszenie o kłopoty. Najlepiej niech jest tylko jeden dla jednego kontrolera, a unikniesz problemów wielu. Stąd proponuję Ci usunięcie jednego i pójście w stronę odpowiedniej kontroli po stronie serwera.