Wyszukiwarka ofert pracy - prośba o opinie Opcje

[Panicz74]

Witam,

Jestem trochę początkujący w PHP, ale trochę czytam, nawet troszkę w Symfony zacząłem się bawić. Chciałbym pokazać wam mój skrypt, który służy do wyświetlania i filtracji ofert pracy z bazy danych. Do wyboru są pola: nazwa branży, rodzaj umowy, lokalizacja. Chciałbym poznać waszą opinię na jego temat, co jest źle || czego za dużo || czego za mało || jakie rozwiązania są lepsze od zastosowanego, itd. Wszystkie hejty mile widziane (IMG:style_emoticons/default/smile.gif)

Oto kod: http://pastebin.com/G9eu90d2

[Panicz74]

Właśnie próbuję odchudzić ten kod. Chciałem zrobić to na switchu ale mając 3x3 możliwości z GET chyba się to mija z celem. Takie coś nie działa:

[PHP] pobierz, plaintext 
else if(!empty($_GET['nazwa']))
		{
			switch($_GET['nazwa'] && $_GET['umowa'] && $_GET['lokalizacja'])
			{
				//2
				case $_GET['nazwa'] !== "---" && $_GET['umowa'] == "---" && $_GET['lokalizacja'] == "---":
							$sql = "SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja, o.date FROM ogloszenia AS o 
									  LEFT JOIN users u ON u.id_usera = o.id_usera 
									  LEFT JOIN branza b ON b.id_branzy = o.id_branzy 
									  LEFT JOIN umowa i ON i.id_umowy = o.id_umowy 
									  LEFT JOIN lokalizacja l ON l.id_lok = o.id_lok 
									  WHERE b.nazwa_branzy = :nazwa_branzy";
				break;
				//3
				case $_GET['nazwa'] !== "---" && $_GET['umowa'] !== "---" && $_GET['lokalizacja'] == "---":
							$sql = "SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja, o.date FROM ogloszenia AS o 
									  LEFT JOIN users u ON u.id_usera = o.id_usera 
									  LEFT JOIN branza b ON b.id_branzy = o.id_branzy 
									  LEFT JOIN umowa i ON i.id_umowy = o.id_umowy 
									  LEFT JOIN lokalizacja l ON l.id_lok = o.id_lok 
									  WHERE b.nazwa_branzy = :nazwa_branzy AND i.rodzaj_umowy = :umowa";				
				break;
				//4
				case $_GET['nazwa'] !== "---" && $_GET['umowa'] !== "---" && $_GET['lokalizacja'] !== "---":
							$sql = "SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja, o.date FROM ogloszenia AS o 
									  LEFT JOIN users u ON u.id_usera = o.id_usera 
									  LEFT JOIN branza b ON b.id_branzy = o.id_branzy 
									  LEFT JOIN umowa i ON i.id_umowy = o.id_umowy 
									  LEFT JOIN lokalizacja l ON l.id_lok = o.id_lok 
									  WHERE b.nazwa_branzy = :nazwa_branzy AND i.rodzaj_umowy = :umowa AND l.lokalizacja = :lokalizacja";
				break;
				//5
				case $_GET['nazwa'] == "---" && $_GET['umowa'] !== "---" && $_GET['lokalizacja'] == "---":
							$sql = "SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja, o.date FROM ogloszenia AS o 
									  LEFT JOIN users u ON u.id_usera = o.id_usera 
									  LEFT JOIN branza b ON b.id_branzy = o.id_branzy 
									  LEFT JOIN umowa i ON i.id_umowy = o.id_umowy 
									  LEFT JOIN lokalizacja l ON l.id_lok = o.id_lok 
									  WHERE i.rodzaj_umowy = :rodzaj_umowy";
				break;
				//6
				case $_GET['nazwa'] == "---" && $_GET['umowa'] == "---" && $_GET['lokalizacja'] !== "---":
							$sql = "SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja, o.date FROM ogloszenia AS o 
									  LEFT JOIN users u ON u.id_usera = o.id_usera 
									  LEFT JOIN branza b ON b.id_branzy = o.id_branzy 
									  LEFT JOIN umowa i ON i.id_umowy = o.id_umowy 
									  LEFT JOIN lokalizacja l ON l.id_lok = o.id_lok 
									  WHERE l.lokalizacja = :lokalizacja";
				break;
				//7
				case $_GET['nazwa'] == "---" && $_GET['umowa'] !== "---" && $_GET['lokalizacja'] !== "---":
							$sql = "SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja, o.date FROM ogloszenia AS o 
									  LEFT JOIN users u ON u.id_usera = o.id_usera 
									  LEFT JOIN branza b ON b.id_branzy = o.id_branzy 
									  LEFT JOIN umowa i ON i.id_umowy = o.id_umowy 
									  LEFT JOIN lokalizacja l ON l.id_lok = o.id_lok 
									  WHERE l.lokalizacja = :lokalizacja AND i.rodzaj_umowy = :rodzaj_umowy";
				break;
				//8
				case $_GET['nazwa'] !== "---" && $_GET['umowa'] == "---" && $_GET['lokalizacja'] !== "---":
							$sql = "SELECT b.nazwa_branzy, u.login, o.tresc, i.rodzaj_umowy, l.lokalizacja, o.date FROM ogloszenia AS o 
									  LEFT JOIN users u ON u.id_usera = o.id_usera 
									  LEFT JOIN branza b ON b.id_branzy = o.id_branzy 
									  LEFT JOIN umowa i ON i.id_umowy = o.id_umowy 
									  LEFT JOIN lokalizacja l ON l.id_lok = o.id_lok 
									  WHERE l.lokalizacja = :lokalizacja AND b.nazwa_branzy = :nazwa_branzy";
				break;
			}
 
			try
			   {
				  $pdo = new PDO($pol, $user, $pass);
				  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
				  if(isset($_GET['nazwa']))
				  {
				  $stmt = $pdo->prepare($sql);
				  if($_GET['lokalizacja'] !== "---"){$stmt -> bindValue(':lokalizacja', $_GET['lokalizacja'], PDO::PARAM_STR);}
				  else if($_GET['nazwa'] !== "---"){$stmt -> bindValue(':nazwa_branzy', $_GET['nazwa'], PDO::PARAM_STR);}
				  elseif($_GET['umowa'] !== "---"){$stmt -> bindValue(':umowa', $_GET['umowa'], PDO::PARAM_STR);}
				  $stmt -> execute();
				  echo '<ul>';
				  foreach($stmt as $row)
				  {
					  echo '<li>'.$row['nazwa_branzy'].': '.$row['rodzaj_umowy'].': '.$row['login'].': '.$row['tresc'].': '.$row['lokalizacja'].': '.$row['date'].'</li>';
				  }
				  $stmt->closeCursor();
				  echo '</ul>';
				  //print_r($_GET);
			   }
			   }
			   catch(PDOException $e)
			   {
				  echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
			   }
			}
[PHP] pobierz, plaintext 

A co myślicie o kursach Symfony2 z Eduweb? Ktoś miał może z nimi styczność?