Forum PHP.pl

Witam,


Mam w bazie w tabeli okolo 350 rekordow, wyswietlenie ich na stronie trwa okolo 1min lub dluzej. Jest to zdecydowanie za dlugo. Ponizej przestawiam swoj kod. Czy ktos moze mi powiedziec jak go zoptymalizowac zeby wyniki wyswietlaly sie szybciej

[PHP] pobierz, plaintext 
<?php
if($_GET['id_mag'] != '111') { // jezeli != 111 wyswietlam wszystkie lokalizacje  
      $where = "WHERE u.magazyn = '$_GET[id_mag]' AND u.magazyn = l.id_mag";
    }
        
    $query = ("SELECT u.id,
        l.nazwa, 
        u.umowa, 
        u.tresc, 
        u.data_zaw, 
        u.data_zak, 
        u.wypow, 
        u.kontrahent, 
        u.nip, 
        u.stawka, 
        u.kaucja,
        u.magazyn, 
        u.stan,
        u.uwagi
        FROM umowy u, lokalizacje l 
        {$where}
        {$order}
        ");
 
echo '<table class="umowy" width="1600" border="1">';
    echo '<th><a href="list_umowy.php?id_mag='.$_GET['id_mag'].'&sort=l'.$sort.'">Lokalizacja</a></th>';
    echo '<th><a href="list_umowy.php?id_mag='.$_GET['id_mag'].'&sort=n'.$sort.'">Nr umowy</a></th>';
    echo '<th><a href="list_umowy.php?id_mag='.$_GET['id_mag'].'&sort=p'.$sort.'">Treść</a></th>';
    echo '<th><a href="list_umowy.php?id_mag='.$_GET['id_mag'].'&sort=z'.$sort.'">Data zawarcia</a></th>';
    echo '<th><a href="list_umowy.php?id_mag='.$_GET['id_mag'].'&sort=t'.$sort.'">Data zakończenia</a></th>';
    echo '<th>Wypowiedzenie</th>';
    echo '<th><a href="list_umowy.php?id_mag='.$_GET['id_mag'].'&sort=k'.$sort.'">Kontrahent</a></th>';
    echo '<th><a href="list_umowy.php?id_mag='.$_GET['id_mag'].'&sort=i'.$sort.'">NIP</a></th>';
    echo '<th>Stawka</th>';
    echo '<th>Kaucja</th>';
    echo '<th>Uwagi</th>';
    echo '<th>Stan</th>';
    echo '<th>Edycja/Usuń</th></tr>';
    
    while($wynik = mysql_fetch_array($wyswietl))    
            {
                    if($wynik['data_zak'] == '2132578800')
                    {     $data_zak = "nieokreślony"; }
                    else
                    { $data_zak = @date('Y-m-d',$wynik['data_zak']);    }
                    
                    if($wynik['stan'] == '0')
                        $wyg = ((++$parz % 2)?'1':'2') . '"\"';
                    else
                        $wyg = '3';             
                                                          
              echo '<tr class="dane'.$wyg.'"><td width="120">'. $wynik['nazwa'].'</td>';
            echo '<td width="100">'. $wynik['umowa'].'</td>';
            echo '<td width="250">'. $wynik['tresc'].'</td>';
            echo '<td width="100">'. @date('Y-m-d',$wynik['data_zaw']).'</td>';
            echo '<td width="130">'. $data_zak .'</td>';
            echo '<td>'. $wynik['wypow'].' dni</td>';
            echo '<td>'. $wynik['kontrahent'].'</td>';
            echo '<td>'. $wynik['nip'].'</td>';
            echo '<td width="200">'. $wynik['stawka'].'</td>';
            echo '<td>'. $wynik['kaucja'].'</td>';
            echo '<td width="150">'. $wynik['uwagi'].'</td>';
            echo '<td width="40"><center><a href="list_umowy.php?act=stan&id_um=' .$wynik['id']. '&mag='.$wynik['magazyn'] .'&stan='.$wynik['stan'] .'"><img src="image/act.png" width="20" height="20" alt="Stan Umowy" /></a></center></td>'; 
            echo '<td width="80">
            <a href="list_umowy.php?act=edit&id_um=' .$wynik['id']. '&mag='.$wynik['magazyn'] .'"><img src="image/edit.png" width="16" height="16" alt="Edycja" /></a>   
            <a href="list_umowy.php?act=del&id=' .$wynik['id']. '&mag='.$wynik['magazyn'].'" onClick="return confirm('Czy usunąć umowę?')"><img src="image/drop.png" width="16" height="16" alt="Usuń" /></a></td></tr>';  
            }
?>
[PHP] pobierz, plaintext 

Dodac paginacje.

stronnicowanie? klik

Cytat

a co to jest 350 rekordów gdybyś póścił nawet to w dwóch pętlach to moim zdaniem powinno się wykonać chwilę , ale żeby to przyśpieszyć możesz sprubować z left join są one szybsze od takich złączeń

1. Sprawdzić, ile czasy wykonuje się zapytanie i je zoptymalizować
2. Oduczyć się wysyłania strony do przeglądarki na raty - echo używamy RAZ
3. Zoptymalizować kod html, poczytać jakiś kurs html - ten jest z innej epoki i będzie mulić ZAWSZE
4. Porcjować wyniki z bazy (stronicowanie)
5. Dobrze jest też oddzielić kod HTML od PHP

A indeksy w bazie, to masz?

ja bym chyba zamiast SELECT u.id, u.cosTam, u.CostTam dał: SELECT * i tak jak wcześniej napisali - stronicowanie przez LIMIT

Indeksy mam, zapytanie z wyswietleniem 350 rekordow przez mysql trwało 0.0037 sekund wiec to raczej nie jest wina zapynia sql
zrobilem wyswietlanie w dwoch ech'ach ale dalej tak zmula :/

ale w SQL odwołujesz się bezpośrednio do servera SQL, a jeśli robisz to na stronie dodatkowo zaprzęgasz do tego paser PHP. i właśnie tu pojawia się problem stronicowania.

ale stronicowanie wykonuj wlasnie za pomocą zapytań SQL, bo w innym wypadku i tak będziesz wyciagal dużo danych, a później bedziesz musiał je obrabiać... SQL ma bardzo duże możliwości!

@lutador: moze strona sie RENDERUJE dlugo - a nie generuje?
Usun na razie generowanie tabelki i wypisuj tylko : ID + nazwa + <br /> i stopniowo dodawaj wiecej danych i tabelki i zobacz co wyjdzie.
I testujesz na localu czy na serwerze zewnetrznym?

probowalem juz usunac czesc pol zeby wszystkiego nie wyswietlal i bylo troche szybciej, ale sproboje tak jak mowisz i wywale calkiem tabele
Do serwera mam predkosc 100MB

A nie widzisz, że:

[PHP] pobierz, plaintext 
<?php
echo '<a href="list_umowy.php?act=edit&id_um=' .$wynik['id']. '&mag='.$wynik['magazyn'] .'">
<img src="image/edit.png" width="16" height="16" alt="Edycja" /></a>';
?>
[PHP] pobierz, plaintext 

- każdy taki element to jedno zapytanie http do serwera? W dodatku ile kodu! Powinno być tak:

[PHP] pobierz, plaintext 
<?php
echo '<a class="edit" title="edycja" href="list_umowy.php?act=edit&id_um=' .$wynik['id']. '&mag='.$wynik['magazyn'] .'"></a>';
?>
[PHP] pobierz, plaintext 

- widzisz różnicę?

Pilsner: ale o czym ty mowisz?
Ze niby 350 edit-ow.png sie wczytuje? Akurat tylko jeden sie wczyta, przegladarka to optymalizuje.