Was ist hier falsch???

Einklappen
Dieses Thema ist geschlossen.
X
X
 
  • Filter
  • Zeit
  • Anzeigen
Alles löschen
neue Beiträge

  • Was ist hier falsch???

    Hallo!

    Bei mir funktioniert das Blättern nicht, wenn die Variablen übergeben werden sollen. Das Skript funktioniert, wenn nach WHERE keine
    Anweisung mehr kommt, bzw. keine Variablen mehr vorhanden sind.
    Wie sollen die Variablen übergeben werden?

    PHP-Code:
    $id=$_GET ['id'];
    $tabellenname=$_GET ['tabellenname'];

    $anzeigen=10;//Anzahl der Datensätze, die auf einer Seite angezeigt werden sollen
    if($_GET[vor]){$start=$_GET[limit]+$anzeigen;}
    if(
    $_GET[back]){$start=$_GET[limit]-$anzeigen;}
    if(!
    $start){$start=0;}
    //Auslesen der Einträge
    $tabellenname="kunde";
                            
    $sql="SELECT * FROM kunde WHERE storno='nein' AND agent='$agent
    AND zeit >= '
    $von1' AND zeit <= '$bis1' ORDER BY $sort LIMIT ".$start." , ".$anzeigen;
    $link mysql_connect("localhost","root","");
    mysql_select_db("db"$link);
    if (!
    $link) {echo "Derzeit ist keine Verbindung zur Datenbank möglich!<br> 
    Bitte versuchen Sie es später oder rufen Sie den Webmaster an!\n"
    ;
                        exit;
    }
    $result mysql_query($sql$link);
    for(
    $i=0;$i<mysql_num_rows($result);$i++)
    {
    $ergebnis[$i]=mysql_fetch_array($result);
    }
    //Ermitteln, wie viele Einträge vorliegen
    $sql1="SELECT COUNT(*) FROM $tabellenname WHERE storno='$status'AND agent='$agent' ";
    $result1 mysql_query($sql1$link);
    $maxmysql_result($result1,0,0);

    //Ausgeben der Einträge
    for($i=0;$i<count($ergebnis);$i++)
    {
    echo 
    "<table border=0><tr><td width=100> ".$ergebnis[$i][anrede].
    </td><td width=140> "
    .$ergebnis[$i][nachname]." </td><td 
    width=140> "
    .$ergebnis[$i][vorname]." </td><td width=180> 
    "
    .$ergebnis[$i][plz]." ".$ergebnis[$i][ort]." </td>";
    if (
    $ergebnis[$i][storno] == nein){
    echo 
    '<td width=40><img src="./button_ok_20.gif"></td>';
    }
    elseif (
    $ergebnis[$i][storno] == ja){
    echo 
    '<td width=40><img src=".\button_error_20.gif"></td>';
    }
    else {}
    echo 
    '<td><a href="anfrage_details.php?tabellenname='.$tabellenname.'&id='.$ergebnis[$i]
    [
    id].'&agent='.$agent.'"><img src=".\details.jpg" border=0></a></td>';
    echo 
    '<td><a href="update.php?tabellenname='.$tabellenname.'&id='.$ergebnis[$i]
    [
    id].'&agent='.$agent.'"><img src=".\update.jpg" 
    border=0></a></td></tr></table>'
    ;
    }
    //Ausgeben der Links vor und zurück
    echo "<table border=\"0\" width=\"400\"><tr><td align=\"left\">";
    if(
    $start>=$anzeigen){echo "<a href=\"".$PHP_SELF."?
    storno="
    .$status."&agent=".$agent."&limit=".$start."&back=1
    \">Zurück</a>"
    ;}
    echo 
    "</td><td align=\"right\">";
    if(
    $start+$anzeigen<$max){echo "<a href=\"".$PHP_SELF."?storno=".$status."&agent=".$agent."&limit=".$start."&vor=1
    \">Weiter</a>"
    ;}
    echo 
    "</tr></table>";
    mysql_close($link);
    ?> 
    Kann mir jemand sagen, wie die Variablen übergeben werden sollen?
    Danke!

  • #2
    was genau ist denn unklar?
    debugging deinerseits ist sinnvoller, als der ganze quellcode.

    Kommentar


    • #3
      Original geschrieben von penizillin
      was genau ist denn unklar?
      debugging deinerseits ist sinnvoller, als der ganze quellcode.
      Und ein paar Zeilenumbrüche an den richtigen Stellen, machen den Code auch wesentlich besser lesbarer, für weitere Tipps, Link in meiner Signatur angucken und eine halbe Stunde für's Lesen opfern...
      (btw. horizontales scrollen ist out, also mach das passend für ne Auflösung von 1024x768!)

      Ein netter Guide zum übersichtlichen Schreiben von PHP/MySQL-Code!

      bei Klammersetzung bevorzuge ich jedoch die JavaCoding-Standards
      Wie man Fragen richtig stellt

      Kommentar


      • #4
        Original geschrieben von ghostgambler
        Und ein paar Zeilenumbrüche an den richtigen Stellen, machen den Code auch wesentlich besser lesbarer
        Nach 1000 Zeilen Code möchte ich denn sehen wo da noch irgendwas findet auf die Schnelle.........
        Ich glaube ich würde aus dem Zimmer rausrennen und wild rumschreien......
        [color=blue]MfG Payne_of_Death[/color]

        [color=red]Manual(s):[/color] <-| PHP | MySQL | SELFHTML |->
        [color=red]Merke:[/color]
        [color=blue]Du brauchst das Rad nicht neu erfinden ! [/color]<-ForumSuche rettet Leben-> || <-Schau in den Codeschnippsels->

        Murphy`s Importanst LAWS
        Jede Lösung bringt nur neue Probleme
        Das Fluchen ist die einzige Sprache, die jeder Programmierer beherrscht.
        In jedem kleinen Problem steckt ein großes, das gern raus moechte.

        Kommentar


        • #5
          Das Skript funktioniert, wenn nach WHERE keine
          Anweisung mehr kommt, bzw. keine Variablen mehr vorhanden sind.
          warum postest du dann den ganzen Quellcode, die Abfrage mit der Where bedingung würde ja reichen, eventuell wo noch die vars herkommen.

          und schon mal was von mysql_error() und error_reporting() gehört?
          mfg
          marc75

          <Platz für anderes>

          Kommentar


          • #6
            Original geschrieben von Payne_of_Death
            Nach 1000 Zeilen Code möchte ich denn sehen wo da noch irgendwas findet auf die Schnelle.........
            Ich glaube ich würde aus dem Zimmer rausrennen und wild rumschreien......
            Naja, aber immerhin kann man alles relativ leicht erkennen...ich hab lieber 3 Codezeilen mehr, als eine Zeile, wo Schleife, Bedingung und möglichst noch Kommentare dazu stehen....

            Ein netter Guide zum übersichtlichen Schreiben von PHP/MySQL-Code!

            bei Klammersetzung bevorzuge ich jedoch die JavaCoding-Standards
            Wie man Fragen richtig stellt

            Kommentar


            • #7
              Ich habe lieber den ganzen Code angegeben.

              Kann mir jemand jetzt sagen, wo der Fehler liegt?
              Die Variablen werden übergeben, das kann ich in der StatusLeiste sehen.
              Aber die werden durch GET nicht übernommen (denke ich).

              Ich bekommen den Fehler in der Zeile
              PHP-Code:
              for($i=0;$i<mysql_num_rows($result);$i++) 
              angezeigt.
              Aber warum, das Skript läuft sonst.

              Danke!

              Kommentar


              • #8
                Kann mir jemand jetzt sagen, wo der Fehler liegt?
                Die Variablen werden übergeben, das kann ich in der StatusLeiste sehen.
                Aber die werden durch GET nicht übernommen (denke ich).
                Kannst du endlich die Ratschläge befolgen. Besonders die Umbrüche, damit man sich deinen Text evtl mal durchliest und ihn nicht links liegen lässt

                Kommentar


                • #9
                  Original geschrieben von ghostgambler
                  Naja, aber immerhin kann man alles relativ leicht erkennen...ich hab lieber 3 Codezeilen mehr, als eine Zeile, wo Schleife, Bedingung und möglichst noch Kommentare dazu stehen....
                  Das kannst du lesen? Nach der dritten Zeile habe ich keine Lust mehr zu lesen........Ich will nicht mehr........

                  Manche Leute müssen irgendwann mal erkennen "Hey, wenn ich da und da ein paar Abstände machen und dort und dort ein paar Einrückungen vornehme, kann ich viel schneller irgendwas erkennen"

                  Das ganze noch an komplizierten Stelle mit ein paar Kommentaren kennzeichnen um was es geht und es wird übersichtlich........

                  Naja und wer glaubt man liest diesen Aufsatz durch der ist wohl sehr naiv...

                  OffTopic:

                  @Tobiaz: hmm wer redet schon gerne gegen die Wand....

                  [color=blue]MfG Payne_of_Death[/color]

                  [color=red]Manual(s):[/color] <-| PHP | MySQL | SELFHTML |->
                  [color=red]Merke:[/color]
                  [color=blue]Du brauchst das Rad nicht neu erfinden ! [/color]<-ForumSuche rettet Leben-> || <-Schau in den Codeschnippsels->

                  Murphy`s Importanst LAWS
                  Jede Lösung bringt nur neue Probleme
                  Das Fluchen ist die einzige Sprache, die jeder Programmierer beherrscht.
                  In jedem kleinen Problem steckt ein großes, das gern raus moechte.

                  Kommentar


                  • #10
                    @Devil-Inside, BITTE DIES: http://www.php-resource.de/forum/sho...threadid=47906 LESEN UND BEFOLGEN, sonst werden wir den Thread schliessen

                    Kommentar


                    • #11
                      Kann mir jemand jetzt sagen, wo der Fehler liegt?
                      Solange der Scrollbalken unten zu sehen ist... Nö.

                      Kommentar

                      Lädt...
                      X