search

EyupTuncel / counter-project

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

AddCounterコンポーネントについて #1

Closed KoichiKiyokawa closed 3 years ago

KoichiKiyokawa commented 3 years ago

コンポーネント名が微妙です ボタンを表すコンポーネントなのでCounterAddButtonとかでしょうか🤔

KoichiKiyokawa commented 3 years ago

また、ボタンの部分をコンポーネントにすることは間違いではないです ただ、なぜコンポーネントにしたのか、考えを教えていただけると嬉しいです🙇‍♂️

EyupTuncel commented 3 years ago

実はそもそもカウンターにユニークなidを与えるためにこの関数を例のコンポーネントの中に入れていました。しかし、アプリのほうでは削除ボタンを押したら複数のカンターが一括に削除されるなどの不具合が生じました。その解決のためにApp.svelteに移動するなどしてコンポーネントをそのまま残しました。 function addNewCounter(e) { e.preventDefault(); dispatch("addcounter", counter); counter = { name: Counter ${id + 1}, value: 0, id: id + 1, }; }

KoichiKiyokawa commented 3 years ago

そのコードだとidの重複が起きるので、複数のカウンターが消えたんだと思います。

このコンポーネントを作る理由の正解としては「scoped cssを使うため」などです。 まあ、とりあえずこのコンポーネントは残す方針で進めて大丈夫です 🙆‍♂️