Optimering af SQL-forespørgsler

OBS: Jeg undskylder mine 2 tidligere ens oprettede tråde, men den ville ikke tage mine point med. Det lykkedes så dog nu.

Dine menuer er vel næppe så store, at de ikke kan rummes i hukommelsen.

En mulighed er - hvis man skal følge den idé - at starte med én query, hvor du henter alt i menuen ind i arrays.

Dine while skal så laves om til foreach efterfulgt at en if (menu_cat = xxx), der sørger for, at du kun processerer relevante items. Og indeholder så ikke nogle queries.

Er det forståeligt og brugbart - som koncept? I så fald kan vi arbejde videre med det.

Hej acore.

Hmm, med mine nuværende menupunkter så trækker den menu alene 114 forespørgsler - det synes jeg er i overkanten, og det kan ligeså mærkes på sidens hastighed.

Hehe, nu er jeg langt fra den skarpeste kniv i skuffen til det her. Alene det jeg har fået skruet sammen indtil nu - ja det har taget mig på den anden side af et helt år. Det være sig ved utallige mange timers læsning/søgning, samt ikke mindst ca. en milliard forsøg på at få skidtet til at virke. Det har jeg så dog langt om længe opnået - men blot for at finde ud af at den bruger for mange queries.

Jeg ved ikke helt, jeg er nok ikke den rette til at svare på om dit forslag er brugbart, eftersom jeg ikke ved hvordan/hvad der skal gøres/hvordan det vil komme til at evt. virke.

Årh. helt ueffen er du jo ikke - det ved jeg. For at forklare det bedre, så se denne kode. Den er IKKE færdig, men jeg har skrevet lidt ind, der bedre forklare ideen:

//  ============================== Main Menu Start ============================== //
$msql = dbquery("SELECT * FROM ".DB_MENUS." ORDER BY menu_order"); // Tag alt med
if (dbrows($msql) != 0) {
  while ($mdata = dbarray($msql)) {
    $m_id = $mdata['menu_id'];
    $m_cat[] = $mdata['menu_cat'];
    $m_access[] = $mdata['menu_access'];
    $m_window[] = $mdata['menu_window'];
    $m_link[] = $mdata['menu_link'];
    $m_name[] = $mdata['menu_name'];
    // ... tilføj dem der mangler
  }
  $n = count($m_id);

  echo "<ul id='nav'>";
  //while ($mdata = dbarray($msql)) {
  for ($i = 0; $i < $n; $i++) {
    if ($m_cat[$i] == 0 && checkgroup($mdata['menu_access'])) { // Her tilføjer jeg det kriterie jeg nu ikke har i SELECY
    // I al efterfølgende kode skal $mdata['xxx'] rettes til $m_xxx[$i]
      $link_target = ($mdata['menu_window'] == "1" ? " target='_blank'" : "");
        if(dbrows(dbquery("SELECT menu_cat FROM ".DB_MENUS." WHERE menu_cat='".$mdata['menu_id']."'"))) {
          $div_class = " class='down'";
          } else {
            $div_class = "";
          }
            if ($mdata['menu_name'] !== "---" && $mdata['menu_link'] == "---") {
              if (preg_match("!^(ht|f)tp(s)?://!i", $mdata['menu_link'])) {
                echo "<li class='top'><div".$div_class."><a href='".$mdata['menu_link']."'".$link_target." class='top_link'>".parseubb($mdata['menu_name'], "b|i|u|img|color")."</a></div>\n";
              } else {
                echo "<li class='top'><div".$div_class."><a href='".BASEDIR.$mdata['menu_link']."'".$link_target." class='top_link'>".parseubb($mdata['menu_name'], "b|i|u|img|color")."</a></div>\n";
              }
            } else if ($mdata['menu_name'] !== "---" && $mdata['menu_link'] == "") {
              if (preg_match("!^(ht|f)tp(s)?://!i", $mdata['menu_link'])) {
                echo "<li class='top'><div".$div_class."><a href='".$mdata['menu_link']."'".$link_target." class='top_link'>".parseubb($mdata['menu_name'], "b|i|u|img|color")."</a></div>\n";
              } else {
                echo "<li class='top'><div".$div_class."><a href='".BASEDIR.$mdata['menu_link']."'".$link_target." class='top_link'>".parseubb($mdata['menu_name'], "b|i|u|img|color")."</a></div>\n";
              }
            } else if ($mdata['menu_name'] == "---" && $mdata['menu_link'] == "---") {
              echo "<li class='top'><hr />\n";
            } else {
              if (preg_match("!^(ht|f)tp(s)?://!i", $mdata['menu_link'])) {
                echo "<li class='top'><div".$div_class."><a href='".$mdata['menu_link']."'".$link_target." class='top_link'>".parseubb($mdata['menu_name'], "b|i|u|img|color")."</a></div>\n";
              } else {
              echo "<li class='top'><div".$div_class."><a href='".BASEDIR.$mdata['menu_link']."'".$link_target." class='top_link'>".parseubb($mdata['menu_name'], "b|i|u|img|color")."</a></div>\n";
              }
            }
//  ============================== Sub Menu 1 Start ============================== //
      // $msql1 = dbquery("SELECT * FROM ".DB_MENUS." WHERE menu_cat='".$mdata['menu_id']."' ORDER BY menu_order"); Dne skal vi ikke bruge nu
      if (dbrows($msql1) != 0) {
        echo "<ul class='sub'>";
        //while ($mdata1 = dbarray($msql1)) {
        for ($i1 = 0; $i1 < $n; $i1++) {
          if ($m_cat[$i1] == $m_id[$i] && checkgroup($mdata1['menu_access'])) { // her tilføjes så udvælgelseskriteriet fra den andet SELECT
    // ...og $mdata1['xxx'] rettes til $m_xxx[$i1]
            $link_target = ($mdata1['menu_window'] == "1" ? " target='_blank'" : "");
              if(dbrows(dbquery("SELECT menu_cat FROM ".DB_MENUS." WHERE menu_cat='".$mdata1['menu_id']."'"))) { // Denne optælling kanogså laves direkte i data og uden SELECT
              $link_class = " class='fly'";
              } else {
                $link_class = "";
              }
                if ($mdata1['menu_name'] !== "---" && $mdata1['menu_link'] == "---") {
                  echo "<li><b>".parseubb($mdata1['menu_name'], "b|i|u|left|center|right|img|color")."</b>\n";
                } else if ($mdata1['menu_name'] == "---" && $mdata1['menu_link'] == "---") {
                  echo "<li><hr />\n";
                } else {
                  if (preg_match("!^(ht|f)tp(s)?://!i", $mdata1['menu_link'])) {
                    echo "<li><a href='".$mdata1['menu_link']."'".$link_target."".$link_class.">".parseubb($mdata1['menu_name'], "b|i|u|left|center|right|img|color")."</a>\n";
                  } else {
                    echo "<li><a href='".BASEDIR.$mdata1['menu_link']."'".$link_target."".$link_class.">".parseubb($mdata1['menu_name'], "b|i|u|left|center|right|img|color")."</a>\n";
                  }
                }
//  ============================== Sub Menu 2 Start ============================== //
            // $msql2 = dbquery("SELECT * FROM ".DB_MENUS." WHERE menu_cat='".$mdata1['menu_id']."' ORDER BY menu_order");
            if (dbrows($msql2) != 0) {
              echo "<ul>";
              // while ($mdata2 = dbarray($msql2)) {
              for ($i2 = 0; $i2 < $n; $i2++) {
                if ($m_cat[$i2] == $m_id[$i1] && checkgroup($mdata2['menu_access'])) {  // og her kriteriet fra den 3. SELECT
    // ...og $mdata2['xxx'] rettes til $m_xxx[$i2]
                  $link_target = ($mdata2['menu_window'] == "1" ? " target='_blank'" : "");
                    if(dbrows(dbquery("SELECT menu_cat FROM ".DB_MENUS." WHERE menu_cat='".$mdata2['menu_id']."'"))) {
                    $link_class = " class='fly'";
                    } else {
                      $link_class = "";
                    }
                      if ($mdata2['menu_name'] !== "---" && $mdata2['menu_link'] == "---") {
                        echo "<li><b>".parseubb($mdata2['menu_name'], "b|i|u|left|center|right|img|color")."</b>\n";
                      } else if ($mdata2['menu_name'] == "---" && $mdata2['menu_link'] == "---") {
                        echo "<li><hr />\n";
                      } else {
                        if (preg_match("!^(ht|f)tp(s)?://!i", $mdata2['menu_link'])) {
                          echo "<li><a href='".$mdata2['menu_link']."'".$link_target."".$link_class.">".parseubb($mdata2['menu_name'], "b|i|u|left|center|right|img|color")."</a>\n";
                        } else {
                          echo "<li><a href='".BASEDIR.$mdata2['menu_link']."'".$link_target."".$link_class.">".parseubb($mdata2['menu_name'], "b|i|u|left|center|right|img|color")."</a>\n";
                        }
                      }
//  ============================== Sub Menu 3 Start ============================== //
                  //$msql3 = dbquery("SELECT * FROM ".DB_MENUS." WHERE menu_cat='".$mdata2['menu_id']."' ORDER BY menu_order");
                  if (dbrows($msql3) != 0) {
                    echo "<ul>";
                    //while ($mdata3 = dbarray($msql3)) {
                    for ($i3 = 0; $i3 < $n; $i3++)
                      if ($m_cat[$i3] == $m_id[$i2] && checkgroup($mdata3['menu_access'])) { // Do her
    // ...og $mdata3['xxx'] rettes til $m_xxx[$i3]
                        $link_target = ($mdata3['menu_window'] == "1" ? " target='_blank'" : "");
                          if ($mdata3['menu_name'] !== "---" && $mdata3['menu_link'] == "---") {
                            echo "<li><b>".parseubb($mdata3['menu_name'], "b|i|u|left|center|right|img|color")."</b>\n";
                          } else if ($mdata3['menu_name'] == "---" && $mdata3['menu_link'] == "---") {
                            echo "<li><hr /></li>\n";
                          } else {
                            if (preg_match("!^(ht|f)tp(s)?://!i", $mdata3['menu_link'])) {
                              echo "<li><a href='".$mdata3['menu_link']."'".$link_target.">".parseubb($mdata3['menu_name'], "b|i|u|left|center|right|img|color")."</a></li>\n";
                            } else {
                              echo "<li><a href='".BASEDIR.$mdata3['menu_link']."'".$link_target.">".parseubb($mdata3['menu_name'], "b|i|u|left|center|right|img|color")."</a></li>\n";
                            }
                          }
//  ============================== Sub Menu 3 End ============================== //
                        }
                    }
                    echo "</ul>\n";
                  }
                  echo "</li>\n";
//  ============================== Sub Menu 2 End ============================== //
                }
              }
              echo "</ul>\n";
            }
            echo "</li>\n";
//  ============================== Sub Menu 1 End ============================== //
          }
        }
        echo "</ul>\n";
      }
      echo "</li>\n";
//  ============================== Main Menu End ============================== //
    }
  }
  echo "</ul>\n";
}

Hej igen acore.

Nu har jeg virkelig givet det rigtig rigtig mange forsøg, men dog uden at kunne få det til at virke.

Jeg har derfor droppet at arbejde videre på menuen.

Du skal i hvert fald have mange tusind tak for dine inputs!

Hvis du smider et svar så kan vi lige afregne.

Fortsat god dag!

Mvh.
Buxxy.

Arh, det var ærgerligt.

Et lille indspark: menuen behøver vel ikke blive genberegnet for hver side. Den ændrer sig vel ikke så tit...?

Hvad med at lave et særligt PHP-script, som beregner HTML-koden for menuen, og gemmer den i et LONGTEXT felt i databasen. Det hentes så op med eet SQL-kald.

Hver gang du har ændringer, som bevirker at menuen skal ændres, så kan du kan kalde scriptet igen.

Så er det egentlig lidt lige meget hvor lang tid det tager.

@ej: Hmm, jeg må desværre melde klart ud at jeg ikke ved hvad du mener - jeg kan kun copy/paste mht. koderiet.

Men du skal også have mange tusind tak for dit input.

Tror jeg holder en pause nu fra alt det her "skidt" - har sgu været i gang så længe nu, at jeg nærmest føler at hovedet er ved at "poppe af"....  ;-)

Anyway - så mange tak til Jer begge to!