VẬN HẠN NHÂM THÌN - Objective C

[nhanios]

link download:

mediafire.com/?3068f3aib2q7wc9

lần đầu tiên code mong đc góp ý:

mail nhanios@yahoo.com
yahoo:nhanios

[hoanv]

Nguyên văn bởi nhanios

link download:

mediafire.com/?3068f3aib2q7wc9

lần đầu tiên code mong đc góp ý:

mail nhanios@yahoo.com
yahoo:nhanios

HI NhanIOS!
Mình mới xem qua ứng dụng của bạn mình có chút góp ý nho nhỏ thôi.
Vì là ứng dụng khá đơn giản (gồm 2 view ) nên cũng chưa nói được gì nhiều
+ Phần code của bạn hiện tại chưa có comment hay chú thíc gì (Chỉ có đoạn liên quan đến việc tạo dữ liệu của SAO, Vận Mệnh, Vận Hạn thì gọi là có chú thíc 2,3 từ)
>> Bạn cần phải comment nhiều hơn vào code nhé! ^_^

+ Phần thông tin để hiển thị trên "thongTinViewController" bạn chưa sử dụng lại được code. Bạn đã có function "-(void)loadDataObjective *)vanHan" Nhưng sau đó khi tab vào button Sao Nam or sao Nữ bạn lại phải copy + pase đoạn code trong này vào đó. Theo mình thì ko ổn lắm ^_^.
>> Bạn có thể thiết kế function này để dùng lại được mà đung ko?. Nhưng việc tính đoán tọa độ để cho các UITextView vào bạn có thể làm các nào đó để nếu về sau có thay đổi về design thì chỉ cần thay đổi rất ít về code hoặc sửa tham số là xong. Còn nếu để như bây giờ sau mà thay đổi giao diện 1 chút là bạn phải tính toán lại khá mất thời gian đó.

>> Đôi dòng ngu ý nếu có gì sai mong anh em chỉ bảo thêm!

[duynk163]

Dear Nhanios,

Kiểm tra Analyze đi, vẫn còn leak đó bạn.

[Mango]

Nguyên văn bởi hoanv

HI NhanIOS!
>> Bạn cần phải comment nhiều hơn vào code nhé! ^_^

không cần thiết, comment nhiều chỉ tổ làm rối code.

[zendobk]

Cứ phang code từ đầu tới cuối, chỗ nào dùng giải thuật thì ghi giải thuật gì sau này đọc cho lẹ, hết.

[hoanv]

Nguyên văn bởi Mango

không cần thiết, comment nhiều chỉ tổ làm rối code.

Mỗi người 1 cách làm thôi bạn!
Với mình thì comment tốt và minh tin là sau nàyng khác dọc code của mình cũng đỡ hơn.

[kemsiro]

Nguyên văn bởi hoanv

Mỗi người 1 cách làm thôi bạn!
Với mình thì comment tốt và minh tin là sau nàyng khác dọc code của mình cũng đỡ hơn.

mình đồng ý với bro này, mình comment chủ yếu ở file header, maintain hoặc collaborate cho dễ

[hungknorton]

Nguyên văn bởi kemsiro

mình đồng ý với bro này, mình comment chủ yếu ở file header, maintain hoặc collaborate cho dễ

Mình thấy code của apple hoặc open source nước ngoài họ comment rất chi tiết. Mình chắc chắc rằng nhiều khi các bác code xong một thời gian sau nhìn lại ko biết mình code cái gì.

[Mango]

mình thấy nhiều người comment code rất dư thừa như:

self.backgroundColor = [UIColor blackColor]; // cho nền màu đen

chỉ nên comment những chỗ thật sự cần thiết

[kemsiro]

Nguyên văn bởi Mango

mình thấy nhiều người comment code rất dư thừa như:

self.backgroundColor = [UIColor blackColor]; // cho nền màu đen

chỉ nên comment những chỗ thật sự cần thiết

cái này là comment nhảm