Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Examples do not fit the described architecture patterns #15

Open
sompylasar opened this issue Apr 29, 2014 · 2 comments
Open

Examples do not fit the described architecture patterns #15

sompylasar opened this issue Apr 29, 2014 · 2 comments

Comments

@sompylasar
Copy link

I like the first part of your book (and that of each chapter) where you describe and analyze different approaches to the architecture. I agree with most of it and I'm very glad someone on the Internet expressed these thoughts at last.

I also like the part where you describe how to refactor an existing project towards better architecture. This is something new around modern web app books, too.

But the second part where you provide examples and implementations does not fit with the first one.
The code mainly uses approaches you label as bad (globals, no explicit dependencies, no packages).
The server-side is opinionated towards Node.js and npm (surprisingly, not all web apps are built upon Node.js).
The browser-side is biased towards Ember-like patterns where "Ember" is replaced with "Framework", though Ember uses globals all around.

Many of your thoughts on architecture describe better approaches than the modern frameworks provide, so having them as examples of your thoughts is confusing.

Please elaborate more on the architecture part and drop the examples which do not fit into. I would be glad to discuss confusing parts here. Thanks in advance.

@mixu
Copy link
Owner

mixu commented Apr 30, 2014

hi @sompylasar, thanks for filing these issues. I'm actually working on a second edition which will address some of these issues - let me know if you'd be interested in an early copy in exchange for some feedback...

@sompylasar
Copy link
Author

Thanks, I'd be glad to give feedback when the updates are ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants