Bardzo długie wykonywanie skryptu Opcje

[mati251195]

Witam,
Napisałem niezbyt skomplikowane narzędzie do sprawdzania, kto skomentował news. Sprawdzanie dotyczy około 100 newsów dla 20 użytkowników, a skrypt wykonuje się od 40 do 60 sekund i wg mnie coś jest nie tak. Albo hosting pracuje bardzo bardzo wolno, albo ja popełniłem jakiś błąd.

Na początku skrypt pobiera listę użytkowników i zapisuje do tablicy ich nicki, następnie pętla while po kolei odczytuje kolejne newsy. W pętli while jest pętla for, która sprawdza po kolei każego użytkownika - pobiera komentarz danego użytkownika do danego newsa, gdy nie znajdzie to wiadomo, że nie skomentował, a gdy znajdzie to sprawdza datę, aby było wiadomo po jakim czasie skomentował newsa. W pętli while od newsów od razu jest warunek, że gdy news jest w tablicy ignorowani lub gdy jego premiera jeszcze nie zastąpiła to nie sprawdza tego, tylko wstawia wszystkie, że nie skomentowali (IMG:style_emoticons/default/smile.gif) Tutaj znajduje się kod skryptu: http://wklejto.pl/173709

Z góry dziękuję za pomoc mi w tej sprawie, ponieważ chciałbym, aby to działało sprawnie (IMG:style_emoticons/default/smile.gif)

[r4xz]

zadanie 1. Policz ile zapytań wysyłasz do bazy dla w/w 100 newsów i 20 użytkowników (IMG:style_emoticons/default/smile.gif)

---edit---
przyjmij że

[PHP] pobierz, plaintext 
in_array($data['news_id'], $ignorowane) || $data['news_datestamp'] >= $teraz
[PHP] pobierz, plaintext 

zawsze zwraca false (IMG:style_emoticons/default/smile.gif)

Ten post edytował r4xz 12.09.2013, 19:17:40

[mati251195]

Tych zapytań jest około 2300. Mam inny skrypt, który wykonuje ponad 1800 zapytań i on działa bardzo sprawnie, czas wykonywania kodu to kilka sekund maksymalnie, a tu killkanaście razy dłużej to trwa (IMG:style_emoticons/default/sad.gif)

Ta linijka kodu nie zwraca zawsze FALSE, niby dlaczego tak uważasz?

[Crozin]

Napisz jakie jest zadanie tego kodu, co chcesz przy jego pomocy osiągnąć. Na pierwszy rzut oka źle zabrałeś się do problemu i nie ma sensu poprawiać tego kodu. Go trzeba napisać inaczej.

[Pyton_000]

Po pierwsze czy zapytania optymalnie korzystają z kluczy w bazie?

Jeżeli nie to czy nie lepiej pokusić się o pobranie wszystkich komentów (tylko wymaganych pól a nie *) i iterować. Na pewno nie zajedzie to bazy a pamięci wystarczy

[foxbond]

narzędzie do sprawdzania, kto skomentował news

To na 100% da się zrobić jednym zapytaniem. Chyba, że coś nie tak zaprojektowałeś db