Repository navigation
Project complete - #40
AaronjMurphy wants to merge 2 commits into
Conversation
🚀 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. |
TheCSharpAcademy
left a comment
There was a problem hiding this comment.
Hi @AaronjMurphy thanks for submitting! At the moment I can't review your project because it doesn't cointain .csproj and .sln project files. Here's a working example:
|
Issues with missing files have been fixed! |
|
@AaronjMurphy Project approved! 😄✅And well done! You’ve implemented calculation history, history deletion, reuse of previous results, a calculation counter, and several additional mathematical operations. 👏👏 Feedback 🔍The one thing I'd do to make the code more maintainable is replacing List history entries with a small Calculation class containing operands, operation, and result. UseCalculation() currently has to recover the result with Split('=').Last(). Storing structured data would let you simply access something like calculation.Result, which is safer and easier to extend. ☕If you like our roadmap, please consider buying us a coffee. We appreciate your help 🙂 Overall great work!✋🏻Looking forward to seeing your next projects! |
This is a completed version of the Calculator app project. The code from following the tutorial has been altered to allow the completion of challenges.