Forum PHP.pl

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ęć (IMG:style_emoticons/default/facepalmxd.gif)


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 

Ten post edytował aras785 6.04.2016, 11:54:36

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 (IMG:style_emoticons/default/wink.gif)
nie zdawałem sobie sprawy za co odpowiadają indeksy, trochę poczytałem i dalej będę drążyć temat indeksów (IMG:style_emoticons/default/wink.gif)

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.