Thanks! - Githubissues

CongNguyen086 / Todo_List

A Todo List built in React Native

0 stars 0 forks source link

Open coderschoolreview opened 5 years ago

coderschoolreview commented 5 years ago

Bài làm hoàn thành rất tốt đầy đủ yêu cầu, có mở rộng tình năng xoá và lọc data cho 2 tab active & complete.
Chú ý lỗi logic id khi thêm mới todo, không nên dùng id tĩnh theo length của mảng sẽ phát sinh lỗi id: todoList[todoList.length - 1].id + 1, nên dùng id duy nhất cho item.
Nhưng nộp bài trễ nên có lí do nhé bạn.

CongNguyen086 commented 5 years ago

Em rất cảm ơn nhận xét của anh/chị. Về lí do nộp bài trễ em sẽ rút kinh nghiệm lần sau sẽ làm bài đúng hạn, em cũng đã gửi email cho chị Mây rồi ạ.
Về lỗi logic id khi thêm mới todo thì em có test khá kĩ nhưng chưa thấy lỗi. id mới của em sẽ lấy id của phần tử cuối cùng trong danh sách todoList[todoList.length - 1].id và thêm 1 đơn vị. Như vậy thì id danh sách luôn luôn tăng và sẽ không bị trùng lặp. Anh/chị có thể giúp em chỉ ra lỗi logic được không ạ.