Repository navigation
Finished. - #43
Finished.#43abdukulov-talgat wants to merge 10 commits into
Conversation
Literally just copy paste from Microsoft.
🚀 Project Submitted!Thanks for submitting your project! I will review it soon. 📋 Please check the dashboard to make sure the project is marked as Pending. |
|
@abdukulov-talgat Project approved! 😄✅Well done for completing the challenges! You followed the tutorials and modified the code very effectively for the challenges. You also cleaned up the messy code from the tutorial quite a lot. Well done! 👏👏 Feedback This doesn't properly support inputs such as negative numbers or decimals, despite the calculator using double. double.TryParse would be a much better fit and would simplify the validation at the same time. 🔍Improve file error handling. Loading calculator-log.json is protected by a try/catch, but Save() isn't. A file permission or disk error could therefore crash the application when performing or clearing a calculation. 🗺️ I saw you completed the Introduction to Object-Oriented Programming, I hope you enjoyed it! Next I recommend taking our LINQ Fundamentals course next. LINQ is one of the most important tools in modern C# and becoming comfortable with it will make your code much cleaner and more expressive. ☕If you like our roadmap, please consider buying us a coffee. We appreciate your help 🙂 Overall good work!✋🏻Looking forward to seeing your next projects! |
This time it's simpler.