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

Ampers: Kaitlin Forsman #35

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

Ampers: Kaitlin Forsman #35

wants to merge 5 commits into from

Conversation

kcforsman
Copy link

Calculator

Congratulations! You're submitting your assignment.

Comprehension Questions

Question Answer
Describe how you stored user input in your program? I stored the user's input in different variables. Then, invoked methods with those variable as an argument to confirm viable inputs were given.
How did you determine what operation to perform? I gave a list in the introduction of the program to the user, then prompted the user to input one of the things on that list. If they failed to do use one of the examples, the program prompts them again.
Do you feel like you used consistent indentation throughout your code? Yes.
If you had more time, what would you have added to or changed about the program? I want to read up more on different methods and find if there are ways I missed that could make the code more concise. For fun, I also would like to make a method so you can invoke the input collection and calculations so you could easily ask the user if they wanted to do another calculation. Also, want to add a safety net that gives user chance to start over if they type the acceptable input (for example, the entered "add" but actually wanted to "subtract").

@CheezItMan
Copy link

Calculator

What We're Looking For

Feature Feedback
Takes in two numbers and an operator and performs the mathematical operation. Check
Readable code with consistent indentation. Check
Summary Nice work with the optional operations. Very nice how you broke things into methods. Very elegant. You hit all the requirements!

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