Conversation
| const [tasks, setTasks] = useState([]) | ||
| const [tasksRemaining, setTasksRemaining] = useState(0) | ||
|
|
||
| useEffect(() => { |
There was a problem hiding this comment.
Sebaiknya effect diberi command sehingga kita bisa tahu ini effect untuk apa
There was a problem hiding this comment.
karena kita nggak bisa kasih nama effect seperti kasih nama variable dan function dengan meaningfull name, makanya di usahakan kasih comment agar yg baca dan kita sendiri sewaktu lihat kode bisa dengan cepat mengerti kegunaan effect ini
EddyPermana22
left a comment
EddyPermana22
left a comment
There was a problem hiding this comment.
saya menemukan masih banyak react component yang tidak mengimport React.
Usahakan selalu import React pada setiap react component.
penggunaan block scope (conditional) yang tidak menggunakan curly brace tolong dihindari, dan gunakan curly brace sebagai conventional block scope yang digunakan oleh programmer javascript.
Ketika mapping, WAJIB ada props key yang valuenya unik.
Ingat react ini melakukan perubahan di virtual DOM, bukan di real DOM. sehingga hal ini sangat penting.
Secara keseluruhan kodenya sudah cukup baik, tinggal di revisi beberapa hal yg saya hightlight tadi.
Good Job !
2 participants
In progress:
https://todolist-react-anggapw.netlify.app/