Как с помощью конструктора создать его же свойство?

Question

[Кирилл Гостьев]

Здравствуйте! Пытаюсь создать игру. Появился вопрос: как правильно создать свойство у объекта игрока, которое будет зависеть от выбора пользователя? Делаю через конструктор:

function СreatePlayer(name, skills, class){
this.name = name,
this.level = 1,
this.skills = skills,
this.inventory = (значение данного свойства должно зависеть от параметра "class")

Как лучше реализовать? Заранее благодарен!

Answer 1

[GavriKos]

А class это что? Если строка или число - switch-case (лучше в отдельном методе, GetInventoryForClass). Если это класс (как единица ООП) - то у самого класса и делайте GetInventory. Второй вариант красивее и правильнее, по идее.

Comments

[Кирилл Гостьев]

Спасибо, разобрался. class - это строка. Сделал методом)

Answer 2

[Андрей Седышев]

switch-case же.
Вопрос лишь в том, где Вы будете его размещать, в зависимости от требований к архитектуре. Что из себя представляют сущности class и inventory?
Если inventory - объект, то логично сделать фабрику.

Comments

[Сергей Сова]

switch/case это bad-practice

Answer 3

[Антон]

Как выше уже писали, createPlayer не должна быть игроком, а должна быть фабрикой - т.е. просто создавать игрока. что-то типа

class BasePlayer {
  
}

class PlayerWithRole extends BasePlayer {
  constructor(properties) {
    super(properties);
  }
}

let playerFactory = function (role, properties) {
  let roles = {
    someRole: PlayerWithRole
  }
  return new roles[role](properties);
}

Comments

[Сергей Сова]

const roles, не?
и лучше бы вынести из функции
И в данном констексте лучше создавать функцию, а не функцию в переменной