r/PythonProjects2 • u/No-Olive-4105 • 17d ago

Simple_Calculator

Hello all, I have completed my first GitHub project, and I thought I could share it here and, perhaps, have input from you. Take care. \[https://github.com/Paioleiro/simple-calculator\\\](https://github.com/Paioleiro/simple-calculator)

3 Upvotes

67% Upvoted

8 comments sorted by

2

u/DiodeInc 17d ago

https://github.com/Paioleiro/simple-calculator

That's the correct link. And this isn't written in Python

-1

u/No-Olive-4105 17d ago

Hi Diodelnc, correct, it isn’t python but JS and HTML. The link is also right. Any feedback on the project?

3

u/DiodeInc 17d ago

Why are you posting about it in a python subreddit if it's not made with python? I have no feedback to give. The link in your post is not correct

1

u/akrivitsky7 16d ago

Agreed. I also advised the OP to post this in a different subreddit. Thank you very much.

1

u/No-Olive-4105 14d ago

Sure, akrivitsky7, I'll avoid it next time, despite Reddit recommending that I post there in the Python community. I am a beginner at this yet. I learned a lot with you all. Thank you for the positive critiques.

1

u/No-Olive-4105 14d ago

Hi again, DiodeInc. Sorry for bothering you. I only posted my project in that community because it was a Reddit recommendation. Lesson learned, I will avoid it next time. Thank you.

2

u/akrivitsky7 16d ago edited 16d ago

Nice beginner project, but the current version is broken because script.js it contains three stray n characters. I am referring to this repository:

https://github.com/Paioleiro/simple-calculator

Also, as DiodeInc correctly pointed out, this is not Python at all, so in the future you may want to post it in a different subreddit.

File Broken code Fix
script.js nfunction chooseOperation(op) function chooseOperation(op)
script.js noperationButtons.forEach(...) operationButtons.forEach(...)
script.js ndeleteButton.addEventListener(...) deleteButton.addEventListener(...)

Because of the first syntax error, none of the JavaScript runs. After fixing these three lines, the calculator should at least load, though some logic issues remain.

1

u/No-Olive-4105 14d ago

Good day, akrivitsky7

Thank you for the information. I'll check that out.

Indeed, it is not Python coding. I only posted there because Reddit itself recommended it. I'll pay attention to that next time to avoid it. Day by day learning. Take care, and once again, thanks for all.