[spacegaier]

Hallo zusammen,

habe schon wieder ein kleines Problem, oder eben nicht. Habe für das Problemchen eine lösung, die mir aber nicht sehr elegant vorkommt, deshalb frage ich hier zur Sicherheit mal nach:

Habe in meinen aktuellen Projekt viele Funktionen, die mir eine Variable namnes $tmp als Rückgabewert liefern.

Nun kommt da aber z.B. folgendes vor:

PHP-Code:

$res = mysql_query("SELECT * FROM Pages");
         
while($row = mysql_fetch_array($res)) 
{ 
    $tmp .= "Texr<br>";    
}

return$tmp; 


Ich habe eine while-Schleife, in welcher ich and den String von $tmp immer etwas anhänge. Nun ist $tmp vor der SChleife aber noch nicht bekannt und somit gibt es eine Warnung. Eine Lösung wäre:

PHP-Code:

$tmp = ''; 


Die scheint mir aber nicht sehr elegant zu sein.

Gibt es dafür noch etwas schöneres / bessere?

Ist es möglich (und sinnvoll) den " = " - Operator zu überladen (geht das in PHP überhaupt) mit einer Funktion, die automatisch prüft, ob die Variable ein String und leer ist, und wenn das der Fall ist, dann den ersten String in die Variable einsetzt und den Rest anhängt?

Grüße - spacegaier

[paracelsus]

--- Beitrag gelöscht ---
Vergiss es, hab Dich falsch verstanden.
Bye

[Boris]

Die scheint mir aber nicht sehr elegant zu sein.

Das ist aber so absolut ok und hat nichts mit "Elegant" zu tun.

Auch könntest du vor der Schleife schrieben:

if(!isset($tmp)) $tmp='';

[ArcVieh]

Zitat von Boris

Das ist aber so absolut ok und hat nichts mit "Elegant" zu tun.

Auch könntest du vor der Schleife schrieben:

if(!isset($tmp)) $tmp='';

Aber vor der Schleife wird $tmp vermutlich noch nicht verwendet, von daher bleibt egtl. nur noch tmp='';, was in diesem Bezug ziemlich legitim ist.
Ansonsten halt:

PHP-Code:

while() {
   if(!isset($tmp)) {
      $tmp = 'text<br/>';
   } else {
      $tmp .= 'text<br/>';
   }
} 


Aber ob die Quantität qualitativ ist...

[spacegaier]

Okay, dann nehme ich halt:

PHP-Code:

if(!isset($tmp)) $tmp=''; 


Sieht schöner aus .

Trotzdem danke!

[Pablo]

Ich denke, $tmp = '' ist das sinnvollste.

Was ich mache, ist tmp-Variablen mehrfach zu benutzen. Und vor jeder Benutzung sollte diese Variable dann auf jeden Fall geleert werden, ich würde deshalb die unsinnige if-Abfrage abschaffen und das gnaze wie ursprünglich gefragt machen.

Gruß,
Pablo

[inta]

PHP-Code:

$tmp = ''; 


ist legitim und richtig, allerdings sollten Variablen "sprechende" Namen haben, also beschreiben was sich hinter ihnen verbirgt.

[ct2oo4]

Ich finde solche zusammengewürfelten Strings immer grauenvoll. Ich arbeite lieber mit Arrays:

PHP-Code:

$res = mysql_query("SELECT * FROM Pages");

$tmp = array();
$i = 0;         
while($row = mysql_fetch_array($res)) 
{ 
    $tmp[$i] = "Texr<br>";    
    $i++;
}

return$tmp; 


(ungetestet)

MfG

// (Was bedeutetet einen Operator "überladen"?)

[code_junkie]

PHP-Code:

$tmp = ''; 


Die scheint mir aber nicht sehr elegant zu sein.

Genau das ist doch elegant. Unschön wäre es hingegen, eine nicht initialisierte Variable zu verwenden.

[ArcVieh]

Zitat von ct2oo4

Ich finde solche zusammengewürfelten Strings immer grauenvoll. Ich arbeite lieber mit Arrays:

PHP-Code:

$res = mysql_query("SELECT * FROM Pages");

$tmp = array();
$i = 0;         
while($row = mysql_fetch_array($res)) 
{ 
    $tmp[$i] = "Texr<br>";    
    $i++;
}

return$tmp; 


(ungetestet)

MfG

// (Was bedeutetet einen Operator "überladen"?)

Wenn dann schon so, würde ein indiziertes Array völlig reichen.

PHP-Code:

$res = mysql_query("SELECT * FROM Pages");        
while($row = mysql_fetch_array($res)) 
{ 
    $tmp[] = "Texr<br>";    
}

return $tmp; 


Erzeugt kein E_NOTICE...

EDIT: Um es nochmal deutlicher zu sagen: $tmp = ''; ist total ausreichend in diesem Kontext. Alles andere ist nur umständliches überprüfen.

PHP-Code:

$tmp = isset($tmp) ? $tmp.'Text<br/>' : 'Text<br>'; 


So könnte man es auch noch machen, wenn man partout kein $tmp = '' verwenden will.