Skip to content

Sockets - Kate #46

Open
KateAnnNichols wants to merge 3 commits intoAda-C11:masterfrom
KateAnnNichols:master
Open

Sockets - Kate #46
KateAnnNichols wants to merge 3 commits intoAda-C11:masterfrom
KateAnnNichols:master

Conversation

@KateAnnNichols
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? I definitely wavered several times on what methods to place where. I found myself getting confused over what belonged in hotel vs. reservation.
What was a design decision you made that changed over time over the project? The reservation file changed over time with the project. I moved things back and forth between hotel and reservation trying to see what makes the most sense. This also impacted the tests.
What was a concept you gained clarity on, or a learning that you'd like to share? I gained more clarity on how important it is to develop and stick with (as appropriate) a solid design from the start.
What is an example of a nominal test that you wrote for this assignment? What makes it a nominal case? An example of a nominal test from this code is when I checked to ensure that data was appropriately stored.
What is an example of an edge case test that you wrote for this assignment? What makes it an edge case? An example of an edge case test that I set for this assignment was checking that the dates were valid.
How do you feel you did in writing pseudocode first, then writing the tests and then the code? I spent a long time with pseudocode to wrap my head around the project and requirements. I think in the future I will rely more on charts and design, rather than placing a majority of the emphasis on pseudocode. I also found it difficult to write tests first, especially as I began to change things in the code. I think becoming used to writing tests first will just require practice.

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