search

Aaretaler / web_project

0 stars 0 forks source link

Context.tsx:17: Den AppStore so nach aussen zu exportieren ist kein gute Idee. #31

Closed Aaretaler closed 1 month ago

Aaretaler commented 2 months ago

Context.tsx:17: Den AppStore so nach aussen zu exportieren ist kein gute Idee. Je nach Bundler- oder HMR- (Hot Module Replacement) Mechanik sitzt man dann auf der falschen Referenz. Gefährlich IMHO. Besser wäre, den “actionCreator” in einen Hook zu verpacken, dann könnte dieser Hook das dispatch() mittels useContext(AppContext) sauber holen.

Aaretaler commented 1 month ago

Kein Punkteabzug. Bei genügend Zeit noch machen.

Aaretaler commented 1 month ago

Lässt sich mit unserer Architektur nicht einfach refactoren. Sehr verflechtet, bedingt grössere Änderungen an der Applikation.

Readme-Kommentar für Tobi hinzugefügt.