search

tuyenhv / exercise_1

1 stars 1 forks source link

First review #1

Open manhnguyen86 opened 6 years ago

manhnguyen86 commented 6 years ago
  1. Main
    • Check if user enter num of point > max support.
    • if ((i < 1) || check_input_valid(points, points[i],i)) -> anh không thấy trường hợp i < 0. Hơn nữa viết hàm điều kiện hoặc này phải cẩn thận, vì khi phép logic bên trái đúng chương trình sẽ skip toàn bộ vế sau -> hàm không chạy -> sai ngoài kiểm soát.

các hàm còn lại khá lằng nhằng a chưa review đc

tuyenhv commented 6 years ago

Hello, Thank you so much for reviewing. This comment will be fixed by commit 3fb839f. Please get the newest source code.

Some functions do not have detail comments or descriptions. I am sorry for that and I will have more explanations for algorithms used in my program later.

Thanks