88871
Goto Top

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 face-sad


- 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 face-smile

    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.....

Content-Key: 179952

Url: https://administrator.de/contentid/179952

Printed on: April 16, 2024 at 18:04 o'clock

Member: ralf234
ralf234 Feb 02, 2012 at 14:31:03 (UTC)
Goto Top
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
Member: SlainteMhath
SlainteMhath Feb 02, 2012 at 15:24:08 (UTC)
Goto Top
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
Member: Arano
Arano Feb 02, 2012 at 17:24:26 (UTC)
Goto Top
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
Member: SimonP
SimonP Feb 12, 2012 at 00:40:26 (UTC)
Goto Top
"/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