Leidiges Problem: cannot redeclare...

[xmedia2000]

Hi Profis,

hab mal wieder ein Problem mit ner Funktion.
Wie kann ich das besser lösen, weil er mir den
Fehler, ... Fatal error: Cannot redeclare aktiv_image() ... bringt.
Ich würde gerne auf das includen einer Datei verzichten...

function aktiv_image() {
  if ($output[aktiv] == "1")
  { echo '<img src="pics/aktiv.gif" border="0">'; }
  else 
  { echo '<img src="pics/inaktiv.gif" border="0">'; }
};
		
$aktiv = aktiv_image();
	
printf("<tr>
           <td valign=\"middle\">$aktiv</td>
           ")
;}

[MelloPie]

function aktiv_image() {
if ($output[aktiv] == "1")
{ echo '<img src="pics/aktiv.gif" border="0">'; }
else
{ echo '<img src="pics/inaktiv.gif" border="0">'; }
};

$output[aktiv] ist innerhalb der Funktion nicht bekannt.
ausserdem hat die Funktion keinen Rückgabewert.
Warum also $aktiv = aktiv_image();??
Die Funktion wird einfach mit aktiv_image(); aufgerufen.
Also:
<tr><td valign="middle"><?$aktiv_image();?></td>
etc.

[xmedia2000]

Also gut,

dann so:

function aktiv_image() {
  if ($output[aktiv] == "1")
  { echo '<img src="pics/aktiv.gif" border="0">'; }
  else 
  { echo '<img src="pics/inaktiv.gif" border="0">'; }
};
		
	
printf("<tr>
           <td valign=\"middle\"><? aktiv_image() ?></td>
           ")
;}

und wie mache ich die SQL-Query $output[aktiv] bekannt?

[MelloPie]

zB mit Global...

[xmedia2000]

Also ich bin kein Genie in Sachen PHP. Du musst mir dass schonmal genauer erklären. Bloß das Problem mit dem redeclare wäre dann immernoch vorhanden oder??

[MelloPie]

function aktiv_image($output[aktiv] ) ist die bessere Sache
Aufruf:

aktiv_image($output[aktiv])

was willst Du eigentlich machen? Deine Funktion macht eigentlich nichts großartiges das kann man bestimmt auch eleganter lösen...

[KinNeko]

Cannot redeclare function heisst doch nur dass du eine Funktion schon deklariert (also eingebunden hast) und sie dann nochmal eingebunden werden soll. => Fehler
Wahrscheinlich hast du die Datei mit der Funktion 2 mal includet. Such mal den Code durch.

Wenn es anders nicht geht..was ich aber nicht glaube..hilft dir wahrscheinlich die datei mit require_once("datei.endung"); einzubinden.

[goth]

Wenn ich mir Deinen Quellcode betrachte fällt mir am Ende eine schließende, geschweifte Klammer ( } ) auf ... normalerweiser ein Indikator dafür das du Dich innerhalb einer Schleife oder zumindest einer weiteren Funktion befindest ...

Sollte dieses der Fall sein ... so ist das der Grund für die Fehlermeldung ...

PS.: Die Deklaration einer Funktion wird nicht mit einem Semikolon ( ; ) abgeschlossen ...

function aktiv_image() {
  if ($output[aktiv] == "1")
  { echo '<img src="pics/aktiv.gif" border="0">'; }
  else 
  { echo '<img src="pics/inaktiv.gif" border="0">'; }
} 

reicht also aus ...

[xmedia2000]

Also, was ich erreichen will...

Das ganze ist ein News-Script, womit ich aktuelle Nachrichten auf meine Website, über ein Administrationsoberfläche schreiben kann.
Nun habe ich eine Funktion eingearbeiten, die es mir erlaubt, einzelne Nachrichten auszuschalten, also inaktiv zu machen. Diese werden dann auch auf der HP nicht angezeigt.
Das ganze funzt auch, nur will ich in der Administrationsoberfläche eine Grafik, aktiv.gif und inaktiv.gif hinzufügen, womit ich auf den ersten Blick sehe, welche Nachricht aktiv oder inaktiv ist. Das funzt aber mit meinem Code eben nicht.

Dazu habe ich in SQL eine Spalte eingefügt, mit der Bezeichnung aktiv. Als Werte kommen da nur 0 oder 1 vor. Also 0 = inaktiv und 1 = aktiv.
Wenn in Spalte aktiv der Wert = 0, dann soll er das Bild inaktiv.gif anzeigen, wenn der Wert = 1, dann eben aktiv.gif.

while($output=mysql_fetch_array($result))
	{	
	
	function aktiv_image() {
  	if ($output[aktiv] == "1")
  	{ echo '<img src="pics/aktiv.gif" border="0">'; }
  	else 
  	{ echo '<img src="pics/inaktiv.gif" border="0">'; }
	}
		

printf("<tr>
           <td valign=\"middle\"><?php $aktiv_image();?></td>
           </tr>")
;}

[admin]

wenn du in der Datenbank 1 oder null speicherst den bennen doch einfach dein bild ebenfalls so

Z.B.

aktiv_0.gif oder aktiv_1.gif
echo '<img src="pics/aktiv_'.$output[aktiv].'.gif" border="0">';


Dann brauchst du auch keine Abfrage die das dür dich macht!

[admin]

PS: in deiner While-Schleife darf natürlich die Funktionsdefinition nicht eingebunden werden.

Sonst bekommst du den Fehler : cannot redeclare...

[xmedia2000]

cool danke Berni,

die Lösung mit dem umbennen des Bildes in den Wert, der in der Spalte steht war genau die richtige...
Vielen Dank!!!!

[goth]

Sicher war das auch eine Lösung ... aber die redeclare-Problematik war die der while() Schleife ... !