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

Question

[VtzzzZZ]

Допустим я использую следующий код для создания объекта newCar :

const vehicleFactory = function (config) {

    const MoveBehaivor = {
        move(){
            console.log('move')
        }
    }

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

    return vehicle 
}

let newCar = vehicleFactory()
newCar.move()	// 'move'

Здесь все просто - задаем поведение , создаем объект (если надо то с параметрами из конфига) и возвращаем

Теперь вносим изменение в фабрику так , чтобы начать производить танки .
При этом нам надо чтобы вновь созданный объект танк бладал тем же методом .move() но при этом более специализированным (уточненным).
Можно ввести обект TankMoveBehaivor с полностью переопределенным методом move() и использовать его :

const vehicleFactory = function (config) {

    const MoveBehaivor = {
        move(){
            console.log('move')
        }
    }

    const TankMoveBehaivor = {
        move(){
            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() большой и сложный то при дублировании его получится много повторного кода , чего конечно хотелось бы избежать

При использовании классовой нотации можно было бы создать class Vehicle от которого class Car и class Tank унаследовали бы метод move(),
уточняя его при необходимости.

Как сделать тоже самое используя только вышеописанный паттерн ?

Comments

[Дмитрий Беляев]

Unconsciously pattern usage detected...

А если честно, на каких говнокурсах такой жести учат?

[VtzzzZZ]

Дмитрий Беляев,
Честный ответ - не проходил ни одних курсов . Прочел learn javacsript ru
В чем жесть ? Как бы вы подошли к решению ?

[Дмитрий Беляев]

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;
  }
}

[VtzzzZZ]

Дмитрий Беляев,
Спасибо . Шикарный ответ .
Действительно, круглые примеры Кантора сложновато применять когда пытаешься что-то писать сам (
Единственно , что хотелось бы отметить - я использую объекты (MoveBehaivor ,TankMoveBehaivor ) , для того чтобы как то складировать функции объединенные общим поведением в один контейнер .
То есть в MoveBehaivor не одна фукнция move() , а :

const MoveBehaivor = {
    move() {
        console.log('move')
    },
    turnToTheLeft() {
    },
    turnToTheRight() {
    },
    // и так далее
}

нельзя ли ваше решение использовать и здесь , только обращаясь к MoveBehaivor напрямую внутри TankMoveBehaivor :

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

скажите, если это тоже плохая практика .
P,S. если вынесете ваш комментарий в ответ и это имеет для вас какое-то значение , то я отмечу его решением

[Дмитрий Беляев]

VtzzzZZ, бращаться к MoveBehaivor напрямую внутри TankMoveBehaivor нормально, но сами объекты стоит тогда вынести из фабрики (если им не нужен конфиг конечно)
И тогда уже и клонирование объекта в фабрике станет уместно, т.к. есть один объект - эталон, и много объектов которые фабрика создает
кстати, при таком раскладе и конфиг можно просто в создаваемый объект класть если он нужен в этих методах, а извлекать его через this