В чём ошибка с присвоением переменно функции?

Question

[DrunkMaster]

Так всё работает https://jsfiddle.net/w14e1t8f/
Так не работает https://jsfiddle.net/sr8nos0o/

Код не работающей функции:

myid.onclick = Main(e);
  
  var Main = function(e) {
  
    var Run1 = function(id){ alert('событие1');    }
    var Run2 = function(id){ alert('событие2');    }
    
    if(e.isTrusted === true){
        Run1(1);
    }
    else {
        Run2(2);
    }
  }

Ошибка Uncaught ReferenceError: e is not defined да я понимаю что он пишет что не видит события и не может передать его в функцию, но как это поправить?

Answer 1

[Максим Павлов]

1)Вам нужно присвоить myid.onclick ссылку на функцию Main. При вызове обработчика туда будет передан параметр - объект события.
2)Как было сказано выше, у Вас функция определяется через function expression, поэтому использовать ее можно только после определения.

Вот, что должно получиться:

var Main = function(e) {
  
    var Run1 = function(id){ alert('событие1');    }
    var Run2 = function(id){ alert('событие2');    }
    
    if(e.isTrusted === true){
        Run1(1);
    }
    else {
        Run2(2);
    }
    
  };
  
  myid.onclick = Main;

P.S. Не рекомендовал бы Вам использовать подобный способ подписки на событие, т.к., если мне не изменяет память, при таком подходе Вы можете определить только один обработчик данного события для данного элемента (и затрете обработчики, которые могли быть определены до этого). Предпочтительнее использовать addEventListener.

Answer 2

[Дмитрий Еремин]

потому что вы сначала вызываете функцию через переменную
а только потом ее определяете (var Main = function(e))

Comments

[DrunkMaster]

Поменял местами, так ошибка появляется сразу при загрузке страницы, а по клику на элемент ничего не происходит...

[Kovalsky]

DrunkMaster, обработчик это функция. В вашем коде вы присваиваете onclick не функцию, а результат её вызова.

[DrunkMaster]

Никита Полевой, а как надо?
myid.onclick = function Main(e);