Hallo Jnnbo,
du sprichst ein wichtiges Thema an. Funktionen so zu schreiben, dass sie a) keine reduntanten (doppelten) Code enthalten und b) für andere, ähnliche Zwecke gebraucht werden können, ist ein wesentliches Qualitätsmerkmal für Programmcode. Mmn ist dies etwas, was ein jeder über die Jahre stetig verbessert.
Vorstellbar wäre auch so etwas...
function admin_Benefits($mysqli, $select, $from, $where=false) {
$select = implode(",", $select);
$where = $where ? "WHERE b_projekt = $where" : "";
$join = "LEFT JOIN projekte ON projekte.p_id = benefits.b_projekt";
$query = "SELECT $select FROM $from $where $join";
$stmt = $mysqli->prepare($query);
$stmt->execute();
#(...)
Aufrufen mit
admin_Benefits(
$mysqli,
array("b_id", "b_projekt", "b_titel", "b_bild", "b_video", "b_beschreibung"),
"benefits",
"bestimmteid_optional"
);
So hast Du zum Beispiel die Funktion auch variabel für den Fall, das zu in einer anderen Tabelle etwas abfragst oder dass Du andere Werte bekommen möchtest. Auch könnte man sich noch überlegen den Join per optionalem Parameter zu steuern. Auch sparst Du so einmal die Zeile ->prepare in Deinem Code.
Zur Diskussion gestellt...
Cheers, Baba
--
Baba kommt von Basketball
Baba kommt von Basketball