Datenbankeinträge sortieren

Hauer

Hallo Leute,

ich möchte gerne die Tabellenausgabe meiner DB per Überschrift sortieren lassen.

Ich habe die Überschrift in ein href eingebunden und und lasse beim Refresh den Zusatz ?order=name ranhängen. Leider passiert dabei nix.

Für etwas Hilfe bin ich sehr dankbar.

PHP

<?php 
     require_once '../inc/login.inc.php';

     $verbindung = @mysqli_connect($server, $benutzer, $passwort);

     if ($verbindung){
     mysqli_select_db($verbindung, $datenbank);

     if(mysqli_error($verbindung)){
     echo 'Fehler: ' . mysqli_error($verbindung);


                  
      }else{
      $sql = "SELECT * from kontakte ORDER BYquelle";
      $abfrage = mysqli_query($verbindung, $sql);

          echo "<table border='1', width='932 px'>";
          echo "<tr><td>Quelle</td>";
          echo "<td>Anrede</td> <td>Vorname</td> <td><a href='adr_manager.php?order=name'>Name</a></td> <td>Firma</td> <td>Strasse</td> <td>PLZ</td> <td>Ort</td>";
          echo "<td>Email</td> <td>Telefon</td></tr>";         

              while($kontakte = mysqli_fetch_assoc($abfrage)){

              echo "<td>" . $kontakte['quelle'] . "</td> <td>" . $kontakte['anrede'] . "</td> <td>" . $kontakte['vorname'] . "</td> <td>" . $kontakte['name'] . "</td>"; 
              echo "<td>" . $kontakte['firma'] . "</td> <td>" . $kontakte['strasse'] . "</td> <td>" . $kontakte['plz'] . "</td> <td>" . $kontakte['ort'] . "</td>";
              echo "<td>" . $kontakte['email'] . "</td> <td>" . $kontakte['telefon'] . "</td> </tr>";

               }
             }
          }

   echo "</table>";


   mysqli_close($verbindung);

?>

Alles anzeigen

Bandit

$sql = "SELECT * from kontakte ORDER BYquelle";

Ein Leerzeichen könnte Wunder bewirken.

Dodo

Ich sehe auch nicht, dass du irgendwo $_GET['order'] abfragst.

Versuch mal das:

PHP

$order = "quelle";
if(isset($_GET['order']) && (
   $_GET['order'] == "name" ||
   $_GET['order'] == "firma" ||
   $_GET['order'] == "email" ||
   $_GET['order'] == "anrede" ||
   $_GET['order'] == "strasse" ||
   $_GET['order'] == "telefon" ||
   $_GET['order'] == "vorname" ||
   $_GET['order'] == "nachname" ||
   $_GET['order'] == "plz" ||
   $_GET['order'] == "ort"))
        $order == $_GET['order'];
$sql = "SELECT * from kontakte ORDER BY ".$order." ASC";

Alles anzeigen

Bandit

Warum soll ich nicht SELECT * schreiben?
http://php.faq-page.net/q-sql-select.html

http://de2.php.net/manual/de/mysqli.error.php

Dodo
das könnte man auch eleganter lösen.

Dodo

Zitat von bandit600

Dodo
das könnte man auch eleganter lösen.

Ja ich weiß ;D Spaltenabfrage, Array, etc.
Wollte aber nicht so viel nachdenken - es ist Wochenende^^

Hauer

Hallo und erstmal danke ihr Beiden.

bandit600=

$sql = "SELECT * from kontakte ORDER BYquelle";

Ist mir leider beim editieren nachträglich reingerutscht, danke.

Dodo=

Hab ich soweit übernommen und bekomme folgende Fehlermeldung=

Line 41:

PHP

if(isset($_GET['order']) && (

Parse error: syntax error, unexpected T_BOOLEAN_AND, expecting ',' or ')' in /event/adr_manager.php on line 41

Dodo

Und so wie es oft bei Syntax-Errors ist, liegt der Fehler darüber.
Zeig deinen jetzigen Code her.

Bandit

Ich sehe da keinen Fehler, deshalb tippe ich auf einen Fehler vor dieser Zeile.

Hauer

Ja, hier mal der Code:

PHP

<?php 
     require_once '../inc/login.inc.php';

     $verbindung = @mysqli_connect($server, $benutzer, $passwort);

     if ($verbindung){
     mysqli_select_db($verbindung, $datenbank);

     if(mysqli_error($verbindung)){
     echo 'Fehler: ' . mysqli_error($verbindung);


     }else{

        $order = "quelle";
        if(isset($_GET['order'] && (
           $_GET['order'] == "name" ||
           $_GET['order'] == "firma" ||
           $_GET['order'] == "email" ||
           $_GET['order'] == "anrede" ||
           $_GET['order'] == "strasse" ||
           $_GET['order'] == "telefon" ||
           $_GET['order'] == "vorname" ||
           $_GET['order'] == "nachname" ||
           $_GET['order'] == "plz" ||
           $_GET['order'] == "ort"))
           $order == $_GET['order'];

          $sql = "SELECT * from kontakte ORDER BY ".$order." ASC";  
          $abfrage = mysqli_query($verbindung, $sql);

          echo "<table border='1', width='932 px'>";
          echo "<tr><td>Quelle</td>";
          echo "<td>Anrede</td> <td>Vorname</td> <td><a href='adr_manager.php?order=name'>Name</a></td> <td>Firma</td> <td>Strasse</td> <td>PLZ</td> <td>Ort</td>";
          echo "<td>Email</td> <td>Telefon</td></tr>";         

              while($kontakte = mysqli_fetch_assoc($abfrage)){

              echo "<td>" . $kontakte['quelle'] . "</td> <td>" . $kontakte['anrede'] . "</td> <td>" . $kontakte['vorname'] . "</td> <td>" . $kontakte['name'] . "</td>"; 
              echo "<td>" . $kontakte['firma'] . "</td> <td>" . $kontakte['strasse'] . "</td> <td>" . $kontakte['plz'] . "</td> <td>" . $kontakte['ort'] . "</td>";
              echo "<td>" . $kontakte['email'] . "</td> <td>" . $kontakte['telefon'] . "</td> </tr>";

               }
             }
          }

   echo "</table>";


   mysqli_close($verbindung);

?>

Alles anzeigen

Dodo

Okay... Preisfrage... Was fällt dir auf?

Zitat von Hauer
Line 41:
PHP
if(isset($_GET['order']) && (
Parse error: syntax error, unexpected T_BOOLEAN_AND, expecting ',' or ')' in /event/adr_manager.php on line 41

Zitat von Hauer

PHP

$order = "quelle";
        if(isset($_GET['order'] && (
           $_GET['order'] == "name" ||
           $_GET['order'] == "firma" ||
           $_GET['order'] == "email" ||
           $_GET['order'] == "anrede" ||
           $_GET['order'] == "strasse" ||
           $_GET['order'] == "telefon" ||
           $_GET['order'] == "vorname" ||
           $_GET['order'] == "nachname" ||
           $_GET['order'] == "plz" ||
           $_GET['order'] == "ort"))
           $order == $_GET['order'];

Alles anzeigen

Bandit

ist "if(isset($_GET['order'] && (" nicht Zeile 41
fehlt da eine schließende Klammer
Sollte man Copy/Paste können

Hauer

...so ne sch.....e !!!

Ok, soweit läde die Seite wieder, doch passiert nix beim klick...

Bandit

Zitat von Hauer

doch passiert nix beim klick...

So ein Pech aber auch. Da hilf wohl nur

Richtig debuggen
1. Man bemerkt, dass ein Skript nicht das tut, was es soll.
2. Man schreibt an den Anfang des Scriptes die Zeile: error_reporting(E_ALL); und ini_set("display_errors", true);
3. Man versucht, die Stelle die daran Schuld sein kann, schonmal einzugrenzen. Falls dies nicht geht, wird zunächst das komplette Skript als fehlerhaft angesehen.
4. An markanten Stellen im Skript lässt man sich wichtige Variableninhalte ausgeben und ggf. auch in bedingten Anweisungen eine kurze Ausgabe machen, um zu überprüfen, welche Bedingung ausgeführt wurde (oder auch nicht).
5. Schritt 3 wird so lange wiederholt, bis Unstimmigkeiten im Skript auffallen
6. Damit hat man das Problem (Unstimmigkeit) gefunden und kann versuchen diese zu beheben. Hierzu dienen dann die PHP-Dokumentation und andere Quellen als Ratgeber.
7. Lässt sich das konkrete Problem trotzdem nicht beheben, kann man in Foren um Rat fragen.
8. Das Programm läuft und man kann die Debug-Ausgaben wieder entfernen.

Hauer

Jo, so sieht´s aus.

PHP

$order == $_GET['order'];

liest keine Variable aus der Adresszeile aus.

PHP

$order=$HTTP_GET_VARS['order'];

macht das

Danke nochmal für die Unterstützung.

Bandit

Boah!!!!!!!! Das ist doch jetzt wohl nicht wahr!!!!!!!!!!

$order == $_GET['order'];

Hallo, geht's noch? Einfach mal die Augen aufmachen und mitdenken!!!

$order=$HTTP_GET_VARS['order']; ist veraltet und sollte auf keinen Fall mehr benutzt werden!

Dodo

Argh, tut mir Leid.
Mein Fehler.
War ich wohl nicht mit den Gedanken dabei.

@Bandit: Ich hoffe, dass das so wie Österreich-Flaggen aussieht, ist unbeabsichtigt durch die rote Schrift ;D

Hauer

Mann, Mann bleib mal locker !!!!

1. Keine Ahnung das es so ist, wie du schreibst.

2. Funktioniert es nach der Vorgabe nicht.

3. Habe ich mich nach deiner Aussage unter Punkt 6. weiter schlau gemacht und das gefunden, ausprobiert und es geht !!!

Wenn es nun falsch bzw. veraltet ist, dann mach doch bitte ein besseren Vorschlag und raste hier nicht aus !

Bandit

Zitat von Dodo

Argh, tut mir Leid.

Wieso entschuldigst du dich? Die Leute hier dürfen ruhig mal mitdenken und die Augen aufmachen. Immer alles nur präzise vorkauen lassen, kann ja nicht der Sinn sein!

Und dass es wie die Österreich-Flaggen aussieht, fällt mir auch gerade erst auf.

Zitat von Hauer

raste hier nicht aus !

Wenn ich ausraste, hört sich das ganz anders an. Ich habe nur wenig Verständnis für Leute, die Null Eigeninitiative zeigen.

Dodo

@Hauer: Ich hab ein = zu viel

Code

$oder = $_GET['order']

Zitat von bandit600

Wieso entschuldigst du dich? Die Leute hier dürfen ruhig mal mitdenken und die Augen aufmachen. Immer alles nur präzise vorkauen lassen, kann ja nicht der Sinn sein!

Ja aber das Script ist von mir.
Und ich nehm meine Arbeit so weit Ernst, dass ich sie fehlerfrei haben will, wenn ich sie ausliefere.

Bandit

Zitat von Dodo

Und ich nehm meine Arbeit so weit Ernst, dass ich sie fehlerfrei haben will, wenn ich sie ausliefere.

Du bist hier aber doch nicht auf der Arbeit, du turnst hier in deiner Freizeit rum und hilfst anderen Leuten. Und wie schon gesagt, Eigeninitiative derer ist angesagt.

Datenbankeinträge sortieren

Tags

Benutzer online in diesem Thema