Что это за шаблон проектирования и насколько он удачен?

Question

[alpha917]

В некоторых приложениях использовал подход, в котором фабрика возвращала объекты определенного класса по id. притом возвращала не сам объект а ссылку на него. Сам объект хранится в массиве в фабрике. Притом при всех запросах к фабрике по некоторому id возвращался всегда именно тот единственный экземпляр класса, которых хранил данные для этого id.

Тем самым я достигал того что во всем приложении всегда актуальная информация в этом экземпляре т.к. имеется только один экземпляр и все ссылаются только на него.

Так же в такой класс я добавлял методы для работы и изменения объекта.

Что это за шаблон проектирования? Вроде как синглтон а вроде и нет.
Как по вашему, он удачен? В моем случае он показал себя хорошо, однако я мог просто не попасть на какие-то проблемные места, что думаете?

На JS это выглядит примерно так:

/// некоторый объект, который и будет предоставлять фабрика
class Obj { 
  constructor(id) {
    this.id = id;
    this.value = 0;
  }
}
/// Фабрика
class Factory { 
  constructor () {
    this.objects = {};
  }
  getById(id) {
    if (id in this.objects)
      return this.objects[id];
    this.objects[id] = this.load(id);
    return this.objects[id];
  }
  load(id) {
    let obj = new Obj(id);
    /* получение данных объекта */
    return obj;
  }
}
let factory = new Factory(); // инициализируем фабрику, она должна быть синглтоном

// использование
let a = factory.getById(123);
console.log(a.value); // 0
let b = factry.getById(123);
console.log(b.value); // 0
b.value = 10;
console.log(a.value); // 10
console.log(b.value); // 10

Comments

[Максим Федоров]

Вы бы лучше код привели, а не просто текстовое описание ...

[evg_96]

this.objects = {};

А в каком месте у вас сохраняются объекты в данную коллекцию?

[alpha917]

Упустил ) Исправил

Answer 1

[Максим Федоров]

Что это за шаблон проектирования? Вроде как синглтон а вроде и нет.

FlyweightFactory

Как по вашему, он удачен? В моем случае он показал себя хорошо, однако я мог просто не попасть на какие-то проблемные места, что думаете?

Сложно судить т.к. удачное использование шаблона зависит только от задачи к которой он применяется. Если Ваши объекты особо не имеют состояний, тогда данный паттерн вполне уместно применить, если же Ваши объекты это сущности - тогда стоит применить паттерн Identity Map

Comments

[alpha917]

Большое спасибо. Не совсем понял разницу между Flyweight Factory и Identity Map.
Первое описывает фабрику, которая возвращает объекты определенного класса/интерфейса/родителя
Второе описывает способ глобального хранения всех или многих сущностей, не привязываясь к какому-то конкретному классу/интерфейсу ?

[Максим Федоров]

alpha917, да но не совсем, разница еще и в ответственности
Flyweight Factory создает объекты или возвращает если они были ранее созданы ранее, Identity Map же хранить "активные" объекты но ничего не создает, созданием и поиском занимается отдельный компонент

[alpha917]

Спасибо, понятно

Answer 2

[Griboks]

Ссылки на то и называются ссылками, что всегда хранят актуальную информацию. С таким же успехом, можно было бы написать window.objects или var objects.
Я бы даже сделал так:

var storage={add:(key,value)=>{if(typeof(storage[key]) ==='undefined')storage[key]=value;}}
storage.add(123,{value:0});

// использование
let a = storage[123];
console.log(a.value); // 0
let b = storage[123];
console.log(b.value); // 0
b.value = 10;
console.log(a.value); // 10
console.log(b.value); // 10

Comments

[alpha917]

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

[Griboks]

alpha917,

В случае если объекта нет во внутреннем хранилище, он его загрузит.

storage.get=_=>{some load, cache or access actions...}

[alpha917]

в таком случае код ничем не отличается от моего. А в контексте Angular это стоит оформить в сервис, что я и во многом сделал даже в этом примере

[Griboks]

alpha917, только вот это не какой-то паттерн, а просто массив ссылок с геттером и сеттером.

[alpha917]

без сетера но с креатором, с логикой по заполнению и контролем, с декорацией и еще кучей всего. так что просто массивом это назвать нельзя. Да и цель разобраться в возможных недостатках подхода