Find den Fehler net..

[Domi]

Hi,

ich steig da net durch.. hab mir jetzt nen paar Funktionen geschrieben für meine Gallery die auch wunderbar klappen, doch eine macht mir Probleme:

PHP-Code:


function db_id_to_filename($idtofind)
{
    $sql_findfilename = "SELECT
                             id,
                             filename
                         FROM
                             ga_pics
                         WHERE
                             id = '" . $idtofind . "'";

    $result_findfilename = mysql_query($sql_findfilename) OR die(mysql_error());

    while ($row_findfilename = mysql_fetch_assoc($result_findfilename)) {
    result $row_findfilename['filename'];
    }
} 


Ich will quasi in einer Zeile wo ne ID steht die Spalte mit dem Dateinamen auslesen. Jede ID kommt nur 1 mal vor. Aber wie mach ich das jetzt? So wie ichs etz geschrieben hab, klappts net .. ich komm einfach net drauf.

[|Coding]

Zitat von Domi

PHP-Code:

//...
    result $row_findfilename['filename'];
//... 


wasn das?
mach es lieber mit ner variablen...

PHP-Code:

//...
    $result = $row_findfilename['filename'];
//... 


[Domi]

und dann:

return $result;

oder? *mal test*

... processing ...

hmm scheint zu gehen..

[|Coding]

kannst es aber auch so machen:

PHP-Code:

function db_id_to_filename($idtofind){ 
    $sql_findfilename = "SELECT 
                             id, 
                             filename 
                         FROM 
                             ga_pics 
                         WHERE 
                             id = '" . $idtofind . "'"; 

    $result_findfilename = mysql_query($sql_findfilename) OR die(mysql_error()); 
    $row_findfilename = mysql_fetch_assoc($result_findfilename);
    return $row_findfilename['filename']; 
} 


brauchst ja keine WHILE-schleife, weil immer nur ein datensatz aus der db kommt.

[|Coding]

ach, ausserdem macht man bei fehlern in der funktion keine ausgaben (wie die()...) das gehört sich nicht (ist kein schöner stil). stattdessen könntest du einen rückgabewert liefern und ausserhalb der funktion prüfen, wie es um die rückgabe deiner funktion steht.

[Domi]

jo,

danke für dein Tipp! So werd ichs machen.

GreeTz Domi

[xabbuh]

Eine "einfachere" Bezeichnung von Variablen (also $sql, $result, $row) ist vielleicht auch sehr sinnvoll. Da diese Variablen ja auch nur in der Funktion gelten, brauchst du dir auch keine Gedanken darüber machen, ob eventuell bestehende Variablen überschrieben werden.

[Domi]

achso!?!? Genau die Gedanken hatte ich, daher hab ich sie vorsichtshalber mal so genannt. Aber gut zu wissen, dass da nix passieren kann.. dann werde ich diese in zukunft wieder normal benennen ;)

[xabbuh]

Zitat von Domi

achso!?!? Genau die Gedanken hatte ich, daher hab ich sie vorsichtshalber mal so genannt. Aber gut zu wissen, dass da nix passieren kann.. dann werde ich diese in zukunft wieder normal benennen ;)

Hier wird das auch noch mal genau beschrieben: http://tut.php-q.net/functions.html#u4