Mach nicht für jeden Markt eine Funktion für den Ausgabewert. Mach eine allgemeine Funktion, die den Markt als ID oder als Name übergeben bekommt und ruf die so auf. Du erzeugst sonst haufenweise duplizierten Code. Wenn du Daten als SQL Skript bereit stellst und da Inserts machst: Tu dir den Gefallen und gib im INSERT INTO auch die Spaltennamen mit an. Sonst sind alle Inserts kaputt, falls du irgendwann mal die Reihenfolge der Spalten tauscht oder irgendwo etwas zwischendrin einfügst. In add.php hast du für jeden Resulttext, den du ausgibst eine eigene Variable. Das führt dazu, dass du ettliche IF Blöcke hast. Da du sowieso immer nur den ersten Error anzeigst oder Success oder eben Insert Fehler (da ist übrigens der Text falsch), kannst du dir die vielen Variablen sparen und einfach nur eine für den Text nehmen. $alert_class = "alert-success"; if (strlen($artikel_name) < 2) { $error = true; $result_text = "Trage den Artikel Namen ein."; } else if ($artikel_anzahl < 1) { $error = true; $result_text = "Trage die Artikel Anzahl ein."; } else if (strlen($artikel_preis) < 1) { $error = true; $result_text = "Trage den Preis des Artikels ein."; } else if (strlen($select_markt) < 1) { $error = true; $result_text = "Wähle einen Markt."; } if ($error) { $alert_class = "alert-danger"; } ----- if (isset($result_text)) { echo "<div class='alert $alert_class'><b>$result_text</b>"; } Ich weiß nicht, ob die syntax so korrekt ist (ich kann nicht mehr wirklich php) aber der Gedanke kommt hoffentlich rüber.
Hallo, Ich werde in naher Zukunft die Einkaufsliste online stellen. Mit Features die nicht Open Source sind. Bevor der Release ansteht benötige ich ein Logo im Material Design. Am besten wäre die Farbe Weiß. Name des Logos: "PyroList". Der Designer des Logos was am Ende genommen wird, wird natürlich auf der Website erwähnt. Liebe Grüße, Maves.
Auch wenn dein PHP Code teilweise recht gruselig ist, jeder fängt halt mal an. Hast du schon eine Domain oder soll ich ggf. auf einem meiner Server einen Webspace zur Verfügung stellen?