Forum PHP.pl

Witam wszystkich.
Nie do końca wiedziałem jaki tytuł wybrać, ale postaram się opisać mój problem. Potrzebuje wykonać małą, prostą aplikacje do zarządzania biblioteką - proste funkcje. Nie poradziłem sobie z wypożyczaniem książki. Sytuacja wygląda tak: w bazie danych mam 3 tabele: czytelnicy(id,imie,nazwisko,pesel), ksiazki (id,tytul,autor) i wypozyczenia(id_wyp,id_ksiazki i id_czytelnika) - wszystko jest ze sobą połączone i działa poprawnie(przez phpmyadmina wpisałem przykładowe wypożyczenia i działa). W aplikacji mam dwie listy rozwijane, które pobierają dane z moich tabel oraz pola typu radio. Przykładowo: z listy wybieram Jana Kowalskiego i Pana Tadeusza, zaznaczam opcje wypożycz no i zatwierdzam - książka powinna być wypożyczona, ale nie wiem(albo nie umiem tego zrobić bo szukałem sporo w internecie) połączyć czytelnika i książki z wypożyczeniem.
Pokaże prawie cały kod, to będzie chyba najprosciej:

[PHP] pobierz, plaintext 
<?php
require "connection.php";
connection();
?>
</div>
<div id="tabela">
<?php
	$qc = mysql_query ('Select * FROM CZYTELNICY'); 
	echo "<select name = 'czytelnicy'></option>";	
	while($wiersz_danych = mysql_fetch_array($qc))
			{
			echo "<option value = $wiersz_danych[ID_CZYTELNIKA]>$wiersz_danych[IMIE] $wiersz_danych[NAZWISKO] </option>";
			}
	echo "</select>";
 
	$qk = mysql_query ('Select * FROM KSIAZKI'); 
	echo "<select name = ksiazka value=''></option>";
		while($wiersz_danych = mysql_fetch_array($qk))
				{
				echo "<option value = $wiersz_danych[ID_KSIAZKI]>$wiersz_danych[TYTUL] - $wiersz_danych[AUTOR]</option>";
				}
	echo "</select>";
?>
 
<?php
if ($_SERVER['REQUEST_METHOD'] == 'POST'){
	$w_status = 'unchecked';
	$wyp_status = 'unchecked';
	$k_status = 'unchecked';
	$odd_status = 'unchecked';
	$ww_status = 'unchecked';
 
	if (isset($_POST['Wykonaj'])){
	$selected_radio = $_POST['button'];
		}
	}
	?>
<br>
<form action= "index.php" method="POST">
	<input type="Radio" name="button" value="wszyscy" <?php print $w_status ; ?>>Wyświetl czytelniów </br> 
	<input type="Radio" name="button" value="ksiazki" <?php print $k_status; ?>>Wyświetl wszystkie książki </br>
	<input type="Radio" name="button" value="wypozyczenie" <?php print $wyp_status ; ?>>Wypożycz </br>
	<input type="Radio" name=:"button" value="oddanie"<?php print $odd_status; ?>>Oddaj</br>
	<input type="Radio" name="button" value="ww" <?php print $ww_status; ?>>Wyswietl wypozyczenia</br>
	<input type="submit" name="Wykonaj" value="Wykonaj"><br><br></form>
 
		<?php
			if($selected_radio == 'wszyscy'){
			$w_status = 'checked';
 
			$query = "SELECT * FROM czytelnicy";
			$dane = mysql_query($query) or die ("wystapily problemy");
			echo "<table><tr><td>Nazwisko</td><td>Imie</td><td>Pesel</td></tr>";	
				while($wiersz_danych = mysql_fetch_array($dane)){
					echo  "<tr>";
					echo "<td>".$wiersz_danych[NAZWISKO]."</td>";
					echo "<td>".$wiersz_danych[IMIE]."</td>";
					echo "<td>".$wiersz_danych[PESEL]."</td>";
				}
		echo "</tr></table>";
		}
 
	if($selected_radio =='ksiazki'){
		$k_status = 'checked';
 
				$query = "SELECT * FROM KSIAZKI";
				$dane = mysql_query($query) or die("wystapily problemy");
					echo "<table><tr><td>Tytul</td><td>Autor</td></tr>";
						while($wiersz_danych = mysql_fetch_array($dane)){
							echo "<tr>";
							echo "<td>".$wiersz_danych[TYTUL]."</td>";
							echo "<td>".$wiersz_danych[AUTOR]."</td>";
							}
					echo "</tr></table>";
		}
 
		if($selected_radio =='wypozyczenie'){
		$wyp_status = 'checked';
				$query = "INSERT INTO WYPOZYCZENIA VALUES (NULL, '.$_POST[czytelnicy]', '.$_POST[ksiazka]') ";
						echo "Wybrano: ".$_POST['czytelnicy']; 		
		}
 
	if($selected_radio == 'ww'){
			$ww_status='checked';
 
			$query = "SELECT C.NAZWISKO,C.IMIE, K.TYTUL	FROM WYPOZYCZENIA W 
				LEFT JOIN CZYTELNICY C ON C.ID_CZYTELNIKA = W.ID_CZYTELNIKA
				LEFT JOIN KSIAZKI K ON K.ID_KSIAZKI = W.ID_KSIAZKI";
				$dane = mysql_query($query) or die("wystapily problemy");
				echo "<table><tr><td>Nazwisko</td><td>Imie</td><td>Tytul</td></tr>";	
		while($wiersz_danych = mysql_fetch_array($dane)){
			echo  "<tr>";
			echo "<td>".$wiersz_danych[NAZWISKO]."</td>";
			echo "<td>".$wiersz_danych[IMIE]."</td>";
			echo "<td>".$wiersz_danych[TYTUL]."</td>";			
		}
	echo "</tr></table>";
	}
?>
</div>
 
[PHP] pobierz, plaintext 

Możliwe, że coś w końcu namieszałem bo troszke zmieniałem kod szukając rozwiązania. Liczę na wasze wskazówki:)

Pozdrawiam

A wiec...

1) po co Ci w 17. linijce value='' ? Przeciez Twoje value ma byc pobrane z listy rozwijanej, tj. ID

2) dlaczego w 79. linijce wstawias ID, ID_CZYTELNIKA, ID_KSIAZKI zamiast ID, ID_KSIAZKI, ID_CZYTELNIKA? Wyzej podales inna kolejnosc kolumn

3) co robia linijki 25. - 37.?

4) nigdy nie uzywalem $_SERVER['REQUEST_METHOD'] == 'POST', co najwyzej:

[PHP] pobierz, plaintext 
if(count($_POST) > 0) {
  // blok instrukcji
}
[PHP] pobierz, plaintext 

Ale nie neguje Twojego rozwiazania, po prostu nie wiem jak sie zachowuje

5) Dlaczego dolaczasz LEFT JOINem tabele czytelnikow i wypozyczen? A co jesli 1 czytelnik ma wiele wypozyczen? Wtedy jest relacja 1-do-wiele i potrzebny jest RIGHT JOIN. Natomiast jesli chodzi o tabele ksiazki... to... tak - LEFT JOIN, zgadza sie. Dolaczasz do tabeli wypozyczenia, bo w 1 wypozyczeniu masz 1 ksiazke, czyli relacja 1-do-1. Chociaz niektorzy powiedza, ze w 1 wypozyczeniu mozna miec kilka ksiazek, ale z tego co zrozumialem, Twoja aplikacja pozwala na wypozyczenie 1 ksiazki, a potem ewentualnie powrot do poprzedniej strony i wypozyczenie na tego samego uzytkownika innej pozycji. ALE kazda taka operacja traktowana jest w bazie jako osobny wpis/rekord.

W JOIN'ach moglem sie pomylic, gdyz pisze ten post na szybko (IMG:style_emoticons/default/wink.gif)

Najlepiej bedzie jesli otworzysz phpMyAdmin'a, otworzysz okienko zapytania i poprobujesz tak, zeby Ci ladna tabelka wyszla ze wszystkimi danymi jakich sobie zyczysz. Przy RIGHT JOIN, o ile mnie pamiec nie zawodzi, sa powtorzenia rekordow i wtedy przydaje sie polecenie GROUP BY. W kazdym razie jak wyjdzie Ci wszystko w okienku zapytania to juz polowa sukcesu (IMG:style_emoticons/default/smile.gif)

6) w celach porzadkowych:
konstrukcje takie jak...

[PHP] pobierz, plaintext 
<?php print $w_status ; ?>
[PHP] pobierz, plaintext 

...mozesz z powodzeniem zastapic...

[PHP] pobierz, plaintext 
<?=$w_status;?>
[PHP] pobierz, plaintext 

Powodzenia (IMG:style_emoticons/default/smile.gif)

Ten post edytował Kaloryfer 3.11.2011, 02:05:28

Dzięki za zainteresowanie. Zastosowałem się do wskazówek, ale w dalszym ciągu nie rozwiązuje to mojego problemu. Przy wybraniu opcji wypożycz, wypożyczenie nie zostaje dodane do tabeli wypozyczenia.

[PHP] pobierz, plaintext 
		if($selected_radio =='wypozyczenie'){
		$wyp_status = 'checked';
				$query = "INSERT INTO WYPOZYCZENIA VALUES (NULL, '.$_POST[ksiazka]', '.$_POST[czytelnicy]') ";
						echo "Wybrano: ".$_POST['czytelnicy']; 		
		}
[PHP] pobierz, plaintext 

Reszta kodu, trochę zmieniłem, według wskazówek:

[PHP] pobierz, plaintext 
<?php
	$qc = mysql_query ('Select * FROM CZYTELNICY'); 
	echo "<select name = 'czytelnicy'></option>";	
	while($wiersz_danych = mysql_fetch_array($qc))
			{
			echo "<option value = $wiersz_danych[ID_CZYTELNIKA]>$wiersz_danych[IMIE] $wiersz_danych[NAZWISKO] </option>";
			}
	echo "</select>";
 
	$qk = mysql_query ('Select * FROM KSIAZKI'); 
	echo "<select name = 'ksiazka' ></option>";
		while($wiersz_danych = mysql_fetch_array($qk))
				{
				echo "<option value = $wiersz_danych[ID_KSIAZKI]>$wiersz_danych[TYTUL] - $wiersz_danych[AUTOR]</option>";
				}
	echo "</select>";
?>
 
 
<br>
<form action= "index.php" method="POST">
	<input type="Radio" name="button" value="wszyscy" <?=$w_status ; ?>Wyświetl czytelniów </br> 
	<input type="Radio" name="button" value="ksiazki" <?=$k_status; ?>Wyświetl wszystkie książki </br>
	<input type="Radio" name="button" value="wypozyczenie" <?=$wyp_status ; ?>Wypożycz </br>
	<input type="Radio" name=:"button" value="oddanie"<?=$odd_status; ?>Oddaj</br>
	<input type="Radio" name="button" value="ww" <?=$ww_status; ?>Wyswietl wypozyczenia</br>
	<input type="submit" name="Wykonaj" value="Wykonaj"><br><br></form>
 
		<?php
 
if (count($_POST) > 0) {
 
	if (isset($_POST['Wykonaj'])){
	$selected_radio = $_POST['button'];
		}
 
 
		if($selected_radio =='wypozyczenie'){
		$wyp_status = 'checked';
				$query = "INSERT INTO WYPOZYCZENIA VALUES (NULL, '.$_POST[ksiazka]', '.$_POST[czytelnicy]') ";
						echo "Wybrano: ".$_POST['czytelnicy']; 		
		}
}
 
[PHP] pobierz, plaintext 

Ten post edytował mihal 3.11.2011, 09:21:59

Nie wnikam w kod, nie czytam tematu (bo za długi (IMG:style_emoticons/default/wink.gif) ) ale sam chyba przyznasz, że takie coś:
<select name = ksiazka value=''></option>
to delikatnie rzecz mówiąc jest źle (IMG:style_emoticons/default/wink.gif)

1) To chcesz powiedziec, ze nic nie dodaje do tabeli? Zadnego rekordu, nawet z brakujacymi danymi? A nie masz bledu w zapytaniu? Zobacz sobie or die i mysl_error()

2) <br /> a nie </br> (IMG:style_emoticons/default/smile.gif)

Dokładnie, nie dodaje rekordu do tabeli wypożyczenia. Mogłem od początku o tym mówić - może byłoby prościej ale wydaje mi się, że błąd jest tutaj:

[PHP] pobierz, plaintext 
$query = "INSERT INTO WYPOZYCZENIA VALUES (NULL, '.$_POST[ksiazka]', '.$_POST[czytelnicy]') ";
[PHP] pobierz, plaintext 

Czy to faktycznie pobiera z listy rozwijanej tytuł książki i czytelnika...

Zmieniłem tak jak pisałeś na

[PHP] pobierz, plaintext 
$query = "INSERT INTO WYPOZYCZENIA VALUES (NULL, '.$_POST[ksiazka]', '.$_POST[czytelnicy]') ";
				$dane = mysql_query($query) or die('wystapily problemy'.mysql_error());
						echo "Wybrano: ".$_POST['czytelnicy']; 		
[PHP] pobierz, plaintext 

i dalej nic się nie wyświetla ani nic się nie dodaje. Nie ma błędu w zapytaniu.

Pozdrawiam

a daj

[PHP] pobierz, plaintext 
print_r($_POST);
[PHP] pobierz, plaintext 

zamiast wywoływać zapytanie do BD

coś takiego: ?

[PHP] pobierz, plaintext 
if($selected_radio =='wypozyczenie'){
		$wyp_status = 'checked';
				$query = "INSERT INTO WYPOZYCZENIA VALUES (NULL, '.$_POST[ksiazka]', '.$_POST[czytelnicy]') ";
				/*$dane = mysql_query($query) or die('wystapily problemy'.mysql_error());*/
						print_r($_POST);
						echo "Wybrano: ".$_POST['czytelnicy']; 		
		}
[PHP] pobierz, plaintext 

W takim wypadku w miejscu wyniku wyświetla:
Array ( [button] => wypozyczenie [Wykonaj] => Wykonaj ) Wybrano:

Przecież selecty z opcjami robisz poza formularzem... weź się zapoznaj wpierw z podstawami obsługi formularzy, a potem bierz się za coś trudniejszego.
I dodatkowo zapoznaj się z tematem:
Temat: Jak poprawnie zada pytanie

no to masz odpowiedź, w tablicy POST nie ma wybranych wartości z selectów. Usunąłeś, jak pisal nospor, zbędne </option>?

tak, usunąłem, teraz kod wygląda tak :

[PHP] pobierz, plaintext 
<?php
	$qc = mysql_query ('Select * FROM CZYTELNICY'); 
	echo "<select name = 'czytelnicy'>";	
	while($wiersz_danych = mysql_fetch_array($qc))
			{
			echo "<option value = $wiersz_danych[ID_CZYTELNIKA]>$wiersz_danych[IMIE] $wiersz_danych[NAZWISKO] </option>";
			}
	echo "</select>";
 
	$qk = mysql_query ('Select * FROM KSIAZKI'); 
	echo "<select name = 'ksiazka' >";
		while($wiersz_danych = mysql_fetch_array($qk))
				{
				echo "<option value = $wiersz_danych[ID_KSIAZKI]>$wiersz_danych[TYTUL] - $wiersz_danych[AUTOR]</option>";
				}
	echo "</select>";
?>
 
<form action= "index.php" method="POST">
	<input type="Radio" name="button" value="wszyscy" <?=$w_status ; ?>Wyświetl czytelniów <br/> 
	<input type="Radio" name="button" value="ksiazki" <?=$k_status; ?>Wyświetl wszystkie książki <br/>
	<input type="Radio" name="button" value="wypozyczenie" <?=$wyp_status ; ?>Wypożycz <br/>
	<input type="Radio" name=:"button" value="oddanie"<?=$odd_status; ?>Oddaj<br/>
	<input type="Radio" name="button" value="ww" <?=$ww_status; ?>Wyswietl wypozyczenia<br/>
	<input type="submit" name="Wykonaj" value="Wykonaj"><br><br></form>
 
		<?php
 
if (count($_POST) > 0) {
 
	if (isset($_POST['Wykonaj'])){
	$selected_radio = $_POST['button'];
		}
 
			if($selected_radio == 'wszyscy'){
			$w_status = 'checked';
 
			$query = "SELECT * FROM czytelnicy";
			$dane = mysql_query($query) or die ("wystapily problemy");
			echo "<table><tr><td>Nazwisko</td><td>Imie</td><td>Pesel</td></tr>";	
				while($wiersz_danych = mysql_fetch_array($dane)){
					echo  "<tr>";
					echo "<td>".$wiersz_danych[NAZWISKO]."</td>";
					echo "<td>".$wiersz_danych[IMIE]."</td>";
					echo "<td>".$wiersz_danych[PESEL]."</td>";
				}
		echo "</tr></table>";
		}
 
	if($selected_radio =='ksiazki'){
		$k_status = 'checked';
 
				$query = "SELECT * FROM KSIAZKI";
				$dane = mysql_query($query) or die("wystapily problemy");
					echo "<table><tr><td>Tytul</td><td>Autor</td></tr>";
						while($wiersz_danych = mysql_fetch_array($dane)){
							echo "<tr>";
							echo "<td>".$wiersz_danych[TYTUL]."</td>";
							echo "<td>".$wiersz_danych[AUTOR]."</td>";
							}
					echo "</tr></table>";
		}
 
		if($selected_radio =='wypozyczenie'){
		$wyp_status = 'checked';
				$query = "INSERT INTO WYPOZYCZENIA VALUES (NULL, '.$_POST[ksiazka]', '.$_POST[czytelnicy]') ";
				/*$dane = mysql_query($query) or die('wystapily problemy'.mysql_error());*/
						print_r($_POST);
						echo "Wybrano: ".$_POST['czytelnicy']; 		
		}
 
	if($selected_radio == 'ww'){
			$ww_status='checked';
 
			$query = "SELECT C.NAZWISKO,C.IMIE, K.TYTUL	FROM WYPOZYCZENIA W 
				RIGHT JOIN CZYTELNICY C ON C.ID_CZYTELNIKA = W.ID_CZYTELNIKA
				LEFT JOIN KSIAZKI K ON K.ID_KSIAZKI = W.ID_KSIAZKI WHERE K.TYTUL IS NOT NULL";
				$dane = mysql_query($query) or die("wystapily problemy");
				echo "<table><tr><td>Nazwisko</td><td>Imie</td><td>Tytul</td></tr>";	
		while($wiersz_danych = mysql_fetch_array($dane)){
			echo  "<tr>";
			echo "<td>".$wiersz_danych[NAZWISKO]."</td>";
			echo "<td>".$wiersz_danych[IMIE]."</td>";
			echo "<td>".$wiersz_danych[TYTUL]."</td>";			
		}
	echo "</tr></table>";
	}
	}
?>
[PHP] pobierz, plaintext 

nadal selecty masz poza formem

Zacznij czytać co się do Ciebie pisze: napisałem ci wyraźnie: masz pola SELECT poza formem, wiec nic dziwngo ze ci nie działa.
To juz drugi raz w tym temacie jak ignorujesz co ci piszę. Nie wiem, może jesteś na mnie uczulony. Nie wnikam. Ale jak jeszcze raz będziesz tak to olewać, to zamknę temat. Skoro ty jestes na mnie uczulony to i ja ci nie będę dłużny (IMG:style_emoticons/default/tongue.gif)

[PHP] pobierz, plaintext 
<?php
	echo "<form action = '' method='post'>";
	$qc = mysql_query ('Select * FROM CZYTELNICY'); 
	echo "<select name = 'czytelnicy'>";	
	while($wiersz_danych = mysql_fetch_array($qc))
			{
			echo "<option value = $wiersz_danych[ID_CZYTELNIKA]>$wiersz_danych[IMIE] $wiersz_danych[NAZWISKO] </option>";
			}
	echo "</select>";
 
	$qk = mysql_query ('Select * FROM KSIAZKI'); 
	echo "<select name = 'ksiazka' >";
		while($wiersz_danych = mysql_fetch_array($qk))
				{
				echo "<option value = $wiersz_danych[ID_KSIAZKI]>$wiersz_danych[TYTUL] - $wiersz_danych[AUTOR]</option>";
				}
	echo "</select></form>";
?>
[PHP] pobierz, plaintext 

nie działa. coś źle robię ?

Przedszkole przedszkolem, ale zacytuje ci moj poprzedni tekst:

i zamknę temat. Wybacz, ale podstawy z obsługi formularzy naprawdę mógłbyś wpierw sam opanować. Na tym forum wymagamy odrobiny wkładu własnego, a znajomość podstaw obsługi formularzy do takiego "wkładu" się zalicza