Zapytanie złożone JOIN ON Opcje

[Panicz74]

Witam,

Mam taki problem. Musze przerobić swoje zapytania złożone na formę z INNER JOIN (poniekąd przy większej liczbie rekordów serwer się zawiesi). Zapytanie takie, które zrobiłem zwraca wszystkie wyniki:

[SQL] pobierz, plaintext 
"SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja FROM ogloszenia AS o 
				  JOIN users u ON u.id_usera = o.id_usera 
				  JOIN branza b ON b.id_branzy = o.id_branzy 
				  JOIN umowa i ON i.id_umowy = o.id_umowy 
				  JOIN lokalizacja l ON l.id_lok = o.id_lok"
[SQL] pobierz, plaintext 

A pojawiają się schody bo muszę teraz wybierać poszczególne elementy i średnio mi to wychodzi. Takie zapytanie również zwraca mi wszystkie wyniki zamiast tylko tych podanych w GET:

[SQL] pobierz, plaintext 
"SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja FROM ogloszenia AS o, branza AS b 
				  JOIN users u ON u.id_usera = o.id_usera 
				  JOIN branza b ON b.id_branzy = o.id_branzy 
				  JOIN umowa i ON i.id_umowy = o.id_umowy 
				  JOIN lokalizacja l ON l.id_lok = o.id_lok 
				  WHERE b.nazwa_branzy = '{$_GET['nazwa']}'"
[SQL] pobierz, plaintext 

Czy ktoś może mi podpowiedzieć jak przerobić to zapytanie aby wyciągnąć tylko te elementy podane w GET??

[nospor]

Po co dodales to:
, branza AS b
?
Przeciez masz juz to w JOIN...

[Panicz74]

Nic to nie zmieniło niestety...

[nospor]

Niemozliwe. Pokaz DOKLADNIE caly kod, pokaz co zawiera $_GET, pokaz jak wyglada wygenerowane zapytanie

[Panicz74]

Dziwne bo zapytanie w PhpMyAdmin ruszylo bez problemu. Skrypt niestety nie działa:

Część skryptu odpowiedzialna za to zapytanie:

[PHP] pobierz, plaintext 
if($_GET['nazwa'] =="elektromechanik" || $_GET['nazwa'] =="murarz" || $_GET['nazwa'] =="slusarz" || $_GET['nazwa'] =="kowal" && $_GET['umowa'] == "---" && $_GET['lokalizacja'] == "---")
		{
			try
			   {
				  $pdo = new PDO($pol, $user, $pass);
				  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
				  if(isset($_GET['nazwa']))
				  {
				  $stmt = $pdo->query("SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja FROM ogloszenia AS o 
				  JOIN users u ON u.id_usera = o.id_usera 
				  JOIN branza b ON b.id_branzy = o.id_branzy 
				  JOIN umowa i ON i.id_umowy = o.id_umowy 
				  JOIN lokalizacja l ON l.id_lok = o.id_lok 
				  WHERE b.nazwa_branzy = '{$_GET['nazwa']}'");
				  echo '<ul>';
				  foreach($stmt as $row)
				  {
					  echo '<li>'.$row['nazwa_branzy'].': '.$row['rodzaj_umowy'].': '.$row['login'].': '.$row['tresc'].': '.$row['lokalizacja'].'</li>';
				  }
				  $stmt->closeCursor();
				  echo '</ul>';
				  //print_r($_GET);
			   }
			   }
			   catch(PDOException $e)
			   {
				  echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
			   }
		}	
[PHP] pobierz, plaintext 

Cały index.php:

[PHP] pobierz, plaintext 
<?php
 
require_once 'connect.php';
// SELECT pusty - pokaż wszystko
if(empty($_GET['nazwa']) || $_GET['nazwa'] == "---" || $_GET['umowa'] == "---" || $_GET['lokalizacja'] == "---")
	{
			try
			   {
				  $pdo = new PDO($pol, $user, $pass);
				  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
				  $stmt = $pdo->query("SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja FROM ogloszenia AS o 
				  JOIN users u ON u.id_usera = o.id_usera 
				  JOIN branza b ON b.id_branzy = o.id_branzy 
				  JOIN umowa i ON i.id_umowy = o.id_umowy 
				  JOIN lokalizacja l ON l.id_lok = o.id_lok");
				  echo '<ul>';
				  foreach($stmt as $row)
				  {
					  echo '<li>'.$row['nazwa_branzy'].': '.$row['rodzaj_umowy'].': '.$row['login'].': '.$row['tresc'].': '.$row['lokalizacja'].'</li>';
				  }
				  $stmt->closeCursor();
				  echo '</ul>';
				  //print_r($_GET);
 
			   }
			   catch(PDOException $e)
			   {
				  echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
			   }
			}
 
		else if(!empty($_GET['nazwa']))
		{
					// SELECT nazwa - pokaż wszystko po nazwie
		if($_GET['nazwa'] =="elektromechanik" || $_GET['nazwa'] =="murarz" || $_GET['nazwa'] =="slusarz" || $_GET['nazwa'] =="kowal" && $_GET['umowa'] == "---" && $_GET['lokalizacja'] == "---")
		{
			try
			   {
				  $pdo = new PDO($pol, $user, $pass);
				  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
				  if(isset($_GET['nazwa']))
				  {
				  $stmt = $pdo->query("SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja FROM ogloszenia AS o 
				  JOIN users u ON u.id_usera = o.id_usera 
				  JOIN branza b ON b.id_branzy = o.id_branzy 
				  JOIN umowa i ON i.id_umowy = o.id_umowy 
				  JOIN lokalizacja l ON l.id_lok = o.id_lok 
				  WHERE b.nazwa_branzy = '{$_GET['nazwa']}'");
				  echo '<ul>';
				  foreach($stmt as $row)
				  {
					  echo '<li>'.$row['nazwa_branzy'].': '.$row['rodzaj_umowy'].': '.$row['login'].': '.$row['tresc'].': '.$row['lokalizacja'].'</li>';
				  }
				  $stmt->closeCursor();
				  echo '</ul>';
				  //print_r($_GET);
			   }
			   }
			   catch(PDOException $e)
			   {
				  echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
			   }
		}	   
		// SELECT nazwa, umowa - pokaż po nazwa && umowa
		else	if(!empty($_GET['nazwa'] =="elektromechanik" || $_GET['nazwa'] =="murarz" || $_GET['nazwa'] =="slusarz" || $_GET['nazwa'] =="kowal" &&
		$_GET['umowa'] =="umowa o prace" || $_GET['umowa'] =="umowa zlecenie" || $_GET['umowa'] =="umowa o dzielo" || $_GET['umowa'] =="umowa na czas okreslony") && $_GET['lokalizacja'] == "---")
			{
			try
			   {
				  $pdo = new PDO($pol, $user, $pass);
				  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
				  if(isset($_GET['nazwa']))
				  {
				  $stmt = $pdo->query("SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja FROM branza AS b, ogloszenia AS o, users AS u, umowa AS i, lokalizacja AS l 
				  WHERE b.nazwa_branzy = '{$_GET['nazwa']}' AND i.rodzaj_umowy = '{$_GET['umowa']}' 
				  AND u.id_usera = o.id_usera AND b.id_branzy = o.id_branzy AND i.id_umowy = o.id_umowy AND l.id_lok = o.id_lok");
				  echo '<ul>';
				  foreach($stmt as $row)
				  {
					  echo '<li>'.$row['nazwa_branzy'].': '.$row['rodzaj_umowy'].': '.$row['login'].': '.$row['tresc'].': '.$row['lokalizacja'].'</li>';
				  }
				  $stmt->closeCursor();
				  echo '</ul>';
				  //print_r($_GET);
			   }
			   }
			   catch(PDOException $e)
			   {
				  echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
			   }
			}
			// SELECT nazwa, umowa, lokalizacja - pokaż po nazwa && umowa && lokalizacja
			else	if(!empty($_GET['nazwa'] =="elektromechanik" || $_GET['nazwa'] =="murarz" || $_GET['nazwa'] =="slusarz" || $_GET['nazwa'] =="kowal" &&
		$_GET['umowa'] =="umowa o prace" || $_GET['umowa'] =="umowa zlecenie" || $_GET['umowa'] =="umowa o dzielo" || $_GET['umowa'] =="umowa na czas okreslony") && $_GET['lokalizacja'] == "Opole" || 
		$_GET['lokalizacja'] == "Krapkowice" || $_GET['lokalizacja'] == "Nysa" || $_GET['lokalizacja'] == "Olesno")
			{
			try
			   {
				  $pdo = new PDO($pol, $user, $pass);
				  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
				  if(isset($_GET['nazwa']))
				  {
				  $stmt = $pdo->query("SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja FROM branza AS b, ogloszenia AS o, users AS u, umowa AS i, lokalizacja AS l 
				  WHERE b.nazwa_branzy = '{$_GET['nazwa']}' AND i.rodzaj_umowy = '{$_GET['umowa']}' AND l.lokalizacja = '{$_GET['lokalizacja']}' 
				  AND u.id_usera = o.id_usera AND b.id_branzy = o.id_branzy AND i.id_umowy = o.id_umowy AND l.id_lok = o.id_lok");
				  echo '<ul>';
				  foreach($stmt as $row)
				  {
					  echo '<li>'.$row['nazwa_branzy'].': '.$row['rodzaj_umowy'].': '.$row['login'].': '.$row['tresc'].': '.$row['lokalizacja'].'</li>';
				  }
				  $stmt->closeCursor();
				  echo '</ul>';
				  //print_r($_GET);
			   }
			   }
			   catch(PDOException $e)
			   {
				  echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
			   }
			}
			//else {echo "SQL Injection Try Detected!!!";}
 
		//else {echo "SQL Injection Try Detected!!!";}
}
 
?>
[PHP] pobierz, plaintext 

Formularz:

[HTML] pobierz, plaintext 
<form action="index.php" method="get">
	<select name="nazwa" value="nazwa">
		<option><?php if(isset($_GET['nazwa'])) {echo ($_GET['nazwa']);} else {echo "---";} ?></option>
		<option>---</option>
		<option>elektromechanik</option>
		<option>murarz</option>
		<option>slusarz</option>
		<option>kowal</option>
	</select>
	<select name="umowa" value="umowa">
		<option><?php if(isset($_GET['umowa'])) {echo ($_GET['umowa']);} else {echo "---";} ?></option>
		<option>---</option>
		<option>umowa o prace</option>
		<option>umowa zlecenie</option>
		<option>umowa o dzielo</option>
		<option>umowa na czas okreslony</option>
	</select>
	<select name="lokalizacja" value="lokalizacja">
		<option><?php if(isset($_GET['lokalizacja'])) {echo ($_GET['lokalizacja']);} else {echo "---";} ?></option>
		<option>---</option>
		<option>Opole</option>
		<option>Krapkowice</option>
		<option>Nysa</option>
		<option>Olesno</option>
	</select>
	<input type="submit" name="submit" value="Zastosuj filtry">
	<a href="index.php">Odśwież</a>
</form>
[HTML] pobierz, plaintext 

[nospor]

A skad wiesz ze dziala w PMA? Wpisales recznie zapytanie do PMA, czy moze wkleiles do PMA to co generuje ci php? Bo jak recznie wpisales, to pewnie masz inaczej niz to co generuje ci php. Sprawdz, pisalem ci juz o tym

[Panicz74]

W PhpMyAdmin zapytanie takie przeszło:

[SQL] pobierz, plaintext 
SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja FROM ogloszenia AS o JOIN users u ON u.id_usera = o.id_usera JOIN branza b ON b.id_branzy = o.id_branzy JOIN umowa i ON i.id_umowy = o.id_umowy JOIN lokalizacja l ON l.id_lok = o.id_lok WHERE b.nazwa_branzy = "elektromechanik"
[SQL] pobierz, plaintext 

W skrypcie wygląda tak i pokazuje wszystko:

[SQL] pobierz, plaintext 
$stmt = $pdo->query("SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja FROM ogloszenia AS o 
				  JOIN users u ON u.id_usera = o.id_usera 
				  JOIN branza b ON b.id_branzy = o.id_branzy 
				  JOIN umowa i ON i.id_umowy = o.id_umowy 
				  JOIN lokalizacja l ON l.id_lok = o.id_lok 
				  WHERE b.nazwa_branzy = '{$_GET['nazwa']}'");
[SQL] pobierz, plaintext 

Formularz:

[HTML] pobierz, plaintext 
<select name="nazwa" value="nazwa">
		<option><?php if(isset($_GET['nazwa'])) {echo ($_GET['nazwa']);} else {echo "---";} ?></option>
		<option>---</option>
		<option>elektromechanik</option>
		<option>murarz</option>
		<option>slusarz</option>
		<option>kowal</option>
	</select>
[HTML] pobierz, plaintext 

Przed próbami wprowadzenia JOIN ON działało bez zarzutu...

[nospor]

Ile jeszcze raz mam powtarzac... sprawdz jak wyglada WYGENEROWANE zapytanie....

echo "SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja FROM ogloszenia AS o
JOIN users u ON u.id_usera = o.id_usera
JOIN branza b ON b.id_branzy = o.id_branzy
JOIN umowa i ON i.id_umowy = o.id_umowy
JOIN lokalizacja l ON l.id_lok = o.id_lok
WHERE b.nazwa_branzy = '{$_GET['nazwa']}'";
I to zapytanie wstaw do PMA

[Panicz74]

Za pomocą skryptu:

[PHP] pobierz, plaintext 
<form action="check.php" method="get">
	<select name="nazwa" value="nazwa">
		<option><?php if(isset($_GET['nazwa'])) {echo ($_GET['nazwa']);} else {echo "---";} ?></option>
		<option>---</option>
		<option>elektromechanik</option>
		<option>murarz</option>
		<option>slusarz</option>
		<option>kowal</option>
	</select>
 
	<input type="submit" name="submit" value="Zastosuj filtry">
	<a href="index.php">Odśwież</a>
</form>
 
<?php
 
require_once 'connect.php';
 
echo "SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja FROM ogloszenia AS o 
JOIN users u ON u.id_usera = o.id_usera 
JOIN branza b ON b.id_branzy = o.id_branzy 
JOIN umowa i ON i.id_umowy = o.id_umowy 
JOIN lokalizacja l ON l.id_lok = o.id_lok 
WHERE b.nazwa_branzy = '{$_GET['nazwa']}'";
 
?>
[PHP] pobierz, plaintext 

otrzymałem:

[SQL] pobierz, plaintext 
SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja FROM ogloszenia AS o JOIN users u ON u.id_usera = o.id_usera JOIN branza b ON b.id_branzy = o.id_branzy JOIN umowa i ON i.id_umowy = o.id_umowy JOIN lokalizacja l ON l.id_lok = o.id_lok WHERE b.nazwa_branzy = 'elektromechanik'
[SQL] pobierz, plaintext 

i po wklejeniu do PhpMyAdmin ruszyło:)
Tymczasem w skrypcie właściwym sprawa wygląda tak:

[PHP] pobierz, plaintext 
if($_GET['nazwa'] =="elektromechanik" || $_GET['nazwa'] =="murarz" || $_GET['nazwa'] =="slusarz" || $_GET['nazwa'] =="kowal" && $_GET['umowa'] == "---" && $_GET['lokalizacja'] == "---")
		{
			try
			   {
				  $pdo = new PDO($pol, $user, $pass);
				  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
				  if(isset($_GET['nazwa']))
				  {
				  $stmt = $pdo->query("SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja FROM ogloszenia AS o 
				  JOIN users u ON u.id_usera = o.id_usera 
				  JOIN branza b ON b.id_branzy = o.id_branzy 
				  JOIN umowa i ON i.id_umowy = o.id_umowy 
				  JOIN lokalizacja l ON l.id_lok = o.id_lok 
				  WHERE b.nazwa_branzy = '{$_GET['nazwa']}'");
 
				  echo '<ul>';
				  foreach($stmt as $row)
				  {
					  echo '<li>'.$row['nazwa_branzy'].': '.$row['rodzaj_umowy'].': '.$row['login'].': '.$row['tresc'].': '.$row['lokalizacja'].'</li>';
				  }
				  $stmt->closeCursor();
				  echo '</ul>';
 
				  //print_r($_GET);
			   }
			   }
			   catch(PDOException $e)
			   {
				  echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
			   }
		}	
[PHP] pobierz, plaintext 

I nadal wyświetla wszystkie rekordy... Może w instrukcji warunkowej coś jest nie tak... Ale tak patrzę i raczej wszystko z nią ok.

Już mam. Pierwszego if zmieniłem z:

[PHP] pobierz, plaintext 
if(empty($_GET['nazwa']) || $_GET['nazwa'] == "---" || $_GET['umowa'] == "---" || $_GET['lokalizacja'] == "---")
[PHP] pobierz, plaintext 

na:

[PHP] pobierz, plaintext 
if(empty($_GET['nazwa']) || $_GET['nazwa'] == "---" && $_GET['umowa'] == "---" && $_GET['lokalizacja'] == "---")
[PHP] pobierz, plaintext 

I działa. Dzięki za czas i pomoc (IMG:style_emoticons/default/smile.gif)