My first Smart Contract (Lottery) 🥳 🥳 🥳

[RobItu]

After completing the 🎉Solidity Basics🎉 I decided to integrate all that knowledge and make my first Smart Contract in Remix 🥳.

                       https://github.com/RobItu/Lottery-Smart-Contract

This is a special Lottery contract, where a predetermined number of players participate (entrance tickets cost 0.01ETH) and the winner, whose chosen at random, withdraws the funds. From what I've read, actual lottery companies give out 50% of their money as the prize and keep the remaining 50% as profit, so I used PaymentSplitter contract from OpenZeppelin to distribute the funds. I obtained a random number from Chainlink's VRF. Only one entry allowed per address (added only for complexity/practice purposes).

There was one bug that I still haven't been able to fix:

• If you spam the retrieve() function from an external wallet many times, some addresses will get in despite the cap on the amount of players. I believe the answer to this error may have to do with block number confirmations, I have not been able to solve this error yet. If you find any more bugs or have an idea on how to solve this let me know!

Took me almost a week to make this but I had tons of fun. Can't wait to revisit this project later and make it more gas efficient and more secured. Thank you for checking it out! I would love to hear your thoughts/criticisms on it 🤗

PS. And thank you @PatrickAlphaC for the great course!

[krakxn]

Awesome! I will take a look.

[krakxn]

@RobItu If this was your first project, I would recommend formatting—by yourself—your codebase in NatSpec. Just for practice, as clear documentation is vital if you decide to pursue this field further.

[RobItu]

Yeah this was my first project, but you're very right I completely missed this part of the project! I was so focused on just getting it to work how I wanted. Greatly appreciate the insight.