[AM WEB] Review Coding Convention

havietduc91 commented 9 years ago

havietduc91 commented 9 years ago

Refactor Code Lần 2

Em gửi lại phần note những điểm cần thay đổi sau khi thực hiện review code.

Chỉnh lại các route cho phù hợp: phương thức GET, POST :+1:
Comment đầu function, tách rời các function cho cách nhau 1 dòng (ví dụ file AdminController) :+1:
Điều chỉnh lại mức độ visibility của các function: có nhiều function đáng nhẽ không để public lại đang public. (ví dụ hàm saveImageFileToTemp) :+1:
Sắp xếp lại thứ tự các function (public protected private) Ví dụ trong file AnnouncementController :+1:
Chưa tuân thủ coding convention (ví dụ hàm initDateOptions, enableInitPassword) :+1:
Vi phạm cách đặt tên biến (ví dụ hàm setOldImage), hàm saveUploadedImages với biến $js_image :+1:
Code chưa an toàn (chưa kiểm tra kiểu dữ liệu truyền vào) trong hầu hết các function.
Trong file constants có rất nhiều lỗi, trong file utility cũng vậy. :+1:
Hardcode: ví dụ hàm createMall, hàm getTotalFavouriteInMonth :+1:
Phần DB:: khi mở transaction ra thì phải rollback nếu mà failed ở createMall... :+1:

Phần view:

Nhiều file view của mall vẫn để nằm ngoài không đưa vào thư mục, shop cũng vậy.
Nội dung trong file cũng không thò thụt cho dễ nhìn, khó debug (ví dụ file mall-list-shop)
Nhiều file js của các view khác không dùng đến ở view A vẫn cho nhúng vào view A. (ví dụ file admin-layout) :+1:
Chưa escape giá trị user nhập: ví dụ file shop-management-newslist.
Truy xuất dữ liệu k an toàn (ví dụ file shop-mypage-detail) -> object nhưng đang truy xuất kiểu array.

havietduc91 commented 9 years ago

Refactor code phần js, file view nào dùng js nào thì mới gọi ko thì thôi :+1:
Refactor lại phần rollback(), đọc lại phần này :+1:
Làm các chức năng mới Nguyệt confirm được
- Phần ảnh của pickup
- Tính số lần login và confirm lại màn hình mypage
- Announcement khi create thì ko validate phần ended_at
Refactor các phần bên trên :+1:
[Important] Test lại toàn bộ các chức năng khi thay đổi refactor code js :+1: