For the last week of my internship, I’ve performed code review on each module that I responsible for. I’ve clean up some redundant codes and functions, renamed variables to be more understandable. Not only that, I’ve also wrote comments to clarify the implementation logic for some functions. <\/p>\n\n\n\n
Secondly, during the code review, I’ve also learned on the usage of Enum, which can be used to replace or represent a group of constants. In my case, instead of assigning magic strings as the line item’s pick status, I’ve replaced it with Enum which contains the variety of pick status. <\/p>\n\n\n\n
Lastly, I’ve moved all the magic strings to Parameter class. By using parameter class, it become more maintainable to all the strings inside the whole solution. For example, changing of dialog message will only require to change the string in parameter class.<\/p>\n","protected":false},"excerpt":{"rendered":"
For the last week of my internship, I’ve performed code review on each module that I responsible for. I’ve clean up some redundant codes and functions, renamed variables to be more understandable. Not only that, I’ve also wrote comments to clarify the implementation logic for some functions. Secondly, during the code review, I’ve also learned … <\/p>\n