Refactor the code to remove clean architecture violations, namely the ones stated in the design document.

[Supermac30]

The main two violations are that

• PreferenceBookCSVReader gateway uses the constructor for a PreferenceBook
• RateRecipeCommand controller calls the getUsername getter method in the User entity
• RecipeCSVReader gateway uses the constructor for a RecipeCollection

[Supermac30]

As stated in the design document: "[...] there are some violations where entities are accessed directly from the controller and gateway layer in our code left to fix. Namely, the PreferenceBookCSVReader gateway uses the constructor for a PreferenceBook, which we can fix by creating a PreferenceBookFactory, and the RateRecipeCommand controller calls the getUsername getter method in the User entity, which we can fix by creating a new UseCase class with this responsibility."

[Supermac30]

Other identified violations include the constructor for the RecipeCollection, an entity, being called in the RecipeCSVReader class, a gateway.