witam mam 3 tablice

department:
id_department
name

employee:
id_employee
fname
lname
pesel
fee

employee_has_department - tablica relacji
id_employee
id_department


probuj zrobic cos takiego aby po wejsciu do jakiegos dzialu wyswietlilo liste aktualnie przypisanych pracownikow. zrobilem to za pomoca zapytania:

[SQL] pobierz, plaintext 
SELECT em.fname, em.lname, em.pesel, em.fee FROM employee em, employee_has_department ed WHERE em.id_employee = ed.id_employee AND ed.id_department = '$id'
[SQL] pobierz, plaintext 

i dalej chce aby byl link edytuj ktory przekieruj do kolejnej strony gdzie zostana wyswietleni wszyscy pozostali pracownicy ktorzy jeszcze nie sa przyporzadkowani do tego danego dzialu.

problem polega na tym ze nie chce mi wyswietlac wynikow. pokazuje mi sie blad:

[HTML] pobierz, plaintext 
Warning: mysql_numrows() expects parameter 1 to be resource, boolean given in C:\xampp\xampp\htdocs\company\depedit.php on line 43
[HTML] pobierz, plaintext 

kod:

[PHP] pobierz, plaintext 
<table width="1000" border="0" cellspacing="0" cellpadding="0">
<?php 
 
	$id = $_GET['ID'];
	$empquery = "SELECT em.fname, em.lname, em.pesel, em.fee FROM employee em, employee_has_department ed WHERE em.id_employee = ed.id_employee AND ed.id_department = '$id'";
	$result=mysql_query($empquery);
 
$num=mysql_numrows($result);  //to jest linia 43
 
$i=0;
while ($i < $num) {
 
$imie=mysql_result($result,$i,"em.fname");
$nazwisko=mysql_result($result,$i,"em.lname");
$pesel=mysql_result($result,$i,"em.pesel");
$pensja=mysql_result($result,$i,"em.fee");
?>
  <tr>
    <td><?php // echo $id; ?></td>
	<td><?php echo $imie; ?></td>
	<td><?php echo $nazwisko; ?></td>
	<td><?php echo $pesel; ?></td>
	<td><?php echo $pensja; ?></td>
  </tr>
    <?php
$i++;
}
?>
  <tr>
    <td colspan="5">
    <a href="depedit.php?ID=<?php echo $id; ?>">edytuj</a>
    </td>
  </tr>
 
</table>
[PHP] pobierz, plaintext 

z gory dziekuje za wszelkie sugestie

1. zapoznaj się z podstawami pobierania wyników z bazy http://www.php.net/manual/en/function.mysql-fetch-array.php
2. Nie sprawdziłeś, czy zapytanie się wykonało i nie wywala błędu, jako początkujący stosuj:

[PHP] pobierz, plaintext 
$result=mysql_query($empquery) or die(mysql_error()); 
[PHP] pobierz, plaintext 

3. Czytaj treści błędów, masz dokładnie napisane, że miałeś przekazać do funkcji zasób, a przekazałeś typ logiczny(boolean)
4. poczytaj o sql injection i zabezpieczaniu zapytania, bo masz tutaj lukę w zabezpieczeniach, a najlepiej to używaj PDO i prepared statements
5. Taka struktura zapytania sql jest dopuszczalna, ale lepiej jest używać LEFT JOIN, jest o wiele czytelniejsza, zwłaszcza jak z czasem zaczniesz pisać bardziej skomplikowane kwerendy

1. zapoznaj się z podstawami pobierania wyników z bazy http://www.php.net/manual/en/function.mysql-fetch-array.php

dopiero pozniej pomyslalem zeby sprawdzic czy zapytanie sql wogole sie wykona. takze tutaj byl blad.

4. poczytaj o sql injection i zabezpieczaniu zapytania, bo masz tutaj lukę w zabezpieczeniach, a najlepiej to używaj PDO i prepared statements

to jest tylko praca przykladowa i pisze narazie tylko glowny zarys projektu takze w tym stadium rozwoju zadne zabezpieczenia nie sa konieczne a raczej powodowalyby utrudnienia w czytaniu kodu.

5. Taka struktura zapytania sql jest dopuszczalna, ale lepiej jest używać LEFT JOIN, jest o wiele czytelniejsza, zwłaszcza jak z czasem zaczniesz pisać bardziej skomplikowane kwerendy

zamiast left joina uzylem:

[SQL] pobierz, plaintext 
$empquery = "SELECT em.fname, em.lname, em.pesel, em.fee FROM employee em WHERE em.id_employee NOT IN ( SELECT ed.id_employee FROM employee_has_department ed WHERE ed.id_department ='$id')";
[SQL] pobierz, plaintext 

dzieki za odpowiedz bede mial na uwadze to co napisales:)

Pierwsze Twoje zapytanie działa prawidłowo, tzn. wybiera wszystkich pracowników, którzy pracują w określonym oddziale. Drugie zapytanie działa odwrotnie, tzn. wybiera wszystkich pracowników, którzy nie pracują w tym okreslonym oddziale. Złączenia są lepsze nie tylko ze względu na "łatwość czytania" (a tutaj zresztą bywa różnie), ale przede wszystkim ze względu na wydajność.