[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ć (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) .

Mam 2 tabele w mysql:
(IMG:http://i43.tinypic.com/2rh4591.png)

(IMG:http://i44.tinypic.com/1g11tg.png)

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ć)

(IMG:http://i42.tinypic.com/kevzew.png)


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?

[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