Forum PHP.pl > [PHP][MYSQL]Optymalizacja kodu

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [PHP][MYSQL]Optymalizacja kodu

lukas22333

22.12.2008, 18:37:13

Witam, napisałem taki kod, wiem że nie jest za bardzo optymalny( i chce to zmienić ) ale na chwilę obecną działa. Próbowałem coś tam zmieniać ale coś nie wychodzi.

Pierwotna wersja:

CODE

<?
$sql_1 = 'SELECT type, name FROM menu';
$result = mysql_query($sql_1) or die(mysql_error());
$ile=mysql_num_rows($result);
if ($ile>0 && ($result)) {

while ($r=mysql_fetch_assoc($result))
{
echo ''."\n";
echo ''.$r['name'].''."\n";

$sql_2 = 'SELECT txt_id, alias FROM txt WHERE menid="'.$r['type'].'"';
$result_1 = mysql_query($sql_2) or die(mysql_error());

$sql_3 = 'SELECT anchor, adres FROM menu_links WHERE type="'.$r['type'].'"';
$result_3 = mysql_query($sql_3) or die(mysql_error());

while ($r1=mysql_fetch_assoc($result_1))
{
echo '» '.$r1['alias'].''."\n";
while ($r2=mysql_fetch_assoc($result_3))
{
echo '» '.$r2['anchor'].'.'."\n";
}
}

echo ''."\n";
}

}
?>

A teraz moje wypociny:

CODE

<?
$result = mysql_query('SELECT type, name FROM menu') or die(mysql_error());
if ($result) {

while ($r=mysql_fetch_assoc($result))
{
echo ''."\n";
echo ''.$r['name'].''."\n";

$result_1 = mysql_query('SELECT txt.txt_id, txt.menid, txt.alias, menu_links.anchor, menu_links.adres, menu_links.type FROM txt LEFT JOIN menu_links ON (txt.menid=menu_links.type) WHERE txt.menid="'.$r['type'].'" AND menu_links.type="'.$r['type'].'"') or die(mysql_error());

while ($r1=mysql_fetch_assoc($result_1))
{
echo '» '.$r1['alias'].''."\n";
while ($r2=mysql_fetch_assoc($result_1))
{
echo '» '.$r2['menu_links.anchor'].'.'."\n";
}
}

echo ''."\n";
}

}
?>

Podejrzewam, że nie tak jest albo te łączone zapytanie albo dwie pętle po nim.

Pozdrawiam

Zmorcia

22.12.2008, 18:52:52

[PHP] pobierz, plaintext 
<?php
$result = mysql_query('SELECT type, name FROM menu') or die(mysql_error());
 
if ($result) {
       while ($r=mysql_fetch_array($result))
        {  
                  echo '\n';
          echo $r['name'].'\n';
          
 $result_1 = mysql_query('SELECT txt.txt_id, txt.menid, txt.alias, menu_links.anchor, menu_links.adres, menu_links.type FROM txt LEFT JOIN menu_links ON (txt.menid=menu_links.type) WHERE txt.menid=\"'.$r['type'].'\" AND menu_links.type=\"'.$r['type'].'\"') or die(mysql_error());
      
           while ($r1=mysql_fetch_array($result_1))
           {
          echo 'ť '.$r1['alias'].'\n';
                while ($r2=mysql_fetch_array($result_1))
                {
                echo 'ť '.$r2['menu_links.anchor'].'\n';
                }
           }
         echo '\n';
             }
    }
?>
[PHP] pobierz, plaintext

// jaki błąd ci wywalało?

lukas22333

22.12.2008, 21:27:31

Nikt nie wie jak to skrocic ?

Tak by było 1 zapytanie.

CODE

while ($r=mysql_fetch_assoc($result))
{
echo ''."\n";
echo ''.$r['name'].''."\n";

$sql_2 = 'SELECT txt_id, alias FROM txt WHERE menid="'.$r['type'].'"';
$result_1 = mysql_query($sql_2) or die(mysql_error());

$sql_3 = 'SELECT anchor, adres FROM menu_links WHERE type="'.$r['type'].'"';
$result_3 = mysql_query($sql_3) or die(mysql_error());

while ($r1=mysql_fetch_assoc($result_1))
{
echo '» '.$r1['alias'].''."\n";
while ($r2=mysql_fetch_assoc($result_3))
{
echo '» '.$r2['anchor'].'.'."\n";
}
}

echo ''."\n";
}

up..

Riklaunim

22.12.2008, 22:20:08

Nie zawsze 1 zapytanie jest lepsze. Stosowanie JOINów może wydawać się ładne i takie "pro", ale dość często przy większym obciążeniu zapytania takie mogą mocno obciążać bazę danych.

lukas22333

22.12.2008, 22:28:30

To co w tej pierwotnej wersji proponujesz zminienic? zamiast tych zapytań.

pinochet

23.12.2008, 02:29:07

Cytat(Riklaunim @ 22.12.2008, 22:20:08 )

Nie zawsze 1 zapytanie jest lepsze.

Prosiłbym o przykład.

[SQL] pobierz, plaintext 
SELECT 
txt.txt_id, 
txt.menid, 
txt.alias, 
menu_links.anchor, 
menu_links.adres, 
menu_links.type 
FROM txt, menu_links, menu
 WHERE txt.menid=menu_links.type AND txt.menid=menu.type
[SQL] pobierz, plaintext

To tak na oko.
Nie chce mi sie na podstawie kodu rozkminiac twojej struktury bazy danych ale stosowanie relacji 1-1 a potem łączenie tabel aby zrobić SELECT-a jest raczej mało optymalne.

nexik

23.12.2008, 17:59:04

nie wykonuj kolejnych zapytan w petli, lepiej zrob sobie tablice ktora bedzie przechowywac kolejne slowniki a w petli tylko korzystaj z zmiennych php a nie operuj na zapytaniach sql

lukas22333

23.12.2008, 23:34:22

Cytat(nexik @ 23.12.2008, 17:59:04 )

nie wykonuj kolejnych zapytan w petli, lepiej zrob sobie tablice ktora bedzie przechowywac kolejne slowniki a w petli tylko korzystaj z zmiennych php a nie operuj na zapytaniach sql

Mozesz jasniej? bo nie bardzo rozumie. Jakis przyklad.

@up

pinochet

24.12.2008, 02:24:31

według mnie to powinno działać szybciej:

[PHP] pobierz, plaintext 
<?
$sql_1 = 'SELECT `txt`.`alias`, `menu`.`name`             FROM `txt`, `menu`             WHERE `txt`.`menid`=`menu`.`type` ORDER BY `menu`.`type` ';
$sql_2 = 'SELECT `menu_links`.`anchor`, `menu`.`name`     FROM `menu_links`, `menu`     WHERE `menu_links`.`type`=`menu`.`type` ORDER BY `menu`.`type`';
$result1 = mysql_query($sql_1) or die(mysql_error());
$result2 = mysql_query($sql_2) or die(mysql_error());
 
 
while ($r=mysql_fetch_assoc($result1)){
    $aliasy[$r['name']][] = $r['alias'];
}
 
while ($r=mysql_fetch_assoc($result2)){
    $submenu[$r['name']] .= 'ť '.$r['anchor'].".\n";
}
 
echo "\n";
 
foreach($menu as $nazwa => $aliasy){
    echo $nazwa."\n\n";
    foreach($aliasy as $jeden){
        echo 'ť '.$jeden."\n";
        echo $submenu[$nazwa];
    }
}
 
echo "\n";
}
?>
[PHP] pobierz, plaintext

Fajna łamigłówka

lukas22333

24.12.2008, 11:11:51

Warning: Invalid argument supplied for foreach() in ...... on line 18

Linijka 18 to ta:

CODE

foreach($menu as $nazwa => $aliasy){

A kod lamiglowka bo nie wiem jak to inaczej by zrobic

pinochet

25.12.2008, 00:23:44

oczywiscie nie $menu tylko $aliasy

Astarot

25.12.2008, 00:31:15

Dobry czlowieku:

Poznaj skladnie PHP
Nastpenie Poznaj Skladnie MySQL

Pytania z ktorymi wyskakujesz sa tak oczywiste i banalne, ze az mi za Ciebie wstyd!

Pozatym, nigdzie nie widze, zadnego mechanizmu cachowania...

chcesz zeby bylo optymalnie zasanow sie kiedy trzeba sie odwolowac do bazy danych, tabele mazwales memu badz podobnie a takie dane powinny byc statyczne anyway: imo, z tych strzepow informacji ktore uzyskalem wydaje mi sie ze baza jest zaprojekowana zle! Wklej caly kod tabli pokaz zaptania, napisz jaki masz cel i czego oczekujesz, poradzimy cos wiecej!

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.