pl: Fragen zur Vorführaufgabe / Daten an Local Storage übergeben

Beitrag lesen

@Rolf B

  1. freeze und thaw wären besser getrennte Funktionen geblieben. Das Mapping auf freeze_thaw bringt aus meiner Sicht keinen Nutzen.

Es bringt den Nutzen, daß es genau eine Funktion ist. Und dadurch daß diese eine Funkion nur referenziert ist, ist die ganze Schnittstelle einfach austauschbar:

var freeze = local ? freeze_thaw : ajax;
var thaw   = local ? freeze_thaw : ajax;

Der Rest vom Code bleibt von einem Austausch des DAL unberührt. Im CODE sind es übrigens 2 Funktionen, so besser lesbar.

  1. warum setzt thaw einen Defaultwert in den stash wenn der localstorage leer ist?

Weil ein Defaultwert gesetzt werden muss. Ansonsten wirft window.JSON.parse() eine Exception.

Ich sehe es mal als Testhilfe an…

Natürlich kannst Du auch mit Exceptions arbeiten wenn Dir das lieber ist. Das liegt in Deinem Ermessen. Ich persönlich halte es jedoch für unzweckmäßig es an dieser Stelle auf eine Exception ankommen zu lassen: Wo ein Default genügt! Wobei lfdnr ohnehin vorbelegt werden muss!

  1. warum ist das Management des stash und das Mapping auf's UI in einem Layer vereint? Du vermischst hier Model und View.

UI und Programmlogik sind per Template doch klar voneinander getrennt!

.