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

Kayla Ecker - Carets #43

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

kaylaecker
Copy link

Calculator

Congratulations! You're submitting your assignment.

Comprehension Questions

Question Answer
Describe how you stored user input in your program. used method def return so that the information that is made would be stored and returned, and used gets.chomp to save input and to convert to appropriate state (such as float etc)
How did you determine what operation to perform? Because it was the one that was needed, and is the best option with my current knowledge and time availability to figure out how to use.
How did you make sure your program behaved correctly? Ran checks of various types, like entering words instead of numbers to check for handling of erroneous input.
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 would have made it allow additional inputs, such as exponents and parentheticals

@CheezItMan
Copy link

CheezItMan commented Aug 14, 2017

Calculator

What We're Looking For

Feature Feedback
Takes in two numbers and an operator and performs the mathematical operation. Check
Baseline
Readable code with consistent indentation. Check
Extras
Nice work, you hit all the requirements. Good work using methods and detecting if the user entered a 0 for the denominator. I am not sure you understood the comprehension questions. #1 was asking about what types of variables you stored information in. #2 dealt with how your program determines if the operation is addition, multiplication etc...

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.

3 participants