Frage zu implode() oder ähnlicher Funktion

[Kirschtomate]

Hallo,

ich habe ein Formular, dessen Eingabefelder ich via MySQL Abfrage erstelle. Unter anderem beinhaltet das Formular auch Checkboxen:

<?php 
$abfrage = "select * from tabelle";
$ergebnis = $db->query($abfrage);
while ($checkbox = $ergebnis->fetch_assoc()) {
 echo '<input type="checkbox" name="checkbox[]" value="' . $checkbox['checkbox_id'] . '">' .  $checkbox['checkbox_name'] . ' | '; 
}
$ergebnis->free();
?>

Als Ergbnis erhalt ich folgende Ausgabe:

□ Checkbox 1 | □ Checkbox 2 | □ Checkbox 3 | □ Checkbox 4 |

Nun möchte ich den letzten Trennstrich entfernen. Kann hierzu implode() verwendet werden oder gibt es eine andere Möglichkeit. Der Code sollte möglichst übersichtlich bleiben.

Schöne Grüße, Kirschtomate

 

[Arcsinh]

Du überprüfst in deiner while-Schleife, ob es der letzte Durchgang ist. Wenn das der Fall ist, fügst du diesen Trennstrich nicht hinzu.

 

[kkapsner]

Letzter Durchgang prüfen ist bei einer MySQL-Abfrage nervig - ich würde die erste Schleifenausgabe seperat abfragen und den Trennstrick einfach vor das <input> schreiben:

<?php 
$abfrage = "select * from tabelle";
$ergebnis = $db->query($abfrage);
$first = true;
while ($checkbox = $ergebnis->fetch_assoc()) {
	if (!$first){
		echo "|";
		$first = false;
	}
	echo '<input type="checkbox" name="checkbox[]" value="' . $checkbox['checkbox_id'] . '">' .  $checkbox['checkbox_name']; 
}
$ergebnis->free();
?>

 

[Arcsinh]

Gute Idee, müsste es aber dann nicht so aussehen:

while ($checkbox = $ergebnis->fetch_assoc()) {
	if (!$first){
		echo " | ";
	}
	echo '<input type="checkbox" name="checkbox[]" value="' . $checkbox['checkbox_id'] . '">' .  $checkbox['checkbox_name']; 
        $first = false;
}

 

[Kirschtomate]

Danke für das Beispiel. Mit der Klammersetzung warst du allerdings etwas zu gutmütig, so funktioniert es jetzt

<?php 
 $abfrage = "select * from tabelle";
 $ergebnis = $db->query($abfrage);
 $first = true;

 while ($checkbox = $ergebnis->fetch_assoc()) {
    if ($first  == false)  echo " | "; 
    $first = false;
    echo '<input type="checkbox" name="checkbox[]" value="' . $checkbox['checkbox_id'] . '">' . $checkbox['checkbox_name']; 
 }

 $ergebnis->free();
?>

 

[Albu]

Mit der Klammersetzung warst du allerdings etwas zu gutmütig

Und Deine reduziert die Lesbarkeit.

 

[kkapsner]

Stimmt - $false an der falschen Stelle geändert...