PHP Eigene Funktion schreiben funktioniert nicht

Ich versuche eine ganz einfache Sache in eine eigene Funktion auszulagern. Aber ich sehe den Wald nimmer. Funktioniert einfach nicht, egal was ich ändere...

Ich möchte die Anzeige eines Userbildes in eine eigene Funktion auslagern. Das funktioniert auch ohne Funktion. Da ich aber anfange Sachen in eigene Funktionen auszulagern scheiterte ich schon mal bei meiner allerersten eigenen Funktion

- Ich möchte ein Userbild anzeigen lassen.
- Der Namen des Userbildes steht in der mySQL samt allen anderen Daten wie Username, E-Mail etc.....
- Als Ausgabe der Funktion möchte ich nur die Spalte aus der mySQL auslesen. Resultat am Ende: test.jpg

Hier meine Funktion, die 100% nicht so funktioniert, aber ich habe derart rumgebastelt das ich es einfach mal so hier poste

    function ShowMyImage (  )
    {
        $queryUserImage = mysql_query("SELECT  
                                                    userImage
                                       FROM
                                                    admin_user
                                       WHERE
                                                    kuid
                                       LIKE
                                                    '".mysql_real_escape_string(trim($_SESSION['userID']))."'  
                                       ",$meineDBconnection)  
                                                    OR die("<pre>user Image\n".$queryUserImage."</pre>\n".mysql_error());  
                            
         $userImage = mysql_fetch_row($queryUserImage);

         
         return ( $userImage );
    }

Als Ausgabe der Funktion auf der Seite benutze ich das hier:

<img src="../PFAD/ZUMBILD/  
<?php 
    echo ShowMyImage ( $_SESSION['userID'] );  
?>
"  width="90px" height="120px" title blabla......>  

Brauch ich überhaupt was (in diesem falle meine userID) zu übergeben ? Habe auch Varianten ohne Übergabewert versucht, mit und ohne RETURN etc....

Also ich kapier das einfach nicht ganz......

Danke fuer jede Hilfe.....

Bitte markiere auch die Kommentare, die zur Lösung des Beitrags beigetragen haben

Content-ID: 179952

Url: https://administrator.de/forum/php-eigene-funktion-schreiben-funktioniert-nicht-179952.html

Ausgedruckt am: 23.12.2024 um 19:12 Uhr

4 Kommentare

Neuester Kommentar

Da du in der Funktionsdeklaration keine Übergabewerte verlangst (Klammern leer) brauchst du nichts mitzugeben. Einfach

echo ShowMyImage();

aufrufen und fertig.

Benutz mal

$userImage = mysql_result($queryUserImage, 0);

anstatt

$userImage = mysql_fetch_row($queryUserImage);

mit mysql_fetch_row ziehst du alles in ein Array rein. Mit return in der Funktion kannst du aber m.W. nur einen String zurückgeben
Als Info:
http://www.php.net/manual/de/function.mysql-result.php
http://www.php.net/manual/de/function.mysql-fetch-row.php

Außerdem:
Deine SQL Query solltest du überarbeiten. bei etwas eindeutigem, wie einer User.ID vergleicht man nicht mit LIKE.

WHERE kuid = '".mysql_real_escape_string(trim($_SESSION['userID']))."'

ist hier passender.

(Code wurde nicht getestet)

Grüße

Ralf

Moin,

ich schätze, drein Problem ist das die Variable $meineDBconnection nicht in der Function "ShowMyImage" deklariert ist.

Lösungsmöglichkeit: 1. Du übergibts die Connection im Funktionsaufruf oder 2. du stellt die Connection innerhalb der Funktion her.

/EDIT: 3te und unsauberste Methode wäre $meineDBconnection als global zu definieren.

lg,
Slainte

Moin

Mit return in der Funktion kannst du aber m.W. nur einen String zurückgeben

Nope, man kann alles zurückgeben: String, Interger, Float, Array, Object, ect.
Zu beachten ist allerdings wie die Rückgabe verwendet wird !
Hier steht der Funktionsaufruf in einem "echo" so das der Rückgabewert der Funktion direkt ausgegeben wird. Daher sollte es sich dabei auch um einen ähm... "direkten Wert" handeln - also einen String, ein Integer, float. (eine Zeichenkette)

Wenn, wie in diesem Beispiel, jedoch ein Array zurückgegeben wird, wird lediglich "Array" vom "echo" ausgegeben. Hier solle der Wert erst in einer Variable gespeichert werden um ihn dann weiterverarbeiten zu können:

function retString(){ return 'Eine Zeichenkette'; }  
function retArray(){ return array('aa','bb','cc'); }  

echo retString();
// Ausgabe: "Eine Zeichenkette" 

echo retArray();
// Ausgabe: Array

$variable = retArray();
echo $variable;
// Ausgabe: Array
echo $variable;
// Ausgabe: "aa" 
echo $variable[2];
// Ausgabe: "cc" 

~Arano

"/EDIT: 3te und unsauberste Methode wäre $meineDBconnection als global zu definieren."

Der Zugriff auf die globale Ressource wäre auch via $GLOBALS['meineDBconnection'] möglich. Aber genau wie Slainte würde ich dir empfehlen, die Verbindung als Argument zu übergeben. Beim herstellen der Verbindung innerhalb der Funktion wäre das Problem, dass du sie nicht im restlichen Script benutzen kannst ;)

Gruß, Simon

Frage PHP Entwicklung

Heiß diskutiert