[MySQL][PHP]zooptymalizowanie skryptu Opcje

[bialko0019]

Witam. Dosyć niedawno zacząłem odkrywać programowanie obiektowe, więc mam pytanie. Stworzyłem skrypt, który wyciąga z bazy danych jednym zapytaniem dzisiaj, wczoraj zalogowanych userów, oraz dzisiaj i wczoraj zarejestrowanych. Wszystko ładnie działa, jednak gdybym tworzył strukturalnie, wydaje mi się było by mniej kodu - tyle, że więcej zapytań do bazy danych - a chciałbym wszystko zminimalizować To w kwestii nauki.

Napisałem tak ( pewnie chaotycznie ); :

[PHP] pobierz, plaintext 
 <?php
 
 mysql_connect();
 mysql_select_db();
 // ile osób w bazie
 // ile dzisiaj zalogowanych
 // ile wczoraj zalogowanych
 // ile dzisiaj rejestrowanych
 // ile wczoraj zarejestrowanych
 
class statystyki{
 
function osoby(){
 
$zapytanie = mysql_query("SELECT * FROM uzytkownicy WHERE login NOT LIKE''");
$zawartosc = mysql_num_rows($zapytanie);
$this->ogolem=$zawartosc;
$i = 0;
while($dana=mysql_fetch_array($zapytanie)){
 
$osoby[$i] = new statystyki;
$osoby[$i] -> zalogowany = $dana['ostatnie_logowanie'];
$osoby[$i] -> rejestracja = $dana['rejestracja'];
 
$i++;
}
$dzisiajz = 0;
$dzisiajr = 0;
$wczorajz = 0;
$wczorajr = 0;
foreach($osoby as $id => $statystyki){
 
if($statystyki -> zalogowany >= date('Y-m-d 00:00:00'))
{
$dzisiajz++;
}
 
if($statystyki -> rejestracja >= date('Y-m-d 00:00:00'))
{
$dzisiajr++;
}
 
if($statystyki -> zalogowany >= date('Y-m-d 00:00:00', strtotime("-1 day")) && $statystyki -> zalogowany < date('Y-m-d 00:00:00'))
{
$wczorajz++;
}
 
if($statystyki -> rejestracja >= date('Y-m-d 00:00:00', strtotime("-1 day")) && $statystyki -> rejestracja < date('Y-m-d 00:00:00'))
{
$wczorajr++;
}
 
 
}
$this->dzisiaj=$dzisiajz;
$this->dzisiajr=$dzisiajr;
$this->wczorajz=$wczorajz;
$this->wczorajr=$wczorajr;
 
 
}
 
 
 
 
 
 
 
 
}
 
$obiekt = new statystyki;
$obiekt->osoby();
echo $obiekt->dzisiajz; // dzisiaj zalogowani
echo $obiekt->dzisiajr; // dzisiaj zarejestrowani
echo $obiekt->wczorajl; // wczoraj zalogowani
echo $obiekt->wczorajr; // wczoraj zarejestrowani
 
 ?>
 
 
 
[PHP] pobierz, plaintext 

Tyle w tym dobrego, że całą klasę mam w innym pliku i go includuje, a w dowolnym miejscu wpisuję tylko dwa wyrazy : $obiekt->zalezy_co_chce_otrzymac . Natomiast strukturalnie napisałbym cztery proste funkcje, które wyciągają odpowiednio z bazy danych odpowiednie wartości, albo jedną, w której wpisuję warunek...

Jakie inne zastosowania mógłbym użyć, by to "programowanie obiektowe" w tym przypadku co podałem, było czytelniejsze, o znacząco mniej objętości ? W tym przypadku obciążam bardziej tylko serwer, a w przypadku wielu zapytań obciążył bym bardzo znowu bazę danych... ? Nie chodzi mi tutaj o gotowe rozwiązanie, tylko o wskazówki, czego jeszcze nie zastosowałem, co ułatwiło by pracę na takich właśnie skryptach o oparciu o obiektowość?

Pozdrawiam!

Ten post edytował bialko0019 13.11.2010, 12:10:14

[nospor]

1) WHERE login NOT LIKE''
Co to ma byc?
2) Pobierasz wszystkich userów tylko po to by wyliczyc ilu ich jest? Tylko po to by wyliczyc ilu zarejestrowalo sie dzisiaj? Przecież ty baze i skrypt zarżniesz czyms takim przy większej liczbie userów.
3) Każdą z rzeczy co chcesz zrobic zrób najpierw jako oddzielne zapytanie. ALe tak, by to zrobic w zapytaniu a nie w php.
DO liczby uzywa sie COUNT a nie w php sie zlicza wszystkie rekordy.
Gdy już będziesz miał te wszystkie zapytania to połącz je przez zwykle UNION. Bedzie to o niebo optymalniejsze od tego co masz teraz.
4) Mozna to oczywiscie zrobic nawet bez union, w jendym zapytaniu, ale juz nie będę ci glowy zawracal bo jest to ciut bardziej skomplikowane

edit:

[PHP] pobierz, plaintext 
while($dana=mysql_fetch_array($zapytanie)){
 
$osoby[$i] = new statystyki;
$osoby[$i] -> zalogowany = $dana['ostatnie_logowanie'];
$osoby[$i] -> rejestracja = $dana['rejestracja'];
 
$i++;
}
[PHP] pobierz, plaintext 

A to niby po co? Usłuszałeś pojęcie OBIEKT i walisz tym wszędzie gdzie popadnie bez opamiętania - no nie na tym to polega
I jeszcze na dodatek tworzysz obiekt STATYSTYKI dla każdego usera już w obiekcie STATYSTYKI :/

[bialko0019]

1) No tak, bo mam trochę w bazie rekordów, które mają puste kolumny `login` i inne, ale ich potrzebuję w innym celu niż statystyki dlatego ich w taki sposób wykluczam teraz.

Czyli w przypadku takiego skryptu założeniem najodpowiedniejszym założeniem by było:

1. Policzenie wszystkich użytkowników, zapytaniem:

[SQL] pobierz, plaintext 
SELECT COUNT(*) AS id FROM uzytkownicy
[SQL] pobierz, plaintext 

2. Znalezienie, ile jest użytkowników z dzisiejszym logowaniem:

[SQL] pobierz, plaintext 
SELECT COUNT( * ) FROM uzytkownicy WHERE rejestracja >= '2010-11-14 00:00:00'
[SQL] pobierz, plaintext 

3. Analogicznie z ostatnim logowaniem.. Rzeczywiście, chcę zobaczyć ile takich rekordów jest, a nie potrzebne są mi dane kto co i jak i kiedy, gdy wywołałem zapytaniem SELECT * FROM uzytkownicy...

Teraz praktycznie:

Tworzę klasę.

Piszę funkcje, które zwracają odpowiednie liczby - zależy co chcę uzyskać. Uzyskaną liczbę przypisuję do obiektu, np. : $this=dzisiaj_zalogowani=$dzisiejszy zalogowani . Zamykam klasę.

Teraz w miejscu, gdy chcę wyświetlić te dane statystyczne, po prostu piszę:

[PHP] pobierz, plaintext 
<?php
 
 
$obiekt = new cos;
$obiekt->funkcja_dzisiaj();
[...]
 
 
?>
[PHP] pobierz, plaintext 

O to Ci chodziło mniej więcej ?


o coś takiego:

[PHP] pobierz, plaintext 
 
 
 // ile dzisiaj zalogowanych
 // ile wczoraj zalogowanych
 // ile dzisiaj rejestrowanych
 // ile wczoraj zarejestrowanych
 
 class statystyki{
 
 
 
 function zalogowani($data){
 
 $ilosc = mysql_fetch_assoc(mysql_query("SELECT COUNT(*) FROM uzytkownicy $data"));
 $this->osoby=$ilosc['COUNT(*)'];
 
 }
 
 }
 
 
 $obiekt = new statystyki;
 $obiekt->zalogowani("WHERE ostatnie_logowanie >= '2010-11-14 00:00:00'");
 
echo $obiekt->osoby;
 
[PHP] pobierz, plaintext 

i ten poprzedni wykonuje się średnio 0.5s, a ten powyżej średnio 0.05, może dłużej - czyli ok 10 razy szybciej. Ale mniej więcej, to jest dobrze napisany skrypt z klasą?

Ten post edytował bialko0019 14.11.2010, 11:14:52

[Mephistofeles]

Można wiedzieć po co masz w bazie puste loginy? Pokaż tabelę, bo mam przeczucie, że jest przekombinowana.

Ten post edytował Mephistofeles 14.11.2010, 12:40:56

[bialko0019]

Mephistofeles, pytałem w temacie o klasę, a nic nie wspominałem o bazie danych, a dokładniej tabelę Dosyć nie dawno piszę obiektowo wykorzystując klasy w PHP dlatego pytam się - czy to jest poprawnie napisany kod Nigdzie tego skryptu nie chcę wykorzystywać, to tylko dla przykładu i nauki podałem akurat wykorzystując bazę danych Chodzi mi o nakierowanie wyłącznie o skrypt PHP

[nospor]

Chodziło mi o to, ze do sprawdzania czy cos jest puste czy nie, nie uzywa się LIKE..... przecież to jest cholernie nie optymalne