Почему данная конструкция не работает?

Question

[Максим]

Сам конструктор у меня работал как надо.
Но не работает после того как я решил добавить метод getValues.
В нем он не определяет weapon.param; Скорее всего получает как undefined.
Потому что результаты сложения this._power = function(){return this.power + getValues(power)}; и тд и тп.
у меня получаются NaN.
Если вместо weapon.param поставить значение то все будет работать.
Чтобы не было вопросов что такое weapon - это свойство в которое передается объект со свойствами.
Сразу скажу оно точно впорядке и все передается, можно вместо weapon.param написать weapon.power (оно равно 0 предположим) все будет работать.
То-есть что то именно не так с записью самого метода и аргумента param.
Подскажите, что я делаю неправильно.

let Char = function (	lvl, name, 
						power, agil, inst, health, 
						crit, crit2, 
						dodge, dodge2, 
						def,  
						dmg1, dmg2,
						weapon ) {
						  
	Obj.apply(this, arguments);

	
	this.weapon = weapon;
	
	getValues = function(param) {
 		let sum = 0;
 		sum = sum + weapon.param;
 		return sum;

 	}

	this._lvl    = function(){return this.lvl};
 	this._hp	 = function(){return this.health*10};
	this._name   = function(){return this.name};
	this._power  = function(){return this.power + getValues(power)};
	this._agil   = function(){return this.agil + getValues(agil)};
	this._inst   = function(){return this.inst + getValues(inst)};
	this._health = function(){return this.health + getValues(health)};
	this._crit   = function(){return this.crit + getValues(crit)};
	this._crit2  = function(){return this.crit2 + getValues(crit2)};//anti-crit
	this._dodge  = function(){return this.dodge + getValues(dodge)};
	this._dodge2 = function(){return this.dodge2 + getValues(dodge2)};//anti-dodge
	this._def    = function(){return this.def + getValues(def)};
	this._dmg1   = function(){return this.dmg1 + Math.round(this._power()*0.2) + getValues(dmg1)};
	this._dmg2   = function(){return this.dmg2 + Math.round(this._power()*0.2) + getValues(dmg2)};

}

Comments

[edward_freedom]

переменная power обьект или число?

[Максим]

edward_freedom, power это свойство объекта Weapon - число.
Объект Weapon это внешний объект который подставляется в свойство weapon при создании объекта класса Char.

[edward_freedom]

max_shane, какой тип обьекта ты передаешь в эту функцию?

getValues = function(param) {

param - обьект или число?
Так как не вижу где ты param применяешь

[Максим]

param это аргумент и переменная метода(как у функций это делается).
Потом этот метод вызывается с различными свойствами которые реально существуют у объекта Weapon (power, inst, agile и тд).
Все они целые числа. Это все работает нормально. Если я буду подставлять напрямую, например weapon.power или weapon.agile - все будет получаться как надо. Но я хочу использовать метод вместо этого. И вот он не работает.

[edward_freedom]

max_shane, я к этому и веду, у обьекта weapon, есть свойство param?

weapon.param

Мне кажется нет, поэтому и ошибка

[Максим]

edward_freedom, Понял вопрос. Нет, такого свойства нету! Я такой записью подразумевал, что вместо param у меня подставится тот ключ с которым вызывается метод.
Это неправильно? А как сделать тогда?

[edward_freedom]

max_shane, попробуй так
weapon[param]

[Максим]

edward_freedom, Попробовал, но на самом деле я уже так пробовал и раньше - тот же результат.
Получаемые значения NaN, а значит weapon.param он видит как undefined.

Answer 1

[Максим]

Я нашел ошибку. Всем спасибо.
Ошибка была в том, что power я пытаюсь передать просто как пустую переменную, без значения. Чтобы просто подставить само слово (hp, power и тд) и вызывать это свойство у объекта weapon.
Но ведь я уже ранее создал переменные в конструкторе power, agile, inst и тд, и им уже присвоены числовые значения. Получается я передаю число в аргументе функции. Потому и не работает.

Comments

[edward_freedom]

типо ты в кавычки не брал?

Answer 2

[Дмитрий]

Потому что
weapon[param] вместо weapon.param

Comments

[Максим]

Да не решение это. Записывать и так и так можно. Результат один и тот же.

[Дмитрий]

max_shane, Записывать и так и так нельзя, вы не знаете основ синтаксиса js, если в param передается имя атрибута, оно должно быть в квадратных скобках.