Link til forrige og næste id

Jo næsten.
Her er lige den nemmeste tilretning af koden:

 function projectLinksNextPrev($id) {
       //Connect to db
       $link = mysql_connect("","","");
       mysql_select_db("", $link);
       
       //Create variables for previous and next links
       $prev_id = $id - 1;
       $next_id = $id + 1;
       
       //call db to find out if previous project_id exists
       $result = mysql_query("SELECT * FROM `cases` WHERE `id`=$prev_id", $link);
       $num_rows_prev = mysql_num_rows($result);
       //call db to find out if next project_id exists
       $result = mysql_query("SELECT * FROM `cases` WHERE `id`=$next_id", $link);
       $num_rows_next = mysql_num_rows($result);
       $output = '';
       if($num_rows_prev != 0) {
           $output .= '<li class="prev"><a href="./?id='.$prev_id.'"><img src="../../img/main/32px/arrow-left2.png" width="32" height="32" alt="Forrige">Forrige</a></li>';
       }
       if($num_rows_next != 0) {
           $output .= '<li class="icon-case"><a href="../"><img src="../../img/main/32px/stack.png" width="32" height="32" alt="Oversigt"></a></li>
           <li class="next"><a href="./?id='.$next_id.'">Næste<img src="../../img/main/32px/arrow-right2.png" width="32" height="32" alt="Næste"></a></li>';
       }
       return $output;
   }

Jeg har blot rettet $project_id til $id så det passer med de ting du har rettet inde i funktionen.

Du kender dit nuværende id: NU_ID som ligger mellem ... < FØR_ID < NU_ID < EFTER_ID < .... (...) er det før og efter.

Så for at finde ID'et før (FØR_ID) skal vi bruge det største ID som er mindre end NU_ID. Ligeledes for at finde EFTER_ID skal vi bruge det mindste ID som er større end NU_ID.

Hvis vi nu fjerner alt større eller lig med NU_ID er vi nede på:
... < FØR_ID. Hvis vi så sorterer efter faldende ID i stedet for vendes rækkefølgende:
FØR_ID, NÆSTE_FØRSTE_ID, ...

Så tager vi blot det første ID der er tilbage. Hvis du vil have de forrige 5 ID'er vælges de første 5 osv..

Så kan vi formulere noget psedo-SQL der gør dette:

Udtræk FØR_ID

 SELECT ID
 FROM MyTable
 WHERE ID < NU_ID
 ORDER BY ID DESC
 LIMIT 1

Hvis databasen ikke svarer med 0 rækker er der ikke nogen før og du behøver ikke lavet linket.

Som Søren skriver så er det ikke specielt kompliceret at lave det du ønsker. Nedenstående kode er blot lavet med mysql_query, men husk at bruge MySQLi eller PDO i stedet, da denne funktion er udfaset.

 function projectLinksNextPrev($project_id) {
     //Connect to db
     $link = mysql_connect("localhost", "mysql_user", "mysql_password");
     mysql_select_db("database", $link);
     
     //Create variables for previous and next links
     $prev_id = $project_id - 1;
     $next_id = $project_id + 1;
     
     //call db to find out if previous project_id exists
     $result = mysql_query("SELECT * FROM `projects` WHERE `project_id`=$prev_id", $link);
     $num_rows_prev = mysql_num_rows($result);
     //call db to find out if next project_id exists
     $result = mysql_query("SELECT * FROM `projects` WHERE `project_id`=$next_id", $link);
     $num_rows_next = mysql_num_rows($result);
     $output = '';
     if($num_rows_prev != 0) {
         $output .= '<a href="projectpage.php?project_id='.$prev_id.'">Previous</a>';
     }
     if($num_rows_next != 0) {
         $output .= '<a href="projectpage.php?project_id='.$next_id.'">Next</a>';
     }
     return $output;
 }

Det er blot et pseudo eksempel og der er ikke taget hensyn til nogen forhold omkring generel sikkerhed når du arbejder med databaser og php. Koden er ikke tiltænkt som brug på et reelt website og det skal opdateres kraftigt før det kan bruges.

Men en lille smule skal du jo også selv have lov at lave

Sådan her eller:

 <li class="prev"><a href="<?php
     
     
     
     $con = mysql_connect("","","");
             if (!$con)
             {
                 die('Could not connect: ' . mysql_error());
             }
             mysql_select_db("", $con);
             mysql_query("SET NAMES utf8");
             mysql_query("SET character_set_results='utf8");
             $result = mysql_query("SELECT id FROM cases WHERE id='".$_GET['id']."' < id='".$_GET['id']."' ORDER BY id DESC LIMIT 1");
     
     
     
     
     
     
     ?>"></li>

Den er der næsten, men lad os lige prøve at indsætte et tal i denne linje for at se hvad der sker:

 $result = mysql_query("SELECT id FROM cases WHERE id='".$_GET['id']."' < id='".$_GET['id']."' ORDER BY id DESC LIMIT 1");

Lad os sige at $_GET['id'] er 37, så ville strengen komme til at se sådan her ud:

 $result = mysql_query("SELECT id FROM cases WHERE id='37' < id='37' ORDER BY id DESC LIMIT 1");

Så kan vi se at det udtryk ser ikke helt rigtigt ud: id='37' < id='37', vi vil sådan set bare gerne have: id < '37' (hvor '37' selvfølgelig kan være et andet tal).

Hvordan gør vi så det ??
- Jeg har ikke lige arbejdet med det her før, så aner ikke lige, hvordan det skal laves.

Hvorfor er der ingen der vil bruge Union? :o)

 SELECT [IdField], 'Prev' AS pos FROM [MyTable] WHERE [IdField] < [CurrentId] ORDER BY ID DESC LIMIT 1
 UNION
 SELECT [IdField], 'Next' AS pos FROM [MyTable] WHERE [IdField] > [CurrentId] ORDER BY ID ASC LIMIT 1

Så slipper man også for et SQL kald.

 // Kun for illustration:
 $rows = $db->fetchAll($sql);
 // The loop
 foreach($rows as $row) {
     if($row['pos'] == 'Prev') {
         // ....
     } elseif($row['pos'] == 'Next') {
         // ....
     }
 }

Hvorfor er der ingen der vil bruge Union? :o)

SQL kode 

 SELECT [IdField], 'Prev' AS pos FROM [MyTable] WHERE [IdField] < [CurrentId] ORDER BY ID DESC LIMIT 1
 UNION
 SELECT [IdField], 'Next' AS pos FROM [MyTable] WHERE [IdField] > [CurrentId] ORDER BY ID ASC LIMIT 1

Så slipper man også for et SQL kald.

PHP kode 

 foreach($rows as $row) {
     if($row['pos'] == 'Prev') {
         // ....
     } elseif($row['pos'] == 'Next') {
         // ....
     }
 }

Kan du ikke lige uddybe den der lidt mere, da den også ser ud til at være en mulig løsning.

Jeg vidste der var en smart måde at trække begge på samtidig kunne bare ikke huske dem .

Det Michael gør er sådan set det samme jeg skrev, han vælger bare begge på en gang og sparer et databasekald. Udover at udtrække selve ID'et 'laver' han en ny kolonne til resultatrækkerne (kaldet 'pos') som han giver en streng, enten 'Prev' eller 'Next' så man ved hvilken ID der er hvad. Man kunne selvfølgelig bare sammenligne talværdierne men det her er mere praktisk (og læsbart IMO).

Hvis det nuværende id f.eks. ar 42 kan det er det første delresultat måske:

| IdField | Pos  |
|   41    | Prev |

Det andet måske:

| IdField | Pos  |
|   44    | Next |

Union ligger dem sammen til en tabel:

| IdField | Pos  |
|   41    | Prev |
|   44    | Next |

$row['pos'] er så enten 'Prev' eller 'Next', og $row['IdField'] er enten 41 eller 44, i det her eksempel.

Men for at komme tilbage til fejlet i dit udtryk:

 id='".$_GET['id']."' < id='".$_GET['id']."'

Gav noget lignende: id='X' < id='X' hvor X er det nuværende id.
Men vi vil have: id < 'X'
Så vi går tilbage retter i WHERE-udtrykket:

 id < '".$_GET['id']."'

Når den så er rettet kan du lave den "modsatte" sql-query, og så sætte dem sammen til en query som Michael har vist.

Okay, men hvordan kommer jeg lige videre.
- Jeg har prøvet en masse, fra andre steder på nettet.

Men synes ikke rigtig noget af det virker.
- Så hvis en eller anden kunne lave et færdigt for mig, ville det være perfekt.

Brug evt det her:

 <div class="caseControl">
     <ul>
     <li class="prev"><a href="#">Forrige</a></li>
     <li class="next"><a href="#">Næste</a></li>
     </ul>
 </div>