Code optimieren

Einklappen
X
 
  • Filter
  • Zeit
  • Anzeigen
Alles löschen
neue Beiträge

  • Code optimieren

    Hi zusammen,

    Ich habe hier einen PHP Code und wollte ihn mal kurz durchsehen
    lassen, sicherlich kann man ihn optimieren nur fehlt mir dafür noch
    das wissen - aber learning by doing ist die devise ....

    Also meckert mal los

    PHP-Code:

      <?
      mysql_connect("localhost",
      "user","password");
      mysql_select_db("usr_web4_1");

    $sql = "SELECT date, event, description from events WHERE Day LIKE 'Samstag'";
    $result = mysql_query($sql);
    $idate=1;
    $ievent=1;
    $idesc=1;
    while ($row2 = mysql_fetch_array($result, MYSQL_ASSOC)) {
    echo '&date'."_sa_". $idate++ . '=';
    echo "<b>".'Sa '."</b>";
    echo"<b>".($row2['date']."\n")."</b>";
    echo '&event'."_sa_". $ievent++ . '=';
    echo"<b>".($row2['event']."\n")."</b>";
    echo '&desc'."_sa_". $idesc++ . '=';
    echo($row2['description']."\n");
    }
    ?>

      <?
      mysql_connect("localhost",
      "user","password");
      mysql_select_db("usr_web4_1");

    $sql = "SELECT date, event, description from events WHERE Day LIKE 'Freitag'";
    $result = mysql_query($sql);
    $idate=1;
    $ievent=1;
    $idesc=1;
    while ($row2 = mysql_fetch_array($result, MYSQL_ASSOC)) {
    echo '&date'."_fr_". $idate++ . '=';
    echo "<b>".'Fr '."</b>";
    echo"<b>".($row2['date']."\n")."</b>";
    echo '&event'."_fr_". $ievent++ . '=';
    echo"<b>".($row2['event']."\n")."</b>";
    echo '&desc'."_fr_". $idesc++ . '=';
    echo($row2['description']."\n");
    }
    ?>

      <?
      mysql_connect("localhost",
      "user","password");
      mysql_select_db("usr_web4_1");

    $sql = "SELECT date, event, description from events WHERE Day LIKE 'Donnerstag'";
    $result = mysql_query($sql);
    $idate=1;
    $ievent=1;
    $idesc=1;
    while ($row2 = mysql_fetch_array($result, MYSQL_ASSOC)) {
    echo '&date'."_th_". $idate++ . '=';
    echo "<b>".'Do '."</b>";
    echo"<b>".($row2['date']."\n")."</b>";
    echo '&event'."_th_". $ievent++ . '=';
    echo"<b>".($row2['event']."\n")."</b>";
    echo '&desc'."_th_". $idesc++ . '=';
    echo($row2['description']."\n");
    }
    ?>
    Grüße,
    Matze

  • #2
    die frage die sich mir stellt ist warum dass du insgesamt

    3x einen datenbank-connect machst.

    PHP-Code:
      mysql_connect("localhost",
      
    "user","password");
      
    mysql_select_db("usr_web4_1"); 
    ich glaub 1x ganz oben würde dir reichen.....
    shit happens

    Kommentar


    • #3
      und die variablen musst du auch nur einmal definieren
      PHP-Code:
      $idate=1;
      $ievent=1;
      $idesc=1
      es wäre u.u. auch möglich nur mit einer abfrage zu arbeiten:
      PHP-Code:
      $sql "SELECT date, event, description, Day from events ORDER BY Day"
      und dann schreibst du das ergebnis in ein assoziatives array, was anschließend ausgewertet wird:

      PHP-Code:
      while ($row2 mysql_fetch_array($resultMYSQL_ASSOC)) 
      {
         
      $content[$row2['Day']]['date'][] = $row2['date'];
         
      $content[$row2['Day']]['event'][] = $row2['event'];

      ist nur eine idee, aber du sparst dir u.u. zwei abfragen

      gruß
      peter
      Zuletzt geändert von Kropff; 25.10.2006, 12:06.
      Nukular, das Wort ist N-u-k-u-l-a-r (Homer Simpson)
      Meine Seite

      Kommentar


      • #4
        Merci für die Antworten ...

        Nun sieht er derzeit so aus ... der Code

        PHP-Code:
        <?
          mysql_connect("localhost",
          "web4","2legalize");
          mysql_select_db("usr_web4_1");

        $sql = "SELECT date, event, description from events WHERE Day LIKE 'Samstag'";
        $result = mysql_query($sql);
        $idate=1;
        $ievent=1;
        $idesc=1;
        while ($row2 = mysql_fetch_array($result, MYSQL_ASSOC)) {
        echo '&date'."_sa_". $idate++ . '=';
        echo "<b>".'Sa '."</b>";
        echo"<b>".($row2['date']."\n")."</b>";
        echo '&event'."_sa_". $ievent++ . '=';
        echo"<b>".($row2['event']."\n")."</b>";
        echo '&desc'."_sa_". $idesc++ . '=';
        echo($row2['description']."\n");
        }
        $sql = "SELECT date, event, description from events WHERE Day LIKE 'Freitag'";
        $result = mysql_query($sql);
        while ($row2 = mysql_fetch_array($result, MYSQL_ASSOC)) {
        echo '&date'."_fr_". $idate++ . '=';
        echo "<b>".'Fr '."</b>";
        echo"<b>".($row2['date']."\n")."</b>";
        echo '&event'."_fr_". $ievent++ . '=';
        echo"<b>".($row2['event']."\n")."</b>";
        echo '&desc'."_fr_". $idesc++ . '=';
        echo($row2['description']."\n");
        }
        $sql = "SELECT date, event, description from events WHERE Day LIKE 'Donnerstag'";
        $result = mysql_query($sql);
        while ($row2 = mysql_fetch_array($result, MYSQL_ASSOC)) {
        echo '&date'."_th_". $idate++ . '=';
        echo "<b>".'Do '."</b>";
        echo"<b>".($row2['date']."\n")."</b>";
        echo '&event'."_th_". $ievent++ . '=';
        echo"<b>".($row2['event']."\n")."</b>";
        echo '&desc'."_th_". $idesc++ . '=';
        echo($row2['description']."\n");
        }
        ?>
        Weitere Vorschläge oder kann man den so lassen?

        Grüße dun danke,

        Matthias

        Kommentar


        • #5
          Weitere Vorschläge
          siehe meines geänderten thread

          gruß
          peter
          Nukular, das Wort ist N-u-k-u-l-a-r (Homer Simpson)
          Meine Seite

          Kommentar

          Lädt...
          X