Optimering samt hjælp til løkke

Du kan lige prøve:

$i = 1;
while ($row = mysql_fetch_array($query)) {
  if(($i%2) == 0) {
    echo "<br />";
  }
  echo "<a href=\"index.php?option=com_content&task=view&id=$row[id]&Itemid=$catid\">$row[title]</a>";
  $i++;
}

Det var tæt på.
$i = 1;
while ($row = mysql_fetch_array($query)) {
  if(($i%4) == 0) {
    echo "<br />";
  }
  echo "<a href=\"index.php?option=com_content&task=view&id=$row[id]&Itemid=$catid\">$row[title]</a>"." - ";
  $i++;
}
}

Løste da det problem...

ovenstående udskriver med andre ord 3 links pr. række...

Åh, tanketorsk fra mig, det er blevet for sent tror jeg. Egentlig ville jeg have skrevet $i%3. Din vil da give 4 per række?

Nej underligt nok ikke... den giver 3 pr. række

Nej, det er vist en tilfældighed hos dig, hvis du ikke har så mange rækker.

Det skal være $i%3 og så skal
$i = 1;
være
$i = 0;

Jeg kan aldrig huske om den skal være 0 eller 1, af en eller anden mærkelig grund.

Hvor stor en testmængde har du brugt? Jeg kan godt gå med til første række bliver på 3, men de næste bliver så vidt jeg kan se på 4.

Det ved jeg ikke... har kun prøvet med 4 links totalt... prøver lige at smide lidt flere på. Men burde det ikke være ens hele tiden?

nej, det kommer jo an på startværdien. $i%4 vil dividere med 4, og så tage restværdien og returnere. Dvs. at hvis du har 4 og en startværdi på 1, bliver restværdien 0 efter 3 gennemløb, men næste gang skal der 4 gennemløb til for at have en rest på 0.

Forståeligt? :)

Ikke med en startværdi på 1. Som jaw siger, skal det køre sådan:

0 - 1 - 2
3 - 4 - 5
6 - 7 - 8

mens dit kommer til at køre sådan:

1 - 2 - 3
4 - 5 - 6 - 7
8 - 9 - 10 - 11

jep, den er forstået :) og det virker bedre med en startværdi på 0 - dog undrer det mig, at den så starter med et <br> i stedet for en række links?

Eller det er jo ikke så mærkeligt når $i = 0

Nej, det er nemlig præcis ikke mærkeligt :) Men du kan flytte din echo-linie ovenover din if, så skulle den være der.

Hvis jeg flytter den oven over min if kommer alle links på en ny række ...

Hvordan gør du det da?

Og det er så grunden til, at jeg aldrig kan huske om det skal være 0 eller 1 - eller også er jeg mere træt nu end jeg troede :)

For hvis du smider echo før din if, så skal startværdien være for $i være 1. Men så skulle det altså også virke.

prøver lige...

Prøvede med:

    echo "<br /><b>Indholdssider</b><br />";
    $query = mysql_query("SELECT * FROM jos_content WHERE catid = $catid ORDER BY title ASC");
       
    $i = 1;
    while ($row = mysql_fetch_array($query)) {
        echo "<br />";
      if(($i%3) == 0) {
      }
          echo "<a href=\"index.php?option=com_content&task=view&id=$row[id]&Itemid=$catid\">$row[title]</a>"." - ";
      $i++;
}

Men det giver kun 1 links pr række...

Det var nu ikke den echo-linie jeg mente :) Sådan her:

    $i = 1;
    while ($row = mysql_fetch_array($query)) {
          echo "<a href=\"index.php?option=com_content&task=view&id=$row[id]&Itemid=$catid\">$row[title]</a>"." - ";
      if(($i%3) == 0) {
        echo "<br />";
      }
      $i++;
}

Det er da ren gøg og gokke det her :o)

Det skal så være if ($i%4 == 0)...

Se her se... det hjalp :)
Så skal jeg bare se om jeg kan gøre noget ved alle mine queries... ville helst ikke have så mange hvis jeg kunne undgå det, men burde jeg oprette et nyt spørgsmål med dette?

Testet Rednex?

Forstår ikke helt hvad du mener med det :) ¿?

Det var ikke til dig, men til Rednex :)

Ang. dine queries, så er det svært at sige om du kan undvære nogen, når jeg ikke f.eks. kan se din side. Hvis ikke dataene har noget tilfælles, så bliver det nok svært at gøre det enklere...

jaw: Nej, jeg har ikke testet det, og nu får jeg lige læst ordentligt efter og set at det skulle være 3 i hver række og ikke 4 :oP

Rednex - Netop :) Men vi har fået rodet rundt i det, det medgiver jeg dig...

daniel_d: Uden helt at kunne gennemskue sammenhængen i dine data, vil jeg tro at du ville kunne få glæde af JOINs i dine queries. Jeg vil gerne hjælpe dig, men det kræver at du lige giver en beskrivelse af de data du har i DB'en og en evt. sammenhæng mellem de forskellige tabeller. Og så selvfølgelig en beskrivelse af hvilket resultat vi arbejder hen imod :o)

ok, men i hvert fald virker det vigtigste nu. jeg har føjet dataerne ind i en tabel nu og følgende virker som det skal. Spørgsmålet er så om det kan optimeres, men flytter det gerne til at andet spørgsmål hvis det er...

<?php
$option = $_GET['option'];
$task = $_GET['task'];

if ($option=="com_content" && $task=="view") {

    echo "<table><tr>";
    echo "<td colspan=\"3\"><b>Indholdssider</b></td></tr><tr>";
    $query = mysql_query("SELECT jos_categories.id AS catid, jos_categories.title AS cattitle, jos_content.catid, jos_content.id AS contentid, jos_categories.title FROM jos_categories, jos_content WHERE jos_categories.title=jos_content.title");
    $i = 1;
    while ($row = mysql_fetch_array($query)) {
        echo "<td><a href=\"index.php?option=com_content&task=view&id=$row[contentid]&Itemid=$row[catid]\">$row[cattitle]</a></td>";
      if(($i%3) == 0) {
        echo "</tr><tr>";
      }
      $i++;
    }
    echo "</tr>";

    $id = intval( mosGetParam( $_REQUEST, 'id', null ) );
    $query = "SELECT catid FROM jos_content WHERE id=$id";
    $database->setQuery( $query );
    $catid = $database->loadResult();
   
    echo "<tr>";
    echo "<td colspan=\"3\"><b>Indholdssider</b></td></tr><tr>";
    $query = mysql_query("SELECT * FROM jos_content WHERE catid = $catid ORDER BY title ASC");
    $i = 1;
    while ($row = mysql_fetch_array($query)) {
          echo "<td><a href=\"index.php?option=com_content&task=view&id=$row[id]&Itemid=$catid\">$row[title]</a></td>";
      if(($i%3) == 0) {
        echo "</tr><tr>";
        }
      $i++;
    }
    echo "</tr></table>";
}
?>

jeg benytter et JOIN i første løkke.

Jeg benytter kun 2 tabeller som er relateret i databasen mellem jos_categories.id og jos_content.catid

dataerne stammer fra et CMS som jeg laver et udtræk fra så jeg får en "katalogvisning" af mini sider og kategorier i overskriftsform.

Se evt. koden live på http://ad.eyemedia.dk nederst på siden (testside)

mine sider even...

I skal klikke på menulinket rejser før at I kan se "kataloget"

Denne del:
    $id = intval( mosGetParam( $_REQUEST, 'id', null ) );
    $query = "SELECT catid FROM jos_content WHERE id=$id";
    $database->setQuery( $query );
    $catid = $database->loadResult();
   
    echo "<tr>";
    echo "<td colspan=\"3\"><b>Indholdssider</b></td></tr><tr>";
    $query = mysql_query("SELECT * FROM jos_content WHERE catid = $catid ORDER BY title ASC");

Vil jeg mene du kan lave til:
    $id = intval( mosGetParam( $_REQUEST, 'id', null ) );
    echo "<tr>";
    echo "<td colspan=\"3\"><b>Indholdssider</b></td></tr><tr>";
    $query = mysql_query("SELECT * FROM jos_content WHERE catid = $id ORDER BY title ASC");

Men ellers tror jeg ikke jeg kan se andet at hente umiddelbart - men det er da også 1 query mindre...

Fantastisk :) - det virker sgu!...

Og så den lange :o)

SELECT cat.id AS catid, cat.title AS title, con.id AS contentid FROM jos_categories cat LEFT JOIN jos_content con ON (cat.title = con.title)

Ups... der var jeg sgu for hurtig. Ved at fjerne det query mister jeg desværre et id jeg skal bruge i min URL

Oh, i url'en, ja det havde jeg så ikke set. Men så hiver du det jo bare ud fra den "nye" query og bruger det i stedet for.

Rednex... øhhh :D

Hvor meget skal dit query erstatte. Dem alle?

Har erstattet mit første query med dit forslag Rednex og det virker også fint.

$query = mysql_query("SELECT jos_categories.id AS catid, jos_categories.title AS cattitle, jos_content.catid, jos_content.id AS contentid, jos_categories.title FROM jos_categories LEFT JOIN jos_content ON (jos_categories.title=jos_content.title)");

For lige at summere op så ser koden således ud nu:

<?php
$option = $_GET['option'];
$task = $_GET['task'];

if ($option=="com_content" && $task=="view") {

    echo "<table><tr>";
    echo "<td colspan=\"3\"><b>Kategorier</b></td></tr><tr>";
    $query = mysql_query("SELECT jos_categories.id AS catid, jos_categories.title AS cattitle, jos_content.catid, jos_content.id AS contentid, jos_categories.title FROM jos_categories LEFT JOIN jos_content ON (jos_categories.title=jos_content.title)");
    $i = 1;
    while ($row = mysql_fetch_array($query)) {
        echo "<td><a href=\"index.php?option=com_content&task=view&id=$row[contentid]&Itemid=$row[catid]\">$row[cattitle]</a></td>";
      if(($i%3) == 0) {
        echo "</tr><tr>";
      }
      $i++;
    }

    $id = intval( mosGetParam( $_REQUEST, 'id', null ) );
    $query = "SELECT catid FROM jos_content WHERE id=$id";
    $database->setQuery( $query );
    $catid = $database->loadResult();
 
    echo "<tr><td colspan=\"3\"><b>Indholdssider</b></td></tr><tr>";
    $query = mysql_query("SELECT * FROM jos_content WHERE catid = $catid ORDER BY title ASC");
    $i = 1;
    while ($row = mysql_fetch_array($query)) {
          echo "<td><a href=\"index.php?option=com_content&task=view&id=$row[id]&Itemid=$catid\">$row[title]</a></td>";
      if(($i%3) == 0) {
        echo "</tr><tr>";
        }
      $i++;
    }
    echo "</tr></table>";
}
?>

Prøv at smide den ind i stedet for din første query. Det er en "rigtig" join.

Jeg har fjernet den ene title (du hev jos_categories.title ud to gange med to forskellige feltnavne).

Det ville altså stadig hjælpe lidt hvis du ville give et praj om hvilke felter de to tabeller indeholder, og hvad de bliver brugt til.

Jeg giver lige mine antagelser, så må du lige rette hvis den er gal:
--- jos_content ---
id = autoincrement for indhold
catid = en reference til en kategori fra jos_categories
title = sidens navn/overskrift

--- jos_categories ---
id = siger sig selv
title = kategoriens navn

Hvis den er helt gal må du lige sige til :o)

Det var godt gættet... Det passer sgu :)
Der er også autoincrement på jos_categories.id

jos_content.catid refererer til jos_categories.id

OK, nu tror jeg at jeg har forstået problemet og det er jo mindst halvdelen af arbejdet :o)

Din første query skal hente kategorierne, og den anden skal vise indholdssiderne i den valgte kategori. Hvis det er tilfældet, så tror jeg det hele kan gøres en del nemmere/pænere, men du må godt lige bekræfte det inden jeg begynder at "skrue" på koden :o)

Er det overhovedet nødvendigt at sende id (jos_contentid) for at vælge en kategori?

det er korrekt forstået.
Og ja, det er nødvendigt at bruge jos_contentid for at vælge en kategori, da det egentlig ikke er en kategori jeg vælger, men en side med samme navn som kategorien og derfor skal jeg bruge dette id.

OK, det forklarer jo så hvorfor du joiner på title i den første. Giv mig lige lidt tid, så strikker jeg lige noget kode sammen.

Det lyder jo ikke mindre end fantastisk :)
må vist hellere smide nogle flere point i puljen.

Men det er vist ole lukøje for mig nu... men glæder mig til at se hvad du kan trylle frem :)

Du må sove godt, så sørger jeg for at der ligger en stump kode når du vågner :o)

Så har jeg fikset et par eksempler.

Jeg har afskaffet dit 2. query igen, da vi alligevel sender catid med i URL'en hver gang. Den første eksempel er standard versionen, hvor jeg har lavet en funktion til at udskrive links'ne. Det var mest for at få noget af alt "fyldet" væk.

Det andet eksempel er en lidt mere ekstrem tilgang til at fjerne duplikeret kode. Jeg vil også skyde på at det er den der performer bedst af de to.

Det første eksempel:
<?php
$option = $_GET['option'];
$task = $_GET['task'];

function displayLinks($links)
{
    $i = 1;
    foreach((array)$links as $link)    {
        echo "<td><a href=\"index.php?option=com_content&task=view&id={$link['contentid']}&Itemid={$link['catid']}\">{$link['title']}</a></td>";
        if(($i%3) == 0) {
            echo "</tr><tr>";
        }
        $i++;
    }
}

if ($option=="com_content" && $task=="view") {

    echo "<table><tr>";
    echo "<td colspan=\"3\"><b>Kategorier</b></td></tr><tr>";
    $query = mysql_query("SELECT cat.id AS catid, cat.title, con.id AS contentid FROM jos_categories cat LEFT JOIN jos_content con ON (cat.title = con.title) GROUP BY cat.title ORDER BY cat.title");

    $links = array();
    while ($row = mysql_fetch_array($query)) {
        $links[] = $row;
    }
    displayLinks($links);
    echo "</tr>";

    echo "<tr>";
    echo "<td colspan=\"3\"><b>Indholdssider</b></td></tr><tr>";

    $catid = intval( mosGetParam( $_REQUEST, 'Itemid', null ) );
    $query = mysql_query("SELECT id AS contentid, catid, title FROM jos_content WHERE catid = $catid ORDER BY title ASC");

    $links = array();
    while ($row = mysql_fetch_array($query)) {
        $links[] = $row;
    }
    displayLinks($links);
    echo "</tr></table>";
}
?>

Det andet eksempel:

<?php
$option = $_GET['option'];
$task = $_GET['task'];

if ($option=="com_content" && $task=="view") {
    $catid = intval( mosGetParam( $_REQUEST, 'Itemid', null ) );
    $queries = array(
        'Kategorier' => 'SELECT cat.id AS catid, cat.title, con.id AS contentid FROM jos_categories cat LEFT JOIN jos_content con ON (cat.title = con.title) GROUP BY cat.title ORDER BY cat.title',
        'Indholdssider' => "SELECT id AS contentid, catid, title FROM jos_content WHERE catid = $catid ORDER BY title ASC"
    );

    echo "<table>";

    foreach($queries as $headline => $stmt) {
        echo "<tr><td colspan=\"3\"><b>$headline</b></td></tr><tr>";
        $query = mysql_query($stmt);

        $i = 1;
        while ($row = mysql_fetch_array($query)) {
            echo "<td><a href=\"index.php?option=com_content&task=view&id={$row['contentid']}&Itemid={$row['catid']}\">{$row['title']}</a></td>";
            if(($i%3) == 0) {
                echo "</tr><tr>";
            }
            $i++;
        }
        echo "</tr>";
    }
    echo "</table>";
}
?>

Tak tak... Det kalder jeg service.

Jeg opdagede lige en fejl som jeg måtte rette inden sengen kaldte... Hvis der ikke var en content.title som passede med kategorien kom der når man prøvede at hente siderne i den kategori. Nu er kategorierne ikke længere aktive links hvis der ikke er en matchende side i kategorien...

<?php
$option = $_GET['option'];
$task = $_GET['task'];

if ($option=="com_content" && $task=="view") {

    echo "<table><tr>";
    echo "<td colspan=\"3\"><b>Kategorier</b></td></tr><tr>";
    $query = mysql_query("SELECT jos_categories.id AS catid, jos_categories.title AS cattitle, jos_content.catid, jos_content.id AS contentid, jos_content.title AS contenttitle FROM jos_categories LEFT JOIN jos_content ON (jos_categories.title=jos_content.title) ORDER BY jos_categories.title");
    $i = 1;
    while ($row = mysql_fetch_array($query)) {
        if ($row[catid]=="") {
        echo "<td>$row[cattitle]</td>";
            if(($i%3) == 0) {
            echo "</tr><tr>";
            }       
        }
        else {
        echo "<td><a href=\"index.php?option=com_content&task=view&id=$row[contentid]&Itemid=$row[catid]\">$row[cattitle]</a></td>";
      if(($i%3) == 0) {
        echo "</tr><tr>";
      }}
      $i++;
    }

    $id = intval( mosGetParam( $_REQUEST, 'id', null ) );
    $query = "SELECT catid FROM jos_content WHERE id=$id";
    $database->setQuery( $query );
    $catid = $database->loadResult();
 
    echo "<tr><td colspan=\"3\"><b>Indholdssider</b></td></tr><tr>";
    $query = mysql_query("SELECT * FROM jos_content WHERE catid = $catid ORDER BY title ASC");
        $i = 1;
        while ($row = mysql_fetch_array($query)) {
            echo "<td><a href=\"index.php?option=com_content&task=view&id=$row[id]&Itemid=$catid\">$row[title]</a></td>";
            if(($i%3) == 0) {
                echo "</tr><tr>";
            }
          $i++;
        }
        echo "</tr></table>";
    }
?>

Og så kalder sengen.

Ja, ok. Det må du selv lige fikse ind i mine eksempler. Og så tror jeg også jeg vil til at futte i seng :o)

Rednex: Du er da for sej :-)
Dit andet eksempel virker jo perfekt :-)

Så smid et svar begge to... Den første får point i dette spørgsmål, og så opretter jeg lige et mere med point til den sidste - det her er super dejligt. Hold da kæft. JUBIII :D

1000 Tak for hjælpen begge to...

Hehe, selv tak da :)

Og tror nu ikke du behøver at oprette et nyt spørgsmål, man må jo sådan set heller ikke give mere end 200 point pr. svar i henhold til reglerne :)

jamen er "sølle" 200 point nok?

Det var så lidt :o)

Jeg synes bare vi skal splitte de 200 point i porten :)

Jamen jeg siger endnu engang tak for hjælpen. Det er jeg sgu glad for.