shota-kagami / TODO-APP

0 stars 0 forks source link

forEachを利用せず、mapで関数的に戻り値を作った方が良い #2

Open nishiurahiroki opened 4 years ago

nishiurahiroki commented 4 years ago

https://github.com/shota-kagami/TODO-APP/blob/8a673376241add2b85477436e63daf531cdea002/src/database/DataManager.js#L27

let todoList = []
snapShot.forEarch(prop => {
 // something.
})

より

const todoList = snapShot.map(prop => {
  // something
})

の方が見通しが良い

shota-kagami commented 4 years ago

masterにpushがうまくいかなくなったため、新しくリポジトリを作り直しました。 "TodoSample"にpushし直しましたので、ご確認ください。

nishiurahiroki commented 4 years ago

👍