Skip to content

Python#21

Open
HeleneCbn wants to merge 6 commits intonathanielpolley:masterfrom
HeleneCbn:master
Open

Python#21
HeleneCbn wants to merge 6 commits intonathanielpolley:masterfrom
HeleneCbn:master

Conversation

@HeleneCbn
Copy link

No description provided.

@nathanielpolley
Copy link
Owner

Very good, Helene. You have both GitHub and PyCharm set up correctly. Let me know if you have any further questions about this process.

Score: 0.5/0.5

Copy link
Owner

@nathanielpolley nathanielpolley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Complete: assignment 0 beginner
Pending: assignment 0 intermediate

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Score: 0.5/0.5

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For this, you have all of the calculations correct. However, you need to make sure that it handles all incorrect user input correctly. For instance, if the user enters "abc", then that cannot be converted to a float, so an error will be thrown. You also cannot take logs of negative numbers, and you can only have whole number integers for numbers of cells. This will be your outer if/else statement that will use isdigit( ) in order to evaluate if the user has input a numeric positive integer value. Also, since this is a "growth" calculator, the final value must always be larger than the initial value, so this will be an internal if/else statement that (after converting to a numerical value) will evaluate if the final is greater than initial. Keep up the good work, and I will revisit this for a final mark once you have implemented the correct conditionals.

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