Task 3 #3
Task 3 #3
Conversation
utils/EventEmitter.js
Outdated
| } | ||
| this.listeners[eventName] = this.listeners[eventName].filter((currFn)=>currFn !== fn); | ||
| } | ||
| } |
There was a problem hiding this comment.
в конце каждого файлик нужно составлять пустую строку. Даже гит значок добавляет. иногда нужно для совместимости , когда на разных средах разрабатывают. вообше можно в вижаке поставить, чтобы у тебя автоматически проставлялась он всегда
There was a problem hiding this comment.
не знала такого, спасибо большое! поправила
| } | ||
|
|
||
| _pushListener(eventName, fn){ | ||
| if(typeof fn !== "function") { |
| if(!this.listeners[eventName]){ | ||
| throw new Error("There is no such eventName!") | ||
| } | ||
| this.listeners[eventName] = this.listeners[eventName].filter((currFn)=>currFn !== fn); |
There was a problem hiding this comment.
а нужно ли хранить listener, если у тебя не останется никаких обработчиков ?
There was a problem hiding this comment.
упустила это, поправила
спасибо!
utils/EventEmitter.js
Outdated
|
|
||
| once(eventName, fn) { | ||
| const onceWrapper = (...args)=>{ | ||
| this.off(eventName, onceWrapper); |
There was a problem hiding this comment.
почему решила, сначала удалить, а потом выполнить функцию ?
There was a problem hiding this comment.
даже не помню, наверное, подумала, что пока функция выполняется, листенер могут вызвать еще раз
лучше поменять местами?
utils/EventEmitter.js
Outdated
| } | ||
|
|
||
| listenerCount(eventName) { | ||
| return this.listeners[eventName].length || 0; |
There was a problem hiding this comment.
this.listeners[eventName] можно быть undefined ? если да, тогда при вызове length буде ошибка
There was a problem hiding this comment.
я почему-то думала, || 0; в этом случае сработает
заменила
| const end = new Date().getTime(); | ||
| const time = (end - start)/1000; | ||
|
|
||
| this.emit('end', time, result); |
| const key = headers[idx] || `field${idx+1}`; | ||
| resultObj[key] = field; | ||
| }); | ||
| return JSON.stringify(resultObj); |
No description provided.