Review#1

[duypv98]

https://github.com/041220/FormQuestion/blob/da1688ec04787dfffb669ab117bcbd584f6043f9/src/components/Container/index.js#L1-L51

File này là 1 container bao gồm tiêu đề và mô tả cũng như danh sách câu hỏi của 1 form. Yêu cầu của anh sẽ là 1 danh sách form, mỗi form nhiều câu hỏi nhé.

Còn review lần này coi như yêu cầu chỉ là 1 form nhiều câu hỏi thôi.

[duypv98]

Ở đây push thêm câu hỏi vào mảng questions https://github.com/041220/FormQuestion/blob/da1688ec04787dfffb669ab117bcbd584f6043f9/src/components/Container/containerSlice.js#L13

thì đoạn này: https://github.com/041220/FormQuestion/blob/da1688ec04787dfffb669ab117bcbd584f6043f9/src/components/Container/index.js#L11-L20

Nếu không phải em dùng Redux Toolkit nó tự assign biến mới cho state thì đoạn useEffect này sẽ không chạy khi list question thay đổi. Chú ý đọc lại về object và Object trong javascript.

Thứ 2 là tại sao đoạn add question, sau khi add xong thì lưu luôn vào storage (viết trong slice cũng được).

[duypv98]

https://github.com/041220/FormQuestion/blob/da1688ec04787dfffb669ab117bcbd584f6043f9/src/components/Questions/index.js#L34-L53

if xong else if không có else hoặc return mặc định thì hơi nguy hiểm. Mặc dù e chắc chắn nó không bao giờ vào trường hợp nào khác ngoài if và else if, nhưng sau này có thêm hoặc lỡ truyền vào 1 type khác thì có thể gây lỗi xong lại phải mò lại đoạn code dài

[duypv98]

https://github.com/041220/FormQuestion/blob/da1688ec04787dfffb669ab117bcbd584f6043f9/src/components/Questions/index.js#L26

hàm render này e tách thành 1 component, với props đầu vào là typeQuestion.