Закрытие Javascript не загружается?

Question

Я начинаю простой сайт и пытаюсь закодировать свой Javascript, используя закрытие. Моя первая цель - показать окно предупреждения при загрузке страницы. Но это неудачно. Головная часть моей страницы содержит:

<script src="js/dukegen.shell.js"></script> 
<script src="js/dukegen.js"></script> 
<script> 
    window.onload = dukegen.init(); 
</script> 

Это dukegen.shell.js:

dukegen.shell = (function() { 
    vat initModule = function() { 
     alert("STATRTED"); 
    }; 

    return {initModule: initModule}; 
}()); 

Это dukegen.js:

var dukegen = (function() { 
    var init = function() { 
     dukegen.shell.initModule(); 

    }; 
    return {init: init}; 
}()); 

Когда я загрузить страницу, то ошибки выполнения («Неподключить TypeError: невозможно вызвать метод initModule« неопределенного ») на dukegen.shell.initModule(); , Мой неподготовленный глаз не может определить проблему. Оба моих файла javascript находятся в одной папке.

Answer 1

Изменить порядок, в котором вы включите скрипты:

<script src="js/dukegen.js"></script> 
<script src="js/dukegen.shell.js"></script> 

На данный момент js/dukegen.shell.js выполняется, dukegen объект не существует еще, следовательно, назначение dukegen.shell = ... выдает ошибку. Или, если по какой-либо причине объект существует, вы должны перезаписать второй скрипт.

Также убедитесь, что исправлены все ошибки синтаксиса (vat initModule = function() {) и используйте window.onload = dukegen.init.

Answer 2

var initModule = function() { 
     alert("STATRTED"); 
    }; 

нет НДС.

Answer 3

попробовать это:

dukegen = (function (shell) { 
    var init = function() { 
     shell.initModule(); 

    }; 
    return {init: init}; 
}(dukegen.shell)); 

Answer 4

Правильно ли отрезок от главы? Вы включаете dukegen.shell.js, определяя dukegen.shell, затем перезаписываете dukegen в dukegen.js

Answer 5

Вы хотите создать пространство имен, ок? Но вы переписываете его ... Я думаю, что лучший способ:

var dukegen = {}; //your namespace, ok? 

//here we are adding an atribute to your namespace, not overwriting it like you did before 
dukegen.main = (function() { 
     var init = function() { 
      dukegen.shell.initModule(); 

     }; 
     return {init: init}; 
    }()); 

//and creating another module 
dukegen.shell = (function() { 
     var initModule = function() { 
      alert("STATRTED"); 
     }; 

     return {initModule: initModule}; 
    }()); 
//bootstrapping your app 
dukegen.main.init(); 

// and a tip, always use dependency inject to call another module, like this: 
dukegen.main = (function (myShellModuleOrAnyOtherModule) { 
      var init = function() { 
       myShellModuleOrAnyOtherModule.initModule(); 

      }; 
      return {init: init}; 
     }(dukegen.shell));