Start networking and exchanging professional insights

Register now or log in to join your professional community.

Follow

Employee getEmployee(int index) { return Employees[index]; } can you provide a code review?

user-image
Question added by Mohammad Shalabi , Cheif architect , Esatrata
Date Posted: 2014/09/20
Mohammad Shalabi
by Mohammad Shalabi , Solution Architect, Android, and IOS Architect , ALM New Way

no defensive programming, what if employee is null. security issue, what Employees[-1] will return. design issue should use uint index as parameter.

More Questions Like This

Do you need help in adding the right keywords to your CV? Let our CV writing experts help you.