[PHP][MySQL]Pobrane id z url metodą get podczas wys. formularza znika Opcje

[sinke]

Hej,
mam następujący problem...
chce dodać dane do bazy danych, ale potrzebuje id. I id jest przekazywane przez get z url i ulr wygląda tak:

Kod

http:[...]/poke_yoke_form_1.php?id=4

I jest ok, bo jest pobierane przez get id i kiedy daje do echo to się wyświetla. Natomiast kiedy wpisuje dodatkowe dane i daje przycisk "dodaj" (wysyłam formularz) to wszystko znika i zapytanie do bazy nie działa. Mam podobnie w innym kodzie i wszystko działa ok. Zapytania są dobrze napisane, bo sprawdzałam w bazie. Potrzebuje wiedzieć, w którym miejscu mam coś źle, albo czemu tak się dzieje?


Poniżej kod:

[PHP] pobierz, plaintext 
<?php
include_once('header.html');
 
if(isset($_GET['id']) && (is_numeric($_GET['id'])) ) 
{
	$id = $_GET['id'];
}
elseif (isset($_POST['id']) && (is_numeric($_POST['id'])) )
{
	$id = $_POST['id'];
}
else {
	echo "BŁĄD Z ID!";
}
 
echo $id;
echo "POST:". $id;
include_once('db_rys.inc.php');
 
 
if (isset($_POST['submitted']))
	{
 
			$q3 = "SELECT nazwa_rys FROM rysunek WHERE  id_rys = $id" ;
			echo $q3;		
			$r3 = mysqli_query ($link, $q3);
			$num3 = @mysqli_num_rows($r3);
			print_r($num3);
 
			if ($num3 == 1)
			{
				while ($row = mysqli_fetch_array($r3, MYSQL_ASSOC) )
							{
								$name =	$row['nazwa_rys'];
							}
 
				$q = "INSERT INTO  poke_yoke ('nazwa_rys', 'nazwa_art', 'kolory_art' ) VALUES ( '$name', '$nazwa_art', '$kolory_art' )";
										echo $q;
										$r = mysqli_query($link, $q);
 
										print_r($r);
 
										if($r)
									{
										echo "<p>Artykuł dodano poprawnie.</>";
 
 
									}
								else { 
										echo "<p>Wystapił blad.</p>";
										echo '<p>' . mysqli_error($link) . '<br/>Zapytanie: ' . $q . '</p>';
 
									 }
 
 
			}
			else
			{
				echo "Błąd nie wyszukano nazwy rysunka na podstawie id. ";
			}
 
	}
 
 
?>
 
<h3>Dodaj nowy artykuł:</h3>
 
<form method="post" action="poke_yoke_form_1.php" role="form" >
 
	<div class="form-group">
    	<label for="nazwa_art" class="pull-left">Nazwa artykułu do rysunku o nazie: <?php echo $name; ?></label>
	    <input type="text" name="nazwa_art" class="form-control" id="nazwa_art" placeholder="Nazwa art" value="<?php if(isset($_POST['nazwa_art'])) echo $_POST['nazwa_art']; ?>">
 
	</div>
 
	<div class="form-group">
    	<label for="kolory_art" class="pull-left">Podaj kolory odzielając je przecinkiem:</label>
	    <input type="text" name="kolory_art" class="form-control" id="kolory_art" placeholder="Kolory" value="<?php if(isset($_POST['kolory_art'])) echo $_POST['kolory_art']; ?>" >
 
	</div>
 
	 <div class="buttons">
 
    <!-- <input type="button" value="wróć" onclick="location.href='http://zamowienie.aleksandralysko.pl/wszystkie_zamowienia_zam.php'" > -->
    <input type="submit" name="submit" value="Dodaj artykuł" />
    <input type="hidden" name="submitted" value="TRUE" />
    </div>
 
 
</form>
 
<?php
mysqli_close($link);
?>  
 
[PHP] pobierz, plaintext 

Po przeładowaniu strony takie komunikaty mi wyskakują;

Kod

BŁĄD Z ID!SELECT nazwa_rys FROM rysunek WHERE id_rys =
Błąd nie wyszukano nazwy rysunka na podstawie id.

[kartin]

Jeśli użyjesz tego formularza w kodzie to id nie jest w żaden sposób przesyłane. Możesz dodać sobie w formularzu:

[HTML] pobierz, plaintext 
<input type="hidden" name="id" value="......">
[HTML] pobierz, plaintext 

W momencie gdy nie masz id i wyświetlasz komunikat BŁĄD Z ID! nie powinnaś wykonywać kodu w liniach 21-62. Jaki jest sens wykonywania tego?

[PHP] pobierz, plaintext 
while ($row = mysqli_fetch_array($r3, MYSQL_ASSOC) )
{
    $name = $row['nazwa_rys'];
}
[PHP] pobierz, plaintext 

W jakim celu jest użyta pętla?

[PHP] pobierz, plaintext 
$q = "INSERT INTO poke_yoke ('nazwa_rys', 'nazwa_art', 'kolory_art' ) VALUES ( '$name', '$nazwa_art', '$kolory_art' )";
[PHP] pobierz, plaintext 

Masz błąd w zapytaniu, a nawet 6 tego samego rodzaju.

[adbacz]

Nie pokazuje sie,ponieważ nie jest z tym formularzem wysyłana ta wartość. Ty ją masz tylko na tej stronie, ale jej nie wysyłasz. Musisz dodać to pole jako HIDDEN do formularza i będzie działać:

[PHP] pobierz, plaintext 
<input type="hidden" name="id" value="<?php echo $id; ?>" />
[PHP] pobierz, plaintext 

[sinke]

@adbacz, @kartin dzięki podziałało z hidden input i id działa.

@kartin
Pętle usunęłam, bo rzeczywiście była zbędna
Natomiast w tym zapytaniu chyba Ci chodziło, że błędem były te apostrofy, tylko użyłam ich po to, bo te trzy zmienne muszą byc w cudzysłowach, ponieważ są w nich _ i , i jak je wpisuje bez apostrofów w bazie to jest błąd więc usunęłam apostrofy z pierwszej części zapytania, przy zmiennych zostawiłam i wszystko działa.

Jeszcze raz dzięki, za szybką dopowiedz i pomoc !

[kartin]

Chodziło o apostrofy, ale w nazwach pól a nie w dodawanych wartościach.

[SQL] pobierz, plaintext 
INSERT INTO poke_yoke ('nazwa_rys', 'nazwa_art', 'kolory_art' )
[SQL] pobierz, plaintext 

[sinke]

Mam jeszcze jeszcze jeden problem(nawet trochę podobny ), chodzi mi, ze gdy update'uje stronę i mam w <form> method=post i action = ta sama strona, to wszystko działa okej, tzn. dodawaj mi się pola do tabelki i do bazy danych.
Ale jak daje w action inną stronę (bo nie chce, żeby można było przez przypadek wysłać ten formularz ponownie) to nie dodawanie do bazy danych nie działa i nie wiem czemu? Pewnie coś znowu przeoczyłam...
Liczę na Waszą pomoc.
Chciałabym tez spróbować html i php oddzielić, żeby w dwóch osobnych plikach to było, ale nie mam pomysłu i nie wiem czy w tym przypadku się da?

Kod poniżej:

[PHP] pobierz, plaintext 
 
<?php
ob_start();
include_once('header.html');
 
if(isset($_GET['id']) && (is_numeric($_GET['id'])) ) 
{
	$id = $_GET['id'];
}
elseif (isset($_POST['id']) && (is_numeric($_POST['id'])) )
{
	$id = $_POST['id'];
}
else {
	echo "BŁĄD Z ID!";
}
 
include_once('db_rys.inc.php');
 
$q1 = "SELECT * FROM  rysunek WHERE  id_rys = $id";	
$r1 = mysqli_query ($link, $q1);
$num1 = @mysqli_num_rows($r1);
 
	if ($num1 > 0)
	{
		while ($row = mysqli_fetch_array($r1, MYSQL_ASSOC) )
		{
			$num_py_m = $row['ile_py'];
			$nazwa_rys = $row['nazwa_rys'];
 
 
			echo '<div class="row">
				<h2>Nazwa RYSUNKU: '.$nazwa_rys.'</h2>
				<form method="post" action="poke_yoke_pokaz.php?id='.$id.'" role="form" >
					<div class="table-responsive">
          			<table class="table table-bordered">
						<tr>
							<th>Nr ID:</th>
							<th>Nazwa artykułu:</th>
			   				<th>Kolory w artykule:</th>';
 
			for ($i=1; $i <= $num_py_m; $i++) 
			{ 
			   	echo '<th class="text-center">'. $i.'</th>';
			}
 
			echo '</tr>';
 
			$nazwa_rys = $row['nazwa_rys'];			
			$q2 = "SELECT * FROM  poke_yoke WHERE  nazwa_rys = '$nazwa_rys' ORDER BY  id_py ASC  ";
			$r2 = mysqli_query ($link, $q2);
			$num2 = @mysqli_num_rows($r2);
			$num_py = $num_py_m + 3;
 
			if ($num2 > 0)
			{
				while ($row2 = mysqli_fetch_array($r2, MYSQL_NUM) )
				{			
					$id_py = $row2[0];
					echo '<tr>
							<td>' .$row2[0]. '</td> <!-- id_py -->
							<td>' .$row2[2]. '</td> <!-- nazwa_art -->
							<td>' .$row2[3]. '</td>';
 
					if(empty($row2[4]))
					{	
						for ($i=4; $i <= $num_py; $i++) 
						{ 	
							echo '<td class="text-center '. $row2[$i] .'">
									<div class="form-group">';
 
							$kolory = $row2[3];
							$tab_kolory = ( explode( ',', $kolory ) );
							array_unshift($tab_kolory, "x");
 
							echo '<select name="kolor_py_'.$i.'" class="form-control">';
 
							foreach ($tab_kolory as $kolor ) 
							{
								echo '<option value="'. $kolor .'">'. $kolor .'</option>';
							}
 
							echo '</select>
									</div>
								</td>';
 
							$select_kolor[$i] = $_POST['kolor_py_'.$i.''];
 
						}
					}
					else
					{
						for ($i=4; $i <= $num_py; $i++) 
						{
							$kolor_klasy = $row2[$i];
							if ($kolor_klasy == "C" || $kolor_klasy == "M" || $kolor_klasy == "Y" || $kolor_klasy == "K" || $kolor_klasy == "x" ) 
							{
								echo '<td class="text-center '. $kolor_klasy .'">'. $kolor_klasy .'</td>';
							}
							else
							{
							 	echo '<td class="text-center pantone">'. $kolor_klasy .'</td>';
							}	
						}
					}
				} //while	
				echo '</tr>
					</table>
				</div>';	
				$num_py_m; //ilosc pokey-yoke
 
				$k = 4;
				for ($j=1; $j <= 30 ; $j++) 
				{ 						
					${'select_py'.$j} = $select_kolor[$k];
					$k++;
				}
 
				echo '<input type="hidden" name="id_py" value="'.$id_py.'" />
					<input type="hidden" name="id" value="'.$id.'" />
							<div class="buttons">
								<input type="submit" name="submit" value="SPRAWDŹ" />
								<input type="hidden" name="submitted" value="TRUE" />
							</div>
					</form>';
	if (isset($_POST['submitted'])) {	
				$q3 = "UPDATE poke_yoke SET 
				py_1='$select_py1',
				py_2='$select_py2',
				py_3='$select_py3',
				py_4='$select_py4',
				py_5='$select_py5',
				py_6='$select_py6',
				py_7='$select_py7',
				py_8='$select_py8',
				py_9='$select_py9',
				py_10='$select_py10',
				py_11='$select_py11',
				py_12='$select_py12',
				py_13='$select_py13',
				py_14='$select_py14',
				py_15='$select_py15',
				py_16='$select_py16',
				py_17='$select_py17',
				py_18='$select_py18',
				py_19='$select_py19',
				py_20='$select_py20',
				py_21='$select_py21',
				py_22='$select_py22',
				py_23='$select_py23',
				py_24='$select_py24',
				py_25='$select_py25',
				py_26='$select_py26',
				py_27='$select_py27',
				py_28='$select_py28',
				py_29='$select_py29',
				py_30='$select_py30' WHERE id_py = $id_py";
				echo $q3;
				$r3 = mysqli_query($link, $q3);
 
				if($r3)
				{
					echo "<p>Dziękujemy za dodanie zamowienia. Zamówienie zostało dodano poprawnie.</p>";
					header('refresh: 1;');
 
				}	
				else 	
				{	 	
					echo "<p>Wystapił blad.</p>";	
					echo '<p>' . mysqli_error($link) . '<br/>Zapytanie: ' . $q3 . '</p>';
				}
			}
			}	
			else
			{
				echo '<p>BAZA POKE-YOKE: Nie ma nazwy takiego rysunku w bazie. <a class="btn btn-default" href="dodaj_rys.php">Dodaj nowy</a></p>';	
			}
		}//while	
	} //if ($num1 > 0)
	else
	{
		echo '<p>BŁĄD!</p>';
	}		
 
mysqli_free_result($r1);
mysqli_free_result($r2);		
mysqli_close($link);
 
include_once('footer.html');
?>
[PHP] pobierz, plaintext 

[kartin]

Ale jak daje w action inną stronę (bo nie chce, żeby można było przez przypadek wysłać ten formularz ponownie) to nie dodawanie do bazy danych nie działa i nie wiem czemu? Pewnie coś znowu przeoczyłam...

Jeśli w action dajesz adres strony/skryptu w którym nie ma dodawania to nie może dodać do bazy.

Pomysł oddzielenia logiki aplikacji od warstwy prezentacji jest zdecydowanie bardzo dobry. W tym jak i każdym innym przypadku da się to zrobić.

[PHP] pobierz, plaintext 
$q3 = "UPDATE poke_yoke SET
py_1='$select_py1',
py_2='$select_py2',
py_3='$select_py3',
...
py_28='$select_py28',
py_29='$select_py29',
py_30='$select_py30' WHERE id_py = $id_py";
[PHP] pobierz, plaintext 

To już jest prawdziwe przegięcie. Zdecydowanie powinnaś sobie poczytać coś o projektowaniu baz danych, postaciach normalnych itp.