Garnicht. Irgendetwas ist hier durcheinander gegangen
(Oder ich verstehe das auch total falsch wie das hier mit den Konstanten zu Beginn gemeint war)
)
if ($row['id'] == 1)if ($row['kennzeichen'] == 'LASTSCHRIFT')
Und? Unterschiedliche Zahlungsarten haben unterschiedliche Daten und somit wird immer ne Fallunterscheidung nötig. Somit kommt man um die Änderung im Code auch nicht drumrum.
... aber fehleranfälliger und aufwändiger, weswegen niemand so programmiert (außer unsere Lose-Kiddies natürlich
).
interface Zahlungstyp { ... }
class ZahlungstypLastschrift implements Zahlungstyp { ... }
// Nach dem Auslesen aus deiner Bestelltabelle, anstatt dem ganzen if-Konstrukt:
$zahlungstyp = new $row['zahlungstypKlasse'];
$zahlungstyp->onBestellung(/* eventuell allgemeine Parameter, z.B. die Zeile aus der Bestellungstabelle */ $bestellung);function zahlungstypLastschriftBestellung() { ... }
...
$row['zahlungstypFunktion'](/* eventuell allgemeine Parameter, z.B. die Zeile aus der Bestellungstabelle */ $bestellung);Ja, das meine ich hier unten.
Nö. Der Primärschlüssel in der DB is numerisch, so is deine Konstante. Du musst sie nur ergänzen, wenn was dazukommt; aber das hab ich ja schon 2- oder 3mal gepostet, dass du diesen Aufwand so oder so hast.
In einer Datenbank existiert grundsätzlich keine Reihenfolge, auch wenn einen z.B. MySQL das Glauben machen will, weil die Werte immer in derselben Reihenfolge rauskommen. Das is "Zufall" und man kann und darf(!) sich nicht darauf verlassen.
Und was musst du da jetzt für ne ID "finden"? Wenn du 1 für Rechnung und 2 für Bankeinzug hast, dann nimm eben die 3 für PayPal. Bei ner INT-Spalte bleiben dir dann immer noch 4.294.967.292 weitere Zahlungsmöglichkeiten.
Es gibt eben immer Sachen, die für einen neu sind. Wenn man es sicher aber dann hat sagen lassen, wie es geht, weiß man es für die Zukunft.
<Datei Konstanten>
define('Bar',0);
define('überweisung',1);
define('paypal',2);
...<php datei>
switch (zahlungs_id) {
case Bar : .. break;
...
case paypal : ... break;
}