Title [Automation FC] - Review Code Template ✅

TIÊU CHÍ REVIEW CODE DAO MINH DAM https://automationfc.vn Email: [email protected] Skype: daominhdam (Đào Minh Đảm) Tài liệu này chỉ sử dụng trong khóa học - vui lòng không chia sẻ công khai ----------------------------------------------------------------------------------------------------------------------- ● Đã finish xong phần Payment (10 Testcases) hoặc dự án NopCommerce (70 Testcases) ○ Bank Guru - Link ~ 70 TCs ○ Nop Commerce - Link ~ 70 TCs ● Viết script cho các dự án khác: ○ Live Guru - Link ~ 20 TCs ○ Sytner - Link ~ 100 TCs ○ PHP Travel - Link ~ 30 TCs
DAO MINH DAM https://automationfc.vn Email: [email protected] Skype: daominhdam (Đào Minh Đảm) Tài liệu này chỉ sử dụng trong khóa học - vui lòng không chia sẻ công khai ----------------------------------------------------------------------------------------------------------------------- ● 1 - Đáp ứng được yêu cầu của khách hàng ○ Người review code có thể là: Team Member khác/ Client/ Manager ○ Client là người trả tiền để các bạn build framework ○ Client clone code về ko chạy/ lỗi ○ Yêu cầu đặt ra giống như các bạn đang làm 1 bài test mà công ty X nào đó - họ gửi yêu cầu làm số lượng xx testcase (build framework) - làm xong đẩy code lên SVC - họ lấy code về run ko bị lỗi gì hết ● 2 - Chạy được trên multi-browser hoặc multi-OS hoặc multi-server ○ Windows: Phải run được trên Chrome/ Firefox/ Edge ○ MAC OS/ Linux: Phải run được trên Chrome/ Firefox/ Safari/ Edge ● 3 - Remove hết những biến/ thư viện ko sử dụng ○ Dùng phím tắt để remove khi code cho thành thói quen trên bất kì 1 IDE nào
DAO MINH DAM https://automationfc.vn Email: [email protected] Skype: daominhdam (Đào Minh Đảm) Tài liệu này chỉ sử dụng trong khóa học - vui lòng không chia sẻ công khai ----------------------------------------------------------------------------------------------------------------------- ● 4 - Convention chuẩn theo từng ngôn ngữ ● Khai báo tên biến/ tên hàm/ tên class/ tên testcase/ tên package/ tên source folder -> đúng convention của Java ○ Đặt tên lớp, interface, tên biến, phương thức, ... ○ Khoảng trắng, tab/ Khai báo và sử dụng biến ○ Comment mã nguồn: tên người tạo, phiên bản, ngày tạo file, lớp, phương thức, người thay đổi, nội dung thay đổi, ... ○ Độ dài tối đa mỗi dòng code/ mỗi file, ... ○ Để làm cho 1 vấn đề phức tạp hơn rất dễ - nhưng để làm cho 1 vấn đề ngắn gọn hơn/ đơn giản hơn mà vẫn đầy đủ thông tin rất khó ○ Refer: ■ Code Review Checklist for Java Beginners ■ Tiêu chuẩn coding trong Java (Coding Standards) ■ [Framework] Build BasePage class ■ Less is more – ngochieu.com
DAO MINH DAM https://automationfc.vn Email: [email protected] Skype: daominhdam (Đào Minh Đảm) Tài liệu này chỉ sử dụng trong khóa học - vui lòng không chia sẻ công khai ----------------------------------------------------------------------------------------------------------------------- ○ Cho cả team đi đúng hướng - làm nhanh hơn - ko mất time mà ngồi cãi nhau ■ Đặt tên biến ntn? ■ Đặt tên hàm ra sao? ■ Đặt tên package? ■ ... ○ Dùng Ctrl-Shift-F để format lại code cho đúng format ■ Client/ Manager đọc code -> đánh giá format/ convention ○ Ví dụ: ■ Verify HomePage displayed => verifyHomePageDisplayed ■ PayeesAccount: ● payeesaccount -> Sai ○ payeesAccount ● gettextuseridinfor -> Sai ○ getTextUserIDInfor ● payeesAccount -> Đúng ● getTextUserIDInfor -> Đúng