Я наступил на яму в процессе разработки и подумал, что это очень интересно, поэтому я придумал вопрос.
// Cache是一个缓存类,使用时先在new方法中注册获取数据的方法,然后可通过get方法获取数据,
// 并且只有第一次调用get会真正调用new中注册的方法获取数据,以后都直接从缓存中返回。
function Cache () {
this.store = {}
}
Cache.prototype.new = function (name, fn) {
if (!name || !fn || typeof fn !== 'function') {
return
}
this.store[name] = {name, fn, data: {}}
}
Cache.prototype.get = function (name, key) {
const self = this.store[name]
key = key || 1
if (self.data[key]) {
return Promise.resolve(self.data[key])
}
return self.fn(key).then(data => {
self.data[key] = data
return data
})
}
Cache.prototype.clear = function (name, key) {
this.store[name].data[key] = null
}
Cache.prototype.clearAll = function (name) {
this.store[name].data = {}
}
// 1.下面的代码说明Cache的实现存在一个bug,尝试修复它
const c = new Cache()
c.new('foo', function (key) {
return Promise.resolve([1])
})
c.get('foo').then(
list0 => {
console.log(list0)
list0.push(2)
return c.get('foo')
}).then(
list1 => {
console.log(list1)
list1.push(3)
return c.get('foo')
}).then(
list2 => {
console.log(list2)
})
// 2.对以上代码提出一些改进意见
Вот ответ:
-
В качестве класса кеша данные, считываемые каждый раз, должны быть одинаковыми, что здесь, очевидно, не так. Так что же пошло не так? В js, если возвращаемое значение не является базовым типом (например, Number, String), оно вернет ссылку на объект, и здесь возвращается ссылка на кешированный объект, чтобы вызывающая сторона могла изменить кешированное содержимое. по желанию. Как это решить? После копирования и последующего возврата связь между кешем и возвращаемым значением обрывается, и чтобы полностью отрезать, это должна быть глубокая копия. Рассмотрение сценария использования заключается в получении внутренних данных, что реализовано сериализацией json в этом месте.
-
Ключ параметра метода get имеет значение по умолчанию 1, и эта логика неверна. Если ключ пуст, значение по умолчанию для ключа 1 будет неправильно передано при вызове метода сбора данных, зарегистрированного в new.
Cache.prototype.get = function (name, key) {
const self = this.store[name]
if (self.data[key]) {
return Promise.resolve(JSON.parse(self.data[key]))
}
return self.fn(key).then(data => {
self.data[key] = JSON.stringify(data)
return data
})
}
Когда я впервые написал эту статью, я думал, что это просто вопрос проверки обратной ссылки, и это было не так уж сложно, но после опроса нескольких человек я обнаружил, что это не так просто. Прежде всего, вам нужно иметь базовое представление об использовании цепочки прототипов для построения классов.Тем инженерам, которые просто реализуют бизнес-логику и не занимаются абстракцией и инкапсуляцией, им, вероятно, не нужно понимать. Второй — это синтаксис Promise и ES6, которые являются относительно новыми.
Почему люди такие разные? Для кого-то один и тот же технический момент - фундамент, о котором и говорить не надо, а для кого-то - фантастика...