2 x MySQL-while driller

Hvad gør database()?

#1

function database($sql){//Funktion der laver alm databasekald og returnerer resultatet
  $config = ini();//Hent databaseparametre
  mysql_select_db($config['db_base']);
  $res = mysql_query($sql); //Udfør forespørgsel
  if (!$res) {//Tjek om det gik godt
    die("<b>Databasefejl:<br></b>" . mysql_error() ."<b><br>MySQL-sætningen var:<br></b>". $sql);//Udskriv fejlmelding og stop udførsel
  }
  return $res;//Returner resultat af forespørgsel
}

Det fungerer fint, hvis jeg bare gør sådan her, men det er vel unødig belastning af databasen:

$sql  ="SELECT `id` FROM `table1`"; //3 poster
$res1 = database($sql);

    while($row1 = mysql_fetch_array($res1))
    {
      echo $row1['id']."<br />";
        $sql  ="SELECT `tal` FROM `table2`"; //5 poster
        $res2 = database($sql);
        while($row2 = mysql_fetch_array($res2))
        {
            echo $row2['tal']."<br />";
        }
    }

Det må være din logik der ikke passer. Hvad du gør i #0 er absolut ikke det samme som du gør i #3.

I #0 udfører du to sql forespørgsler lige efter hinanden - og så færdig. Resultatet af den anden forespørgsel forsøger du så at genanvende i en while to gange (tre gange i alt) og det kan du ikke med mysql_fetch_array() da indeks pointeren allerede er flyttet helt til slut i den resource.

Ydermere, giver det ikke umiddelbar mening at din database() funktion skal vælge database hver gang du kører funktion - det er unødvendig belastning af serveren. Funktionen sparer dig heller ingenting, da logikken må være at fin funktion alligevel skal gøre tre gange for at kunne gennemløbe den anden forespørgsel tre gange.

Et bedre alternativ ville være at gennemløbe den anden forespørgsel en gang, lægge indholdet i et array og iterere hen over det de tre gange.

Og jeg kunne også have spurgt, hvorfor der ikke sker noget, når "while" køres igennem 2.gang og senere:

$sql  ="SELECT `id` FROM `table1`"; //3 poster
$res1 = database($sql);

    while($row1 = mysql_fetch_array($res1))
    {
      echo $row1['id']."<br />";
    }

    while($row1 = mysql_fetch_array($res1))
    {
      echo $row1['id']."<br />";
    }

#4
OK - jeg forstår.
Det giver også svar på #5.

Array-løsningen virker som den rigtige - så det laver jeg i stedet.

Tak for bidragene og send venligst et svar. :)

Du fik svar her.

Det er unødvendigt at oprette et array. Brug istedet mysql_data_seek() til at resette pointeren i din resource.

#8 OK - det skal da også prøves. Tak for tippet.

#8
Er 'unødvendigt' virkelig det rigtige ord? Selvom det også løser opgaven, hvordan kan det så være unødvendigt' at ville anvende arrays i stedet?

@nemlig Velbekomme.

@repox I langt de fleste programmeringsopgaver er det besværlige ikke at løse opgaven, men derimod at gøre det smart, effektivt og hurtigt. Du bruger her unødvendigt meget RAM, idet du gemmer al dataen to gange.

#11
Så vil jeg nok nærmere kalde det en holdningssag - det er ikke just hukommelsen jeg bekymrer mig om i sådanne tilfælde. Og ofte vil jeg heller ikke lægge en database funktionalitet sammen med output delen, men hellere have et abstraktionslag til at udføre det for mig i et lokalt scope som returnere et array eller objekt jeg kan behandle i stedet. Dette giver mig stadig en MySQL ressource der frigives efter funktionen er kørt.

@repox RAM-forbrug er aldrig en holdningssag. Overvej, hvis det er et 10MB blob udtræk han laver.

Jeg er enig i, at han også burde benytte en MVC-lignende struktur, men det er ikke det denne tråd handler om.

#13
Jeg er bestemt ikke enig med dig. MySQL og PHP anvender ikke samme hukommelse, da de ikke bruger den samme proces. Så udover at man skulle skamme sig over at bruge blobs på 10MB, så er det også et performance-mæssigt spørgsmål som jeg mener falder uden for kategorien her.

For at hente trivielt data - tekst, varekataloger, nyheder og andet skrammel - ud, ser jeg det bestemt ikke som en fordel at bibeholde dataene i en ufleksibel MySQL-ressource. Derfor mener jeg at det er en holdningssag at ville optimere i forhold til RAM forbruget. Skulle det ske at mængden af data fra MySQL ville være et performance-problem vil jeg til enhver tid råde folk til at anvende cache-systemer.

Med andre ord; at du vil genbruge en MySQL ressource i det omfang du vil, er naturligvis op til dig. Jeg ser bare ikke selv nogen fordel i det og anvender derfor andre metoder som jeg mener giver mig den performance som jeg har behov for.
Der er trods alt heller ikke kun en rigtig vej for at nå målet.