[Jan]

Hallo, folgendes Skript klappt, bis auf das immer nur ein Datensatz gelesen wird, statt fünf.

Wo liegt der Fehler? Ist der Code zu verbessern? In Bezug auf die beiden Abfragen.

PHP-Code:

<?php
/**
Bringt alle aus den Klassenstufen 11,12 und 13 in die Gruppe 'Oberstufe' (15)
 */

define("IN_MYBB", 1);
require_once "./global.php";

/**    Letzten User auswählen    */
if($_GET == '') { $lastid = 0; } else { $lastid = $_GET['lastid']; }

/**    Die Userfields auslesen    */
$query = $db->query("SELECT * FROM ".TABLE_PREFIX."userfields WHERE ufid > '".$lastid."' ORDER BY ufid LIMIT 0,5");
while($userfields = $db->fetch_array($query)) /**    Für jeden User ausführen    */
{

    $lastid = $userfields['ufid'];

    /**    Abfrage der User Infos    */
    $query = $db->query("SELECT * FROM ".TABLE_PREFIX."users WHERE uid = '".$userfields['ufid']."'LIMIT 0,1");
    while($users = $db->fetch_array($query)) /**    Für jeden User ausführen    */
    {

        $name = $users['username'];

        /**    Prüfen ob der User in der Klassenstufe 11,12 oder 13 ist    */
        if(($userfields['fid5'] == '11') || ($userfields['fid5'] == '12') || ($userfields['fid5'] == '13'))
        {
            /**    Prüfen ob der User alle Angaben gemacht hat    */
            if(($userfields['fid7'] == '') || ($userfields['fid8'] == ''))
            {
                $adresse = 'no';
            }
            if(($userfields['fid7'] == "UNBEKANNT") || ($userfields['fid8'] == "LEER"))
            {
                $adresse = 'no';
            }
            if(($userfields['fid7'] == "BITTE EINGEBEN") || ($userfields['fid8'] == "BITTE EINGEBEN"))
            {
                $adresse = 'no';
            }
            if($userfields['ufid'] == "15")
            {
                $adresse = 'no';
            }
    
            /**    Der User hat alle Angaben gemacht, bringen wir ihn nun in die Gruppe    */
            if($adresse != 'no')
            {
                $gruppe = $users['additionalgroups']; /**    Seine alten Gruppen speichern    */
                $test = preg_split("/,/",$gruppe); /**    Seine Gruppen trennen    */
                foreach($test as $i) /**    Für jeden Gruppe prüfen    */
                {
                    if ($i == '15') /**    Ist der User schon in der Gruppe?    */
                    {
                        $stop = 'stop';
                    }
                }
                /**    der User ist nicht in der Gruppe, geben wir im die neue Gruppe dazu    */
                if ($stop != 'stop')
                {
                    $gruppe .= ',15';
                    $otto = array('additionalgroups' => $gruppe);
                    /**    Daten wieder in die Datenbank schreiben    */
                    $db->update_query(TABLE_PREFIX."users", $otto, 'uid = '.$userfields['ufid']);
                    echo 'Der User '.$users['username'].' wurde der Gruppe \'Oberstufe\' hinzugefügt';
                }
            }
        }
    }
print 'Wir sind jetzt bei User '.$userfields['ufid'].' - '.$name;
}
    print('<hr /><br /><a href="gruppe.php?'.SID.'&amp;lastid='.$lastid.'">Weiter</a>');
?>

[Jann Hendrik]

select * from
Warum soll ich nicht SELECT * schreiben?
http://www.php-faq.de/q/q-sql-select.html

Dann ist es weiterhin sinnvoll sql-statements lesbarer zu schreiben:
Beispiel:

PHP-Code:

<?php
  $sql = "SELECT
              `feldname`, `nocheinfeldname`, `undnochfeldname`
          FROM
              `tabellenname`
          WHERE
              `feldname` LIKE '%suchwort%'
            AND
              `nocheinfeldname` = 'wert'
          ORDER BY
              `undnochfeldname` ASC
          LIMIT
              0 , 30";
  $result = mysql_query($sql) or exit('<h2>Fehler</h2>sql-querry: '$sql.'<br />error: '.mysql_error());
  // ...
?>

Darüber hinaus hast du einen Fehler im zweiten sql-string. Du hast hinter dem letzten ' und dem Limit kein Leerzeichen, was dir beim debugging aber sicherlich auch aufgefallen wäre

[Chr!s]

Passiert das denn immer mit deinen SQL-Abfragen?

PHP-Code:

<?php
while($users = $db->fetch_array($query))
?>

Vielleicht liefert die $db->fetch_array() funktion dann einfach nur zu früh den false wert?

[if('Miikku' == $Name)]

Ich würd' dir helfen, wenn ich verstehen würd', welche Datensetze von wo nicht gelesen werden! Da werden doch viele Datensätze eingelesen, oder bin ich jetzt doof?
Nur kurz, was den Programmierstyle angeht: man kann ein Kommentar am ende einer Zeile auch mit

PHP-Code:

//einleiten und außerdem genügt auch 
/*dieses,*/ //anstatt
/**diesem!*/ 


...

mfg
$Name = 'Miikku'; echo $Name;

[Jan]

@ Jann Hendrik
Okay, hab das Leerzeichen eingeführt.
Werde das Statement noch umschreiben. In Bezug auf "SELECT *" und auf lesbarkeit.

@ Chr!s
Warum sollte sie FALSE liefern?
Was soll immer passieren?

@ if('Miikku' == $Name)
Ich weiß
Aber durch diese Kommentare bekomm ich ein schönes Syntaxhighlighting (?)
Die hälfte der Kommentare ist auch nur für mich gewesen, damit ich den Sinn schnell erkennen kann.

Okay, Fehler gefunden.
Ich musste einfach einmal $query umbenennen. Er hat die Variable leider nicht überschrieben.
Jetzt geht es wie gewünscht.

[dsxs]

Sollte $db->fetch_array nicht ein Array zurückliefern - schon nur daher, weil Du die Methode entsprechend benennst.

ED:

Sehe gerade, dass $_GET['lastid'] uU direkt in die querie eingebaut wird - sehr gefährlich! Mach vorher einen preg-check, zB ob der übermittelte Wert numerisch ist. Dasselbe bei allen anderen Userinputs, welche über GET, POST oder COOKIE kommen - dass globals off sind versteht sich von selbst.

[Jann Hendrik]

Zitat von Jan

Aber durch diese Kommentare bekomm ich ein schönes Syntaxhighlighting (?)

auf diese 'Frage' ein klares: nein.
Warum auch?

[mepeisen]

Zitat von Jan

Okay, Fehler gefunden.
Ich musste einfach einmal $query umbenennen. Er hat die Variable leider nicht überschrieben.
Jetzt geht es wie gewünscht.

Richtig, aber denk über die Begründung noch einmal nach, damit du das richtig verstehst. Die ist ein wenig merkwürdig, wie du den Fehler begründest...

Ansonsten noch ein Ansatz zum Tuning: Bei solchen Konstrukten (Selektiere A und für jedes Ergebnis aus A führe Select B aus) empfiehlt sich ein Join. Ein Join ist im Grunde nichts anderes als die Mengenleere in Mathematik (glaube 3. Schuljahr oder so). Schau dir dazu das Tutorial deines Vertrauens an

[ljungi]

Zitat von Jann Hendrik

auf diese 'Frage' ein klares: nein.
Warum auch?

Nun ja, manche Editoren unterscheiden zwischen /* und /**

[Artemis]

Sollte sie ja auch.

Nur, kauf ich mir ein neues Haus, weil meins rot ist und ich es blau haben möchte? Nein, ich streiche es anders.

Soll heißen, wenn es aufs Aussehen ankommt, sollte man einfach mal die Optionen ändern.

Denn: /** leitet einen Dokumentations-Kommentar ein, zum Beispiel vor Klassen oder Funktionen, der mit PHPDoc oä. ausgelesen wird.

http://de.wikibooks.org/wiki/Website...Doc-Kommentare
http://www.ulf-wendel.de/schulung/core/kommentare.php

[ljungi]

Jo, das is mir klar.

Aber wer ohne phpDoc arbeitet (ich nehm mal an, das ist hier der Fall), und wenn der Editor keine vernünftige Config anbietet, dann is das nun mal so

Freilich würd ich das auch umstellen

[Jann Hendrik]

okay... wieder was gelernt...