Code review - Githubissues

bacdk / PTBac2

0 stars 0 forks source link

Code review #1

Open tiennth opened 7 years ago

tiennth commented 7 years ago

1. Coding convention

Tên biến, tên hàm em cần phải theo chuẩn - coding convention.
Điểm chưa tốt là em không có comment khi code.

2. Code design

Scope của biến (local, global) hiện tại tốt.
Em có tách method ra để xử lý => Good. Tránh việc viết toàn bộ xử lý vào 1 method nhé.
Code xử lý business (giải phương trình) nên tách biệt khỏi code xử lý UI. Hiện tại cả 2 đang gom chung. Vd: Tách việc giải phương trình ra 1 method. Method giải phương trình xử lý và trả về kết quả. Kết quả đó được show lên UI trong 1 method khác.

3. Logic.

Em đã có ý thức check trường hợp textfield để trống và trường hợp user input ký tự không hợp lệ => Good. Điều đó sẽ tránh cho việc app bị crash hoặc gặp các bug về data khác. Tuy nhiên code validate của em chưa hiệu quả nhé.

4. License

Anh thấy có 2 bạn có cùng sản phẩm, không biết bạn nào là tác giả nhỉ?

bacdk commented 7 years ago

Em cảm ơn Anh! Em sẽ có gắng sửa đổi và chú ý hơn. Còn về license thì bài này em làm, bạn em hôm đó hông đi nên 2 đứa dùng chung code.