search

EyupTuncel / counter-project

This is for a test.
0 stars 0 forks source link

親要素にイベントでidを渡す必要はない #11

Open KoichiKiyokawa opened 3 years ago

KoichiKiyokawa commented 3 years ago

https://github.com/EyupTuncel/counter-project/blob/3aa416aa240cececf289b7ed45acb427375c7ad8/src/components/Counter.svelte#L15

イベントで値を渡すのは極力避けるべきです 依存関係が一方向にならないためです

ただ、イベントを使うのはあっているので、書き直してみてください🙆‍♂️

KoichiKiyokawa commented 3 years ago

また、イベント名はremoveで十分です。 小さなコンポーネントなので、「カウンターがremoveされる」ということは十分伝わります

KoichiKiyokawa commented 3 years ago

ヒントとしては、

// App.svelte
<Counter @remove={() => removeCounter(id)} />

です。※このままでは動きません