Why I Don't Like AMD And What I Will Do About It
If you’re still having challenges understanding the need for Require.js, the community have you covered https://gist.github.com/4686136 :)
About AMD Modules and Require.js
Defining the dependencies of your files is more than a requirement for a large scale project. There simply is no other way you can accomplish maintainability and collaboration between teams.
However i beg to differ from calling everything a module and certainly i am not fond of AMD and its’ derivatives. I am a fan of namespaced applications, it’s a choice. There’s no right or wrong there.
In regards to Require.js specifically, i have stated time and again I am not a big fan. It is a very good effort in trying to solve the dependency problem, it’s wildly popular and afaik it’s the only package that JS developers go to when talking about dependency management.
My main concern with Require.js is that in trying to serve all the needs and requirements out there it has become a very complex and hard to work with package. Especially when trying to test the modules, mocking and stubbing the requirements can be a huge pain.
From there on, everything else has to do with the Asynchronous Module Definition pattern that Require.js and the other packages rely or are based on.
There are 3 reasons for that.
AMD Is Not Focused On Solving Dependencies
It is also a way of defining how your code will be structured by forcing you to wrap it in a huge anonymous function, called the factory:
define(id?, dependencies?, factory);
Some people like this style some people don’t. The main reasoning being that you don’t want to leak variables to the global namespace. I understand that but the AMD pattern is not the only way to avoid global namespace pollution.
AMD Creates Closures For Each File
Since the definition of AMD requires that you wrap your code in an anonymous function this results in creating one extra closure for every file in your codebase.
We all are knowledgeable folks here and know what this means or doesn’t mean.
AMD creates verbosity that’s not required on production
The declaration / requirement statements are there in production code.
Whatever you state that each file requires on the top of your module, will be there in the production code. If AMD was a purely dependency management solution this should not happen. It should plainly make sure that the files are concatenated in the right order based on their dependencies and get out of the way after that.
You are sending over the wire ~1-3% more bytes, from what your JS app size is. That is too much.
What I Plan On Doing About it
Google’s Closure dependency system is the most elegant solution i know by far. It does what it says it does, takes care of your dependencies, nothing more, nothing less and it does it very very well.
It can be ripped of the closure tools and used independently on any kind of project. It’s what i plan on doing pretty soon, as soon as i find a free weekend.
It doesn’t dictate how you write your code. You want namespaced hierarchy? Cool! You want AMD modules? Certainly! You want something in between? Can do… no restrictions
And closure compiler in SIMPLE_OPTIMIZATIONS will take care and remove all the dependency statements from your production code leaving it bare as it should be.
blog comments powered by Disqus