Комментарии пользователя Дмитрий Беляев

Как уточнять поведение объекта, при использовании паттерна Фабрика вместо Классов?

Дмитрий Беляев @bingo347 Куратор тега JavaScript

VtzzzZZ,

Прочел learn javacsript ru

все сразу встало на свои места)) Проблема Кантора, что он теоретик, он никогда ничего кроме примеров своего учебника не разрабатывал.

В чем жесть ?

// потеряли имя функций, хорошего дебага
const vehicleFactory = function (config) {

    // создаем 2 (3, 5, 10?) объекта в памяти, и пофик, что по факту нам нужен будет только 1
    const MoveBehaivor = {
        move(){
            console.log('move')
        }
    }

    const TankMoveBehaivor = {
        move(){
            // я так понимаю тут подразумевается дублирование кода из MoveBehaivor?
            console.log('move')
            console.log('as tank')	// отличие от метода для автомобилей
        }
    }

    switch (config.type) {
        case 'car' :
        // зачем то копируем объект, хотя создали мы его в этой же функции
        const car = Object.assign({},MoveBehaivor)

        return car
    
        case  'tank' :

        const vehicle = Object.assign({},TankMoveBehaivor)

        return vehicle
    }
}

ну и еще в догонку, а вот эти функции move в наших конструируемых объектах, они как то зависят от замыкания внутри нашей фабрики? Ну например от config? Если нет, как в примере, то категорично нужно выносить за пределы фабрики, дабы не плодить лишних функций в памяти.

Как бы вы подошли к решению ?

// функции вынесли из фабрики
// фабрика скорее всего будет экспортирована из модуля
// а эти функции нет, так что это нормально
function move() {
  console.log('move');
}
function moveTank() {
  // вместо дублирования кода передаем управление другой функции
  move();
  console.log('as tank');
}

// даем функции имя, дабы видеть его в дебаге, а не (anonymous function)
export function vehicleFactory(config) {
  switch(config.type) {
  case 'car':
    // создаем объекты там где они нужны
    const car = {move};
    return car;
  case 'tank':
    const vehicle = {move: moveTank};
    return vehicle;
  }
}

Войдите на сайт