[PHP][SQL]Dzienniczek ucznia, czy nie dało by się lepiej? Opcje

[cycofiasz]

Witam, zrobiłem sobie skrypt dzienniczka ucznia. Zasady działania nie muszę za pewne tłumaczyć .

Mam 2 tabele w mysql:




Skliciłem taki oto skrypt:

[PHP] pobierz, plaintext 
<?php
echo '<table style="border-collapse: collapse;" border="1">'."\n";
echo ' <caption style="caption-side: top;padding: 2px;"><b>Oceny ucznia nr. 20</b></caption>'."\n";
echo ' <tr align="center"><td width="30"><strong>Lp.</strong></td><td width="200"><strong>PRZEDMIOT</strong></td><td width="200"><strong>OCENY</strong></td>'."\n";
 
// pobieramy listę przedmiotów 
$sql_przdmioty = mysql_query ("SELECT `przedm_id`,    `przedm_nazwa` FROM `dzienniczek_przedmioty`");
 
while ($row = mysql_fetch_array($sql_przdmioty))
{    
    $oceny=''; $o=''; //resetowanie zmiennych
    $sql_oceny = mysql_query ("SELECT `ocena_stopien` ,`ocena_waznosc` FROM `dzienniczek_oceny` WHERE `ocena_przedm_id` = '$row[przedm_id]'");
    while ($row2 = mysql_fetch_array($sql_oceny)) 
    {
        if($row2[ocena_waznosc] == '1') $o = '<span style="color:red">'.$row2[ocena_stopien].'</span>'; // kolorowanie sprawdzienów
        elseif($row2[ocena_waznosc] == '2') $o = '<span style="color:blue">'.$row2[ocena_stopien].'</span>'; //kolorowanie innych ocen
        $oceny .= $o.', ';
    }
    
    echo " <tr><td>$row[przedm_id]</td><td>$row[przedm_nazwa]</td><td>$oceny</td></tr>\n";
}
 
echo '</table>'."\n";
?>
[PHP] pobierz, plaintext 

co dało taki rezultat (taki jak ma być)




Jak zauważyliście skrypt ten pobiera najpierw listę przedmiotów i podczas każdego przebiegu pętli jest wykonywane zapytanie o oceny do tego przedmiotu... Nie wydaje się wam się też że to jest nieoptymalne? Jak pobrać to wszystko bez zawierania w pętlach zapytań do sql?

[zegarek84]

[SQL] pobierz, plaintext 
SELECT dzienniczek_przedmioty.przedm_id, dzienniczek_przedmioty.przedm_nazwa, dzienniczek_oceny.ocena_stopien, dzienniczek_oceny.ocena_waznosc
FROM dzienniczek_przedmioty LEFT JOIN dzienniczek_oceny ON dzienniczek_przedmioty.przedm_id=dzienniczek_oceny.ocena_przedm_id
ORDER BY dzienniczek_przedmioty.przedm_id
[SQL] pobierz, plaintext 

jeszcze jedno zauważyłem że "dokładasz" do ocen - wyświetlając w pętli to zapytanie zrób sobie przed petlą jakąś zmienną i w pętli sprawdzaj czy jej wartość jest różna od id_przedmiotu - jesli różna wyświetlisz cały wiersz, jeśli taka sama to pomiń prawie cały rekord i dopisz ocenę - po tym na koniec pętli przypisz wartość tej zmiennej co porównujesz wartość id_przedmiotu....

mam na myśli taką sytuację zwróconą z bazy:

Kod

przedm_id     ||przedm_nazwa    || ocena_stopien    || ocena_waznosc
    1    ||    j.pol    ||    10/20    ||    1
    1    ||    j.pol    ||    3    ||    2
    2    ||    j.ang    ||    5    ||    2

lub w 2 zapytaniach prawie tak jak zrobiłeś tyle że pobież odrazu pełne tabele te co potrzebujesz i przyrównaj to w php gdyż odwołanie w pętli do bazy nie jest najlepszym pomysłem...

Ten post edytował zegarek84 17.01.2009, 06:21:52

[cycofiasz]

Ten 2 sposób wydaje się chyba sensowniejszy.

Czyli najpierw wykonuję zapytanie sql_oceny

[SQL] pobierz, plaintext 
SELECT ocena_przedm_id , ocena_stopien , ocena_waznosc FROM  `dzienniczek_oceny`
[SQL] pobierz, plaintext 

Przekazuję wynik do tablicy $oceny

[PHP] pobierz, plaintext 
<?php
while ($row = mysql_fetch_array($sql_oceny)) $oceny[]=$row
?>
[PHP] pobierz, plaintext 

Otrzymam taką zawartość tablicy $oceny:

[PHP] pobierz, plaintext 
<?php
Array
(
    [0] => Array
        (
            [0] => 2
            [ocena_przedm_id] => 2
            [1] => 5
            [ocena_stopien] => 5
            [2] => 2
            [ocena_waznosc] => 2
        )
 
    [1] => Array
        (
            [0] => 3
            [ocena_przedm_id] => 3
            [1] => 5
            [ocena_stopien] => 5
            [2] => 1
            [ocena_waznosc] => 1
        )
     ...dalsze wartości...
)
?>
[PHP] pobierz, plaintext 

A potem pobieram listę przedmiotów z bazy i przy każdym przebiegu pętli, zamiast dawać kolejne zapytanie do mysql, filtruję tablicę $oceny za pomocą pętli foreach:

[PHP] pobierz, plaintext 
<?php
//pobieramy wszystkie oceny
$sql_oceny = mysql_query ("SELECT ocena_przedm_id , ocena_stopien , ocena_waznosc FROM `dzienniczek_oceny`");
 
while($row1 = mysql_fetch_array($sql_oceny)) 
$oceny[] = $row1;
 
 
// pobieramy listę przedmiotów 
$sql_przdmioty = mysql_query ("SELECT `przedm_id`,    `przedm_nazwa` FROM `dzienniczek_przedmioty`");
 
while ($row2 = mysql_fetch_array($sql_przdmioty))
{    
    $ocenki = '';
    foreach($oceny as $wartosc)
    {
        if($wartosc[ocena_przedm_id] == $row2[przedm_id])
        {
            $ocenki .= $wartosc[ocena_stopien].', ';
        }
    }
    
    echo " <tr><td>$row2[przedm_id]</td><td>$row2[przedm_nazwa]</td><td>$ocenki</td></tr>\n";
}
 
echo '</table>'."\n";
?>
[PHP] pobierz, plaintext 

Dobrze zrozumiałem? Czy to jest wydajniejsze niż dawanie zapytań przy każdym przedmiocie? Ta tablica $oceny z czasem może stać się większa...

Ten post edytował cycofiasz 17.01.2009, 11:03:21

[zegarek84]

sory że tak puźno odpisuję ale dopiero dziś znalazłem czas (w końcu po zaliczeniach ^^)

Dobrze zrozumiałem? Czy to jest wydajniejsze niż dawanie zapytań przy każdym przedmiocie? Ta tablica $oceny z czasem może stać się większa...

to i tak bedzie wydajniejsze od zapytań w pętli do bazy myql - choć bawiąc się bazą sqlite i używając sesji miałbyś podobny efekt ...

ja proponuje dane które "sciągasz" z bazy danych zapisywać do tablicy wielo wymiarowej mniej więcej o takiej konstrukcji:

Kod

Array
(
    [$id_przedm] => Array
        (
            [nazwa_id_przedm] => j.pol
            [ocena] => Array
                (
                    [0] => Array
                        (
                            [stopien] => 2
                            [warznosc] => 1
                        )

                    [1] => Array
                        (
                            [stopien] => 3
                            [warznosc] => 2
                        )
                )
        )
)

głównie dlatego żeby nie przeszukiwać w pętli całej tablicy ocen tylko jakby to posortować.... z kolei jesli tych ocen miało by być multum (za bardzo obciążające pamięć przez operacje na tablicach) to mógłbyś w pętli pobierać większą liczbę tych ocen sortując po id_przedmiotu i korzystając z LIMIT w zapytaniu a potem następna działkę do tablicy pamiętając o zwolnieniu pamięci tego co już się "zoperowało" czyli kasowaniu fragmentów starej tablicy przez urzycie funkcji unset()... pobieranie każdej oceny w petli z bazy to trochu by to spowolniło wszystko (w ostatecznosci jeśli aż tak dużo ocen to porcjami - choć wątpię by było aż tak dużo) i operacja jak już wspomniałem na tablicach - ale jednak pasuje przypisać i operować na tablicy wielowymiarowej żeby nie "przelatywać i sprawdzać" w pętli za każdym razem tych ocen - poprostu posortować podczas wpisów do tablicy i operować jakby to nazwać na "podtablicach"...

jeszcze raz przepraszam że tak puźno odpisuję i o ile to przeczytasz mam nadzieje że zrozumialeś - przegladam tematy w których się wypowiadam więc jeśli coś napiszesz to za jakiś czas znowu tu zajrzę

PS.
przez $id_przedm mam na myśli te cyferki dla przedmiotu (można by w sumie dac inna konstrukcję i odrazu nazwę przedmiotu tutaj zamiast wewnątrz tego fragmentu tablicy ale jak już kto woli - pozatym chcący tu odrazu dawać nazwę przedmiotu to pasowało by trzymać w pamięci podczas przepisywania ocen tablicy z przedmiotami i przeszukiwać żeby zamiast id_przedmiotu pod ocenami urzywać nazwy przedmiotu - ale raczej to wiele nie zmieni ... czyli dla 2 przedmiotów takich co masz mniej więcej tablica wyglądała by tak:

Kod

Array
(
    [1] => Array
        (
            [nazwa_id_przedm] => j.pol
            [ocena] => Array
                (
                    [0] => Array
                        (
                            [stopien] => 10/20
                            [warznosc] => 1
                        )

                    [1] => Array
                        (
                            [stopien] => 3
                            [warznosc] => 2
                        )
                )
    [2] => Array
        (
            [nazwa_id_przedm] => j.ang
            [ocena] => Array
                (
                    [0] => Array
                        (
                            [stopien] => 5
                            [warznosc] => 2
                        )    
                )
                
)
)

Ten post edytował zegarek84 18.01.2009, 13:33:41