Sehr schön, das Du die 'hässliche' Generierung der
SQL-Befehle so versteckst. Das schafft Übersicht.
Wieso verstecken?
Das sind Grundfunktionen und für jede Tabelle ausführbar und gehören daher zur Übersichtlichkeit des Quellcodes in separate Funktionen.
1. Wieso deklarierst du nicht einfach ein Feld als 'PK', dann kannst Du dir das 'WhereOldValues' sparen?
Man sollte etwas über den Tellerrand hinausschauen und dann würde man feststellen, dass ein PK nicht das leistet, was die Generierung dieses
SQL-Statements leistet.
Durch das
SQL-Statement ist gewährleistet, dass der Datensatz nur geändert wird, wenn zwischenzeitlich keine Änderungen des Datensatzes durch z.B einen anderen User durchgeführt wurden.
Der PK "erkennt" dies ja nicht.
2. Deine NULL-Behandlung sieht (auf den ersten Blick beim Frühstück) etwas wackelig aus. Du behandelst '0' als NULL-Äquivalent,
aber den leeren String nicht?
Weil dies nicht notwendig ist.
Programmieren ist die Kunst aus Nullen und Einsen etwas sinnvollen zu gestalten.
Der bessere Künstler ist allerdings der Anwender, denn dieser findet Fehler, welche sich der Programmierer nicht vorstellen konnte.