For Schleife optimieren

rs-web · #1 (**permalink**) 29.07.2010, 12:22

Hey,
ich hab ne for schleife mit der ein DB ergebniss ausgegeben wird:

PHP-Code:


			
            for($i = 0; $i < $result->num_rows; $i++){

                if($i == 0){

                    $this->gesuchteBenutzer[$i] =  "<tr>"."<th>"."Benutzername"."</th>"."<th>"."E-mail"."</th>"."<th>"."Aktiviert"."</th>"."<th>"."Gesperrt"."</th>"."</tr>\n";

                }else{

                $array = $result->fetch_assoc();

                $this->gesuchteBenutzer[$i] =  "<tr>"."<td>".$array[benutzername]."</td>"."<td>".$array[email]."</td>"."<td>".$array[aktiviert]."</td>"."<td>".$array[gesperrt]."</td>"."</tr>\n";

                }

            }

Bei dieser Schleife wird aber das erste Ergebniss ausgelassen. Hat jemand von euch eine Idee wie ich das Problem lösen kann?!
LG rs-web

xm22 · #2 (**permalink**) 29.07.2010, 12:40

Ist nicht getestet, aber so ist eigentlich die normal Vorgehensweise.

PHP-Code:


			
$this->gesuchteBenutzer[] =  "<tr>"."<th>"."Benutzername"."</th>"."<th>"."E-mail"."</th>"."<th>"."Aktiviert"."</th>"."<th>"."Gesperrt"."</th>"."</tr>\n";
while ($array = $result->fetch_assoc()) {
    $this->gesuchteBenutzer[] =  "<tr>"."<td>".$array[benutzername]."</td>"."<td>".$array[email]."</td>"."<td>".$array[aktiviert]."</td>"."<td>".$array[gesperrt]."</td>"."</tr>\n";
}

Bei Deiner Schleife war das else unnötig. Das hat ja dafür gesorgt, dass entweder die Titelzeile ODER das Resultat ausgegeben wurde. Das letzte Resultat sollte also eigentlich fehlen..

rs-web · #3 (**permalink**) 29.07.2010, 13:07

Ok, danke.
Ich hab noch ne Frage: Kann man for schleifen auch mit negativen werten machen? Ich hab anstatt Null man -1 probiert aber es gibt keinen Fehler, funktioniert aber auch nicht.
LG rs-web

xm22 · #4 (**permalink**) 29.07.2010, 13:13

Geht:

PHP-Code:


			
for ($i = 0; $i > -9; --$i) {
        echo $i.'<br />';
    }

Was genau geht bei Dir nicht?

Praktikant · #5 (**permalink**) 29.07.2010, 13:37

Ich glaube er meint

PHP-Code:


			
for($i = -1; $i < $result->num_rows; $i++) {
   echo $i;
}

sollte aber eigentlich auch kein Problem sein. Dennoch würde ich die Schleife von xm22 nehmen, der Übersicht wegen.

rs-web · #6 (**permalink**) 29.07.2010, 16:51

Hi,
ja ich meinte Praktikants Bsp.

Ja ok ich werde dann die while Schleife nehmen.
LG rs-web

mantiz · #7 (**permalink**) 29.07.2010, 17:00

Sicher, dass immer das erste Ergebnis ausgelassen wird, sieht für mich gerade eher so aus, als wenn immer das letzte Ergebnis ausgelassen wird.

Variationen sind viele möglich, eine einfache Lösung müsste sein, aus dem "<" ein "<=" zu machen, also:

PHP-Code:


			
            for($i = 0; $i <= $result->num_rows; $i++){

                if($i == 0){

                    $this->gesuchteBenutzer[$i] =  "<tr>"."<th>"."Benutzername"."</th>"."<th>"."E-mail"."</th>"."<th>"."Aktiviert"."</th>"."<th>"."Gesperrt"."</th>"."</tr>\n";

                }else{

                $array = $result->fetch_assoc();

                $this->gesuchteBenutzer[$i] =  "<tr>"."<td>".$array[benutzername]."</td>"."<td>".$array[email]."</td>"."<td>".$array[aktiviert]."</td>"."<td>".$array[gesperrt]."</td>"."</tr>\n";

                }

            }

Oder aber Du füllst das Array ganz einfach mit den Resultaten und wenn es nach der Schleife nicht leer ist, dann shiftest Du die Überschriften vorne dran.

Wie gesagt, gibt zahlreiche Möglichkeiten, hauptsache Du verstehst warum bei Deiner Variante ein Resultat fehlt.

Walter IT-Services · #8 (**permalink**) 12.08.2010, 00:29

Du hast nen array.
Warum nutzt du dann nicht einfach foreach?
Dann brauchst du num_rows auch nicht.

PHP-Code:


			
foreach($this->gesuchteBenutzer as $index => $values){

  if($index == 0){ 

    $this->gesuchteBenutzer[$index] =  "<tr>"."<th>"."Benutzername"."</th>"."<th>"."E-mail"."</th>"."<th>"."Aktiviert"."</th>"."<th>"."Gesperrt"."</th>"."</tr>\n"; 

  }else{

    $array = $result->fetch_assoc(); 

    $this->gesuchteBenutzer[$index] = "<tr>"."<td>".$array['benutzername']."</td>"."<td>".$array['email']."</td>"."<td>".$array['aktiviert']."</td>"."<td>".$array['gesperrt']."</td>"."</tr>\n"; 

  } 

}

LG
Micha

mantiz · #9 (**permalink**) 12.08.2010, 10:06

Ich denke das Array wird vorher leer sein (es wird ja durch die Schleife gefüllt), Dein Ansatz bringt also genau mal nix.

rs-web · #10 (**permalink**) 12.08.2010, 10:46

Hi,
ich habs jetzt so gelöst:

PHP-Code:


			
 $this->gesuchteBenutzer[$i] =  "<tr>"."<th>"."Benutzername"."</th>"."<th>"."E-mail"."</th>"."<th>"."Aktiviert"."</th>"."<th>"."Gesperrt"."</th>"."</tr>\n";



 for($i = 0; $i < $result->num_rows; $i++){

                $array = $result->fetch_assoc();

                $this->gesuchteBenutzer[$i] .=  "<tr>"."<td>".$array[benutzername]."</td>"."<td>".$array[email]."</td>"."<td>".$array[aktiviert]."</td>"."<td>".$array[gesperrt]."</td>"."</tr>\n";

                }

            }

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Ansicht
Linear-Darstellung Zur Hybrid-Darstellung wechseln Zur Baum-Darstellung wechseln

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
jQuery html() optimieren	Eppi	Javascript & Ajax	2	20.11.2008 14:27
Meine Queries optimieren	insanic!	Serveradministration und serverseitige Scripte	30	18.03.2006 15:05
kann man den code optimieren?	Sentinel	CSS	2	23.08.2005 11:45
120 dpi bildschirmanzeige --> website optimieren?	captain	CSS	1	16.06.2005 14:59
Seite optimieren	Petty	(X)HTML	31	15.03.2005 23:22