Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sara Nilsen - Space #35

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Conversation

sarabrandao21
Copy link

Assignment Submission: Hotel

Congratulations! You're submitting your assignment. Please reflect on the assignment with these questions.

Reflection

Question Answer
What was a design challenge that you encountered on this project? Deciding how many classes to start with was very challenging.
What was a design decision you made that changed over time over the project? I added a class called room later on and instantiated in the hotel controller class creating an array of instances of room.
What was a concept you gained clarity on, or a learning that you'd like to share? I gained more confidence writing tests and classes. At first I had a hard time trying to connect the class, but started being more confident after 3 days reviewing the lessons about classes and instances.
What is an example of a nominal test that you wrote for this assignment? What makes it a nominal case? "date range can be initialized with two dates" is a nominal test because is what I expect my algorithm to do.
What is an example of an edge case test that you wrote for this assignment? What makes it an edge case? "It is an an error for negative length ranges, raises an argument error.". It is an edge case because is not something I am expecting to happen but if it happens I need to take care of it.
How do you feel you did in writing pseudocode first, then writing the tests and then the code?
I can see why is effective but I still struggle writing the tests before the program. I found really helpful to write pseudocode first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants