Skip to content

Queues - Sofia Kim - Calculator#43

Open
Sofia15 wants to merge 1 commit intoAda-C7:masterfrom
Sofia15:master
Open

Queues - Sofia Kim - Calculator#43
Sofia15 wants to merge 1 commit intoAda-C7:masterfrom
Sofia15:master

Conversation

@Sofia15
Copy link
Copy Markdown

@Sofia15 Sofia15 commented Feb 10, 2017

Calculator

Congratulations! You're submitting your assignment.

Comprehension Questions

Question Answer
Describe how you stored user input in your program? I created a method to function the calculator upon the data that user inputted.
How did you determine what operation to perform? I used a case statement to determine by user's input whether they want operator or string of the operators.
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 want to make the calculator be able to calculate with more than two numbers.

@kariabancroft
Copy link
Copy Markdown

Calculator

What We're Looking For

Feature Feedback                                                                              
Takes in two numbers and an operator and performs the mathematical operation. Well Done
Baseline
Readable code with consistent indentation. Well Done
Extras
Methods Nice use of creating methods. Methods names give context to actions being performed, and are being well utilized for code that is being repeated. Remember to have '?' on method names of methods that only return true or false.
Variables Get in the practice of giving variable names semantic meaning, instead of 'a', 'b', 'c' could be 'num1', 'num2', 'operator'.
Code is easy to read, nice clean case statement.

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