▲ | sunir 3 days ago | |||||||||||||||||||||||||
That’s a fair critique. Though the snark was unnecessary. I took the time to reread the literature and review my actual code. My updated understanding. Model controls and holds the state of the system. It ensures the data is always valid. Controller controls the boundary between the system and the user. View represent the model and capture user intent to change the model. They ask the model for data they need; however I disagree with the practice that views change data directly but instead prefer they send an intention to change to the app which does the work. Auth was not considered in 1979 as far as I can tell. Authentication is part of the “controller” but in middleware usually because it’s part of the user input boundary, and generally better if done in one place early in the event lifecycle. Authorization is part of the model. App logic is decomposed into workflows or use cases in the app layer. Events coming in through the controller are translated into what the system understands and then passes it onto the workflow to execute. Thus these should take change intent from the view and then actually tell/ask the model what needs to change. This allows the app to catch errors from the model, recover if they can, or handle multi step flows. Results and errors are then sent back to the view (eg GUI dialog) or controller (eg api call) that initiated the workflow. This makes it easier to put different views over the same app logic (mobile, web, api, agent) and also test workflows in isolation. Modern views have their own controllers for mouse and keyboard. That’s fine. Don’t care. That’s effectively outside the system in the client experience (eg browser) anyway. Where I have trouble is when I - put a ton of app logic in the controller - put a ton of model updates in the view - have a single controller for the entire system instead of one per system boundary/interface of user events. The (DDD?) style of app logic being encapsulated outside of the controller makes a lot more sense to me now that I see it. | ||||||||||||||||||||||||||
▲ | mpweiher 3 days ago | parent [-] | |||||||||||||||||||||||||
Hmmm...the problems you recount are exactly the problems of putting (way too much) logic into the controller...and not enough logic into the model...which is exactly what MVC recommends you not do, and which is exactly what I criticized about your approach...and which is exactly the critique you seem unwilling to accept. ¯\_(ツ)_/¯ What gives? Put all the logic in the model. If you want authentication, put that in an "authenticated model" that wraps your model. Don't put it in the controller. > Controller controls the boundary between the system and the user. Nope. That is Apple-MVC. Aka Massive-View-Controller. It is not MVC. The model API is the boundary between the model and the rest of the system. https://blog.metaobject.com/2015/04/model-widget-controller-... | ||||||||||||||||||||||||||
|