search

hoangphantich / eula.web16.1

Code repos. Homepage:
https://hoangphantich.github.io/webeula16.1/
1 stars 3 forks source link

[Review] Phạm Duy Nguyễn #12

Open hoangphantich opened 7 years ago

hoangphantich commented 7 years ago

@IceNeoMax review 2 bạn này nhé @trieubkpro Trần Hải Triều|1649777238|https://www.facebook.com/trieuthbkpro|||trantrieu.2010@gmail.com @nguyenvanthuan1993 Nguyễn Văn Thuận|963340529|https://www.facebook.com/fxthuan|||thuannv.hust@outlook.com

IceNeoMax commented 7 years ago

Trần Hải Triều: 8 Nguyễn Văn Thuận:7

hoangphantich commented 7 years ago

Em thêm một chút review đi, ví dụ tại sao lại là 8 và 7 mà không phải 10. Nếu có thể thì chụp lại màn hình 1 tẹo. Sau này, công tác review code cũng rất quan trọng đấy nhé. Chiếm 1/2 thời gian coding.

Thường khi làm việc:

IceNeoMax commented 7 years ago

Trần Hải Triều: 8 tính năng chạy ok code dễ hiểu, dễ đọc trừ điểm UI và UX trừ điểm sử dụng nhiều document.getElementID Nguyễn Văn Thuận: 8 tính năng chạy ok UX tốt trừ điểm sử dụng nhiều document.getElementID

hoangphantich commented 7 years ago

Please review Test02 also.

IceNeoMax commented 7 years ago

Trần Hải Triều: 8 tính năng chạy ok code dễ hiểu, dễ đọc trừ điểm UI Nguyễn Văn Thuận: 8 tính năng chạy ok UI chưa giống