4 ms·

var MyApp = MyApp || {}; This is an anti-pattern, only use it for polyfilling. Use local variable scope!

• mileycyrusXOXO 9y ago
  I agree this is an anti-pattern; you get better namespace encapsulation by wrapping your code in an immediately invoked function.
• vilmosi 9y ago
  I used this pattern when you had multiple files extending the same global state. For example when lazy loading based on the page you're on. It can definitely be overused, which is probably why it's an anti pattern, but it had it's uses before webpack.
  • z3t4 9y ago
    Here's how I do it: "use strict"; if(window.hasOwnProperty("MyApp")) throw new Error("MyApp already exist!"); var MyApp = {}; But I recommend using a module loader.
    • vilmosi 9y ago
      That's not exactly the same code as above but yeah, use module loaders.