gemmen29/ToDo-List

Peer to peer code review

Closed this issue · 0 comments

Great job done so far on this project, you have really put in a lot of effort to bring your code to the level where it is at the moment. Just minor issues below:

  • You have 3 classes for 1 entity (Task), maybe to reduce the amount of files a person has to navigate between, you can combine these files and have one file focusing on the Task entity.
  • To make your code more easy to read, trying naming your variables and function names with full terms. Example instead of tasksUL you could write it as tasksUnorderedList, createTaskLi you could write it as createListItem.