Какой метод или свойство надо переопределить у класса?

Question

[Алексей Ярков]

let ls = {}

class MemoryStorage {
  getItem (name) {
    return (name in ls) ? ls[name] : null
  }

  setItem (name, value) {
    ls[name] = value
  }

  removeItem (name) {
    const found = name in ls
    if (found) {
      delete ls[name]
    }
  }

  clear () {
    ls = {}
  }
}

const storage = new MemoryStorage()

console.log(Object.keys(window.localStorage)) // выведет массив ключей window.localStorage
console.log(Object.keys(storage)) // выведет пустой массив

UPD:

class MemoryStorage {

  get length () {
    return Object.keys(this).length
  }

  getItem (name) {
    return (name in this) ? this[name] : null
  }

  setItem (name, value) {
    this[name] = value
  }

  removeItem (name) {
    const found = (name in this)
    if (found) {
      delete this[name]
    }
  }

  clear () {
    for (const k in this) {
      delete this[k]
    }
  }

}

const storage = new MemoryStorage()

storage.setItem('getItem', 'getItem') // Переопределяется getItem [КАК ИЗБЕЖАТЬ???]
const item = storage.getItem('getItem') // TypeError: storage.getItem is not a function
console.log('item:', item)
storage.removeItem('getItem')
storage.setItem('test', 'test')
console.log('Length:', storage.length)
console.log(Object.keys(storage))
storage.clear()
console.log(Object.keys(storage))
console.log('Length:', storage.length)

Comments

[Roman]

Мне кажется что ассоциативный массив ls должен быть атрибутом

Answer 1

[Алексей Ярков]

Не совсем то, чего я хотел, но все же решил оставить так:

/* @flow */

const storage: Object = {}

class MemoryStorage implements MemoryStorageInterface {
  get length (): number {
    return Object.keys(this.storage).length
  }

  get storage (): Object {
    return storage
  }

  getItem (key: string): string | null {
    return (key in this.storage) ? this.storage[key] : null
  }

  setItem (key: string, value: any): void {
    this.storage[key] = value
  }

  removeItem (key: string) {
    const found = (key in this.storage)
    if (found) {
      delete this.storage[key]
    }
  }

  clear () {
    for (const k in this.storage) {
      delete this.storage[k]
    }
  }

}

export default new MemoryStorage()

Comments

[Archakov Dennis]

https://www.google.ru/search?q=%D1%81%D0%B0%D0%BC%...

[Алексей Ярков]

Archakov Dennis, =)

Answer 2

[Archakov Dennis]

Ничего переопределять не надо. Ты возвращаешь экземпляр класса. Тебе надо создать доп. метод, типа getStorage(), который будет возвращать ls при каждом вызове метода.