Function Laienfrage

[Onyxagargaryll]

Naja ich brauch die Queries nicht allzu oft, darum finde ich das mit dem neu connecten nicht schlecht.

Aber bitte, wenn ihr alle so gegen meine Klasse wettert, werde ich sie eben auslassen ^^

    $query="SELECT userName FROM userlog WHERE userID=1 LIMIT 1";
     $link            =    mysql_connect($settings['mysql']['host'],
                                        $settings['mysql']['user'],
                                        $settings['mysql']['passwd']);
    $selected_db    =    mysql_select_db($settings['mysql']['db'], $link);
    $result            =    mysql_query($query, $link);
    echo(mysql_error());
    $getname=mysql_fetch_array($result,MYSQL_BOTH);
    echo(mysql_error());
    echo($getname['userName']);
    
    exit(); 

Das geht... oO hmm, bis jetzt hat die Klasse immer funktioniert (und glaubt mir, sie wurde oft und lange gebraucht ^^).

Ich werde mal sehen was ich finden kann.


Danke für eure Geduld, guten Appetit
Onyx

[onemorenerd]

In 4 von 5 Fällen gibt die query-Methode mysql_error() zurück. Du prüfst das in deiner Funktion aber nicht sondern gehst einfach davon aus, dass du ein Resultset bekommst. Wenn das nicht so ist, wird die While-Schleife nie betreten, $found bleibt 0 und deine Funktion gibt false zurück.

Du verstehst unter Debuggen anscheinend etwas anderes. Wir meinen es ungefähr so:

function getname($plid) {
    $found=0;
    //load mysql class & settings
    global $mysql,$settings;
    //query for user
    $sql = "SELECT userName FROM userlog WHERE userID=".$plid;
    var_dump($settings['mysql']['db'], $sql);
    $getname_r=$mysql->query($settings['mysql']['db'], $sql);
    var_dump($getname_r);
    while($getname=mysql_fetch_array($getname_r)) {
        var_dump($getname);  
        $found=$getname['userName'];
    }
    var_dump($found);
    //return false if no user was found
    if ($found==0) { 
        return false; 
    } else {
        return $found; 
    }
} 

[Blackgreetz]

Deine Klasse hat fast den gleichen Charakter wie die von tut.php-q.net ...

Warum ist das neu connecten nicht schlecht? Ressourcen nutzen, wenn man sie hat?

mfg

[TobiaZ]

Naja ich brauch die Queries nicht allzu oft, darum finde ich das mit dem neu connecten nicht schlecht.

Aber du brauchst mit Sicherheit ne menge andere Queries in dem Script, oder?

Das geht... oO hmm

Übrigens, echo ist KEINE Funktion, $link brauchst du nicht angeben, wenn du sowieso nur eine Verbindung zur DB hast, mysql_fetch_array($result,MYSQL_BOTH) kann in deinem Fall durch ein simples mysql_fetch_assoc($result) ersetzt werden:

mysql_connect($settings['mysql']['host'],
              $settings['mysql']['user'],
              $settings['mysql']['passwd']);
mysql_select_db($settings['mysql']['db']);

$query="SELECT userName FROM userlog WHERE userID=1 LIMIT 1";
$result = mysql_query($query)
  or die(mysql_error());

$user=mysql_fetch_assoc($result);
echo $user['userName']; 

Das ist deutlich weniger Code als du zuvor mit deiner Klasse benötigt hast.

[Onyxagargaryll]

Hehe, ok ok ^^ Es scheint so als gäbe es sicherlich noch einiges zu verbessern.

Aber back to topic:
Ich hab zwar (fast) alles gemacht, wies pekka gepostet hat, aber eines habe ich ausgelassen:

               {  
         $found=$getname['userName'];
        }
        var_dump($found); // DAS HIER
     //return false if no user was found
     if ($found==0){ return false; }
     else { return $found; }
    } 

Aber jetzt bin ich verwirrt oO
var_dump für $found hat mir string(10) "guestlogin" ausgegeben.

Also heisst das:
"guestlogin" == 0 !

Ich bin mit dieser Logik nicht ganz einverstanden... Aber das heisst doch schonmal, dass er immer ein Resultat gefunden hat, aber an am letzten IF gescheitert ist... wir hätten uns eine Menge an Arbeit ersparen können, wenn man nur richtig debuggt hätte, nicht wahr, Pekka? ^^

MfG
Onyx


EDIT: In dem Script sinds 4 Queries ^^ (bitte nix mehr zu dem Thema sagen)

[pekka]

Hm.
Wie überprüfst du denn, ob das Resultat der Funktion tatsächlich false ist?

[TobiaZ]

Trott das Problem auch bei meiner (kürzeren und wesentlich weniger redundanten) Lösung auf?

[Onyxagargaryll]

@pekka echo var_dump(getname(1));

@Tobiaz
EDIT: Deine hat nicht funktioniert

Leute ich bin echt verwirrt...
Dass er mit "guestlogin"==0 Mühe hat hätte ich echt nicht gedacht.

MfG

EDIT: Hab mir nochmals 3 min Zeit genommen: Deine funktioniert, Tobiaz..

[TobiaZ]

Ich habe keine Ahnung warum

mysql_error? Muss man mal wieder alles vorkauen?

Leute ich bin echt verwirrt... Dass er mit "guestlogin"==0 Mühe hat hätte ich echt nicht gedacht.

Ich auch nicht. Eigentlich sind Computer diejenigen, die weniger Probleme mit der Logik haben!

EDIT: Hab mir nochmals 3 min Zeit genommen: Deine funktioniert, Tobiaz..

Danke! generell erst min 5 Minuten Testen (bei so Kleinigkeiten) und dann posten. Dass die Antworter schneller posten ist klar, die müssen auch nicht ihren Code anpassen und testen.

[Onyxagargaryll]

Siehe EDIT, hab nochmals debuggt und jetzt funktionierts, aber meine nicht

Soll ich einfach deine nehmen und den Vorfall vergessen? Oder soll ich rausfinden warum meine nicht funktiniert hat hmm

MfG

[TobiaZ]

Wenn du viel zeit hast, debugg weiter.

Aber schöner ist die kurze variante (abgesehen von der while, die sich aber schnell durch ein if ersetzen lässt).

[Onyxagargaryll]

Ich benutz deine Variante...

if "guestlogin" == 0
= true
bleibt leider bestehen.


Ich danke für die Hilfe und die Geduld aller

Herzlichen Gruss
Onyx

[TobiaZ]

Wenn du es lösen möchtest, kannst du ja mal den aktuellen Code posten.

[Onyxagargaryll]

Vielleicht werde ich dieses Angebot später gerne annehmen, aber im Moment lass ich es sein, das hebe ich mir für s'Wochenende auf.

Ich weiss ja jetzt wie man debuggt
Vielleicht ergibt sich so was.

Ansonsten melde ich mich auf jeden Fall wieder

Schönes Wochenende und Gruss
Onyx