UserManager instance attribute

[Tyler-CY]

What should the instance attribute of UserManager be?

Should it be a Map<String, User> or Map<String, String>?

The former makes the most sense as it stores the username as key and User datatype as value.

The latter makes some sense if we want to have addUser(String username, String password), or else we need to create an entity (User) for the second argument.

[maxBridgewater]

I think it should be the first one. What we can do is have a controller class that when it tells teh userManager to add a user, all the user manager needs to do is return the type User. Then the controller class can take that user entity and write it out to the .json file. I am a little confused as well, so we'll see what other people think too.

[Tyler-CY]

Conclusion: Map<String, User> is the final decision. The first argument is the username (String) and the second argument is a User entity.