параметры по умолчанию

[cyklop77]

помогите пожалуйста разобраться

 

так я передаю параметры для начала игры и сообщений об ошибке в консоли нет. а значит в app.options.level записано определённое значение

 

а так я не передаю параметры для начала игры, следовательно в app.options.level должно записаться значение по умолчанию: 1. то есть должна отработать вот эта инструкция:

this.options = {  level: options.level || 1  ...  .....  ....}

но этого не происходит. в результате я получаю в консоли сообщение об ошибке: 

Uncaught TypeError: Cannot read property 'level' of undefined

 

 

почему так происходит? ведь оператор  || должен присвоить или одно или другое

Edited February 11, 2016 by cyklop77

[Sensei]

Вы не передали объект конструктору, а стало быть options === undefined.

Попытка чтения свойства у undefined приводит к ошибке, так как undefined не объект и никаких свойств иметь не может.

 

Чтобы ошибки не было, можно:

- передать в конструктор пустой объект, или сделать как показали выше.

- совершать присваивание с помощью try-catch

- использовать jQuery.extend() или свою функцию

- использовать возможности es6 — Object.assign() или параметры по умолчанию

 

Нормальным путем будет применение одного из последних двух пунктов)

[andrey7287]

  Так происходит потому что идёт обращение к свойствам несуществующего объекта.  Копируй options во внутреннюю переменную вот так.

this.options = options ? options : {}; 

 И соответственно при обращении к this.options твой код будет работать. 

[cyklop77]

  Так происходит потому что идёт обращение к свойствам несуществующего объекта.  Копируй options во внутреннюю переменную вот так.

this.options = options ? options : {}; 

 И соответственно при обращении к this.options твой код будет работать. 

код то будет работать, но переменной level не присвоится по умолчанию значение 20

Edited February 12, 2016 by cyklop77

[andrey7287]

но переменной level не присвоится по умолчанию значение 20

 

 

Ну почему же ?!

var Game = function(options) {  this.options = options ? options : {};  this.options = {    level: this.options.level || 1,  };};