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

[Dana] (Simple) Calculator Program #65

Merged
merged 2 commits into from
Mar 6, 2025
Merged

Conversation

kkgiorgos
Copy link
Contributor

The program reads the input line by line and on each line it expects two positive integers separated by an operator (any other character among them is ignored). If the input is valid it outputs the operation's result otherwise it fails on that line with a message "Invalid"

There is also a sample input and output for the program

The program takes reads the input line by line and
on each line it expects an integer followed by and operator and then
another integer, any other character among them is ignored.
If the input is valid it outputs the operation's result otherwise it
fails on that line with a message "Invalid"

There is also a sample input and output for the program
@kostis
Copy link
Owner

kostis commented Mar 6, 2025

Thanks for your contribution. It looks good, but perhaps some of the information in the commit message of this PR should appear as comments in the program's text. (So that it is clear what the program does.)

Can you please add them in the program and then I'll merge this.

@kkgiorgos
Copy link
Contributor Author

I've made a small change in the existing comments of the program.

@kostis kostis merged commit e46d0c7 into kostis:master Mar 6, 2025
2 checks passed
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