Skip to content

Grace - Sockets#44

Open
gracemshea wants to merge 26 commits intoAda-C11:masterfrom
gracemshea:master
Open

Grace - Sockets#44
gracemshea wants to merge 26 commits intoAda-C11:masterfrom
gracemshea:master

Conversation

@gracemshea
Copy link
Copy Markdown

Hotel

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
What was a design challenge that you encountered on this project? Partway through, I decided to make a Helpers class to hold all my helper methods. Then I decided to make sorting methods, and binary search methods. I am not finished, so I think that reflects my difficulty accomplishing this. Another design choices was breaking out room. I debated on whether this was necessary. It probably isn't.
What was a design decision you made that changed over time over the project? Well, I'm somewhat regretting the refactoring, I saw an example of binary search application on Github and kind of restructured some of my helper methods to orient around the idea. I'm not sure this was a good move at all, because I'm still working through a lot of errors.
What was a concept you gained clarity on, or a learning that you'd like to share? I didn't feel really great about writing tests prior to this project. I think just having to write ALL the tests was really helpful, and keeping an eye on coverage was really instructive.
What is an example of a nominal test that you wrote for this assignment? What makes it a nominal case? Makes an instance of Reservation
What is an example of an edge case test that you wrote for this assignment? What makes it an edge case? Invalid date range
How do you feel you did in writing pseudocode first, then writing the tests and then the code? My processed ended up being: 1- name the test, 2- then pseudocode, 3- then pseudocode the test, 4- write the code in full, 5- write the test in full. I think I did okay, I just circulated through my files and kept adding more each time.

@droberts-sea droberts-sea changed the title sti Grace - Sockets Mar 27, 2019
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.

1 participant