Почему ошибки в терминале?

Question

[v- smerti]

Мой код работает корректно. (alert("test"); срабатывает)
Но в терминале говорит о ошибке

//создаём регистр в который будем заносить все скрипты которые импортировали
var registrImport = {};


//пишем текущий скрипт в регистр
registrImportUpdate("/js/kernel/importer/importer.js");

//в этом блоке будем грузить все системные пакеты. НА данный момент только package
registrImportUpdate("/js/kernel/package/package.js");

var s = loadJs("/js/kernel/package/package.js");
s.onerror = function() { 
      alert("Ошибка при загрузке системного пакета package.js");
};


//функция для запуска скриптов и отработки кода. 
//пример использования
/*
imports([
      "/js/src/supports/supports_html5.js",
      "/js/src/lib/jquery.js"      
  ], 
  function (){
    alert('test');
  }, 
  function (err){
    alert(err);
  });

где 

[
      "/js/src/supports/supports_html5.js",
      "/js/src/lib/jquery.js"      
]
скрипты которые необходимо загрузить.

function (){
    alert('test');
  }
функция которая будет выполнена если всё загрузим


function (err){
    alert(err);
}
функция которая будет вызвана в случае ошибки
*/
function imports(arr, onSuccess, onError){
	//количество скриптов для загрузки
  var count = arrCount(arr);
  //сколько скриптов передано на загрузку
  var coutLoad = 0;
  //скрипты перебираем в цикле
  arr.forEach(function(item) {
    //добавляем 1 загрузку в щётчик
    coutLoad++;
    //если пакет до этого не импортирован то вернёт undefined
	    if (registrImport[item] == undefined){
         //пакета в регистре ещё нет. Добавим его туда
         registrImportUpdate(item);
         //загрузим скрипт
         var script = loadJs(item);
         //выполним одну из функций 
         // onError если ошибка
         // onSuccess если загружено
         loadFunc(script,item, onError, onSuccess, count, coutLoad);     
      } else {
      	 console.info(item + "	загружен ранее.");
      }
  });
}

//считаем количество скриптов в масиве
function arrCount(arr){
  var count = 0;
  arr.forEach(function(){
    count++;
  });
  return count;
}

//грузит скрипт по url
function loadJs(url){
  var script = document.createElement('script');
  script.src = url;
  script.async = false;
  script.defer = true;
  document.head.appendChild(script); 
  return script;
}


function loadFunc(script,url, onError, onSuccess, count, coutLoad){
  //если ошибка
  script.onerror = function() { 
    onError(url);
  };

  //если всё 
  script.onload = function() { 
  //Что бы срабатывало только когда загружен последний елемент масива
    if(count == coutLoad){
        onSuccess(); 
    }
  };
}

//добавляет запись в регистр
function registrImportUpdate(url){
  registrImport[url] = 0;
}

Вызываю так

imports([
			"/js/src/supports/supports_html5.js"			
	], 
	function (){
		alert('test');
	}, 
	function (err){
		alert(err);
	});

Answer 1

[IceJOKER]

потому что у вас нет функции onSuccess();
там же ярко-красным на светло-красном написано! :D

Comments

[v- smerti]

я же её передал для выполнения. И она стати выполняется ))

[IceJOKER]

Владимир Грабко: точно, что за запутанная система %) нет уж, увольте, не буду разбираться в этом

[v- smerti]

IceJOKER: снабдил каждую строку кода "каментами" )

[IceJOKER]

Владимир Грабко: да я знаю как работает код, мне лень разобраться в самой запутанной системе, усложнили код как могли

[v- smerti]

IceJOKER: а как её упростить? Мне просто необходимо запустить функцию после загрузки всех скриптов из масива

[v- smerti]

IceJOKER: ну и отловить ошибки

Answer 2

[M K]

Разобраться действительно трудно, видимо Вы пришли из процедурного программирования в JS, а для реализации подобных систем обычно используют объекты, потому что они работают быстрее, чем отдельные функции, не надо тащить кучу аргументов из функции в функцию, и ещё много плюсов у этого подхода:

function Loader(){
  // Описываем все нужные методы и свойства
  this.queue=[]
  this.clearqueue=function()[this.queue=[]}
  this.loadScripts=function(array_of_scripts){ /* ... */ }
 // и т.д.
}
//Создаём объект
var l=new Loader()
l.loadScripts(["/script_1.js","/script_2.js"])

Так же ознакомьтесь с шаблоном "Очередь" Queue

Comments

[v- smerti]

я пришел из Golang. Только importer написан через жопу. Все следующие будут написаны с использованием обьектов

[M K]

v- smerti: лучше переписать сразу, иначе это может привести к такому нагромождению в коде, что сами потом не разберётесь

[v- smerti]

Максим Кудрявцев: я в таком стиле написал importer и packages
Теперь я получил более мение похожий на Golang js ))

imports([
	"/js/kernel/json_p.js"
	], 
	function (){
		if (packages("init")){
			//я конечно могу сделать парсер в котором автоматом буду каждую функцию пихать в инит.
			//но это не етично))
			init.test = function (){
				alert("test");
			}

			init.testing = function (){
				alert("testing");
			}

			init.test();
			init.testing();

		} else {
			//пока не реализовано
			//по идее должно вызвать вторую переданую функцию
		    err("Пакет init уже создан");
		}
	}, 
	function (err){
		alert(err);
	});

Answer 3

[v- smerti]

я в одном скрипте забыл обновить вызов. Твою жеш "№;"!№