Witam,

czy można jakoś zoptymalizować poniższe działanie skryptu php, zastanawiam się jakby to zrobić:


tabele:

users
id | imie | nazwisko |
------------------------
1 | jan | kowalski |


users_images
users_id | image |
-----------------------
1 | img1.jpg
1 | img2.jpg
1 | img3.jpg

[PHP] pobierz, plaintext 
function get_total_images($we)
{
  $sql="select users_id, count(*) from users_images where users_id = ".$we;
  $result1 = mysql_query($sql);
  $row = mysql_fetch_array($result1); 
  return $row[1];
}
 
 
$sql = "select * from users";
 
$result = mysql_query($sql);
 
while ($row = mysql_fetch_array($result))
{
echo "Imie: ".$row['imie']."<br>Nazwisko: ".$row['nazwisko']."<br>Ilość zdjęć: ".get_total_images($row['id']);
}
[PHP] pobierz, plaintext 

niby to jakoś działa ale przez zastosowanie funkcji get_total_images zwiększa mi się liczba zapytań do bazy bo każdy wyświetlany rekord "odpytuje" baze ile dany użytkownik ma zdjęć

np.
mając 1000 użytkowników i chcąc ich wyświetlić to w pętli WHILE muszę odpalić 1000 razy jedno zapytanie aby sprawdzić ile posiadają zdjęć


czy dało by się użyć tylko jednego zapytania sql aby wyświetlić użytkowników oraz liczbę zdjęć?


pozdrawiam

Do tego celu uzywa sie LEFT JOIN users_images
a potem group by oraz count

[PHP] pobierz, plaintext 
select users.*,count(users_images.id) as ilosc_zdjec from users left join users_images on users.id=users_images.users_id group by users.id;
[PHP] pobierz, plaintext 

super, dziękuję bardzo za pomoc,

ale mam pytanie odnośnie tego zapytania które podałeś:


spróbowałem je odpalić ale niestety skrypt się zawiesza kończąc działenie informacją o przekroczeniu czasu na wykonanie skryptu

czy nie ma w nim jakiegoś błędu?

zmieniłem je trochę dodając limit i aby pobieral z tabeli users tylko imie

[SQL] pobierz, plaintext 
SELECT users.imie,count(users_images.id) AS ilosc_zdjec FROM users LEFT JOIN users_images ON users.id=users_images.users_id GROUP BY users.id LIMIT 100
[SQL] pobierz, plaintext 

[PHP] pobierz, plaintext 
$sql = 'SELECT users.*,count(users_images.id) AS ilosc_zdjec FROM users LEFT JOIN users_images ON users.id=users_images.users_id GROUP BY users.id';
 
$result = mysql_query($sql);
 
while ($row = mysql_fetch_array($result)) {
  echo "Imie: ".$row['imie']."<br>Nazwisko: ".$row['nazwisko']."<br>Ilość zdjęć: ".$row['ilosc_zdjec'];
}
[PHP] pobierz, plaintext 

jeszcze raz dzięki za pomoc,


dla mnie zapytanie na oko wygląda że jest poprawne ale niestety skrypt się zawiesza (przekroczenie czasu),

może w zapytaniu jest jakiś błąd? niestety ja nie jestem w stanie tego sprawdzić, pomożesz jeszcze raz?



Pozdrawiam

No jesli masz duuuuuzo userow i duuuuuzo zdjec do userow i jeszcze dodatkowo nie zalozyles indeksow na te tabele, to skrypt moze trwac dlugo

userow jest 43000

[SQL] pobierz, plaintext 
CREATE TABLE `users` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `first_name` char(50) DEFAULT NULL,
  `last_name` char(25) DEFAULT NULL,
  `email` char(25) DEFAULT NULL
  PRIMARY KEY (`id`),
  UNIQUE KEY `id` (`id`),
  UNIQUE KEY `email` (`email`),
  UNIQUE KEY `profile_name_2` (`profile_name`)
) ENGINE=InnoDB DEFAULT CHARSET=latin2
[SQL] pobierz, plaintext 

a zdjec 82000

[SQL] pobierz, plaintext 
CREATE TABLE `images` (
  `users_id` int(11) DEFAULT NULL,
  `images` char(100) DEFAULT NULL
) ENGINE=InnoDB DEFAULT CHARSET=latin2
[SQL] pobierz, plaintext 

a czy z indeksami można coś pozmieniac w tych tabelach?

No wypadaloby zalozyc index na users_id

założyłem indeks na users_id (PRIMARY KEY) i różnica jest kolosalna, bez jakiegokolwiek limitu wyświetla mi wszystkich userów w mgnieniu oka, albo nawet szybciej
nie zdawałem sobie sprawy za co odpowiadają indeksy, trochę poczytałem i dalej będę drążyć temat indeksów

mam jeszcze jedno pytanie, które zapytanie będzie "lepsze" aby policzyć liczbę rekordów w tabeli:

[SQL] pobierz, plaintext 
SELECT COUNT(*) AS total FROM users
 
czy
 
SELECT COUNT(id) AS total FROM users
[SQL] pobierz, plaintext 

Zapewne te, które będzie wykonywać się krócej - sprawdź czas w jakim się wykonuje. Przy małych bazach i tak niezauważalne.