bespoyasov / solidbook

Book about the SOLID principles and object-oriented software design.
https://solidbook.vercel.app
MIT License
609 stars 58 forks source link

Некорректное описание примера на странице с паттернами #202

Closed sunriselink closed 1 year ago

sunriselink commented 1 year ago

Страница https://solidbook.vercel.app/srp/patterns

На ней есть пример неверной реализации класса CachedRequest

image

Указано "Этот вариант всегда ссылается на значение из кэша, даже если его нет"

На самом деле данный вариант всегда возвращает результат с сервера и никогда из кэша. При таком описании пример должен был выглядеть примерно так

async request(url: string): Promise<any> {
  const result = await this.cached.request(url);
  return Promise.resolve(this.cache[url]);
}

P.S. Мне кажется у поля cached: RequestClient неудачное название, сбивает с толку. Это скорей просто client: RequestClient

bespoyasov commented 1 year ago

Обновил. Спасибо за внимательность!