Ball class with fields color and number created with toString() metho… #1737
Conversation
…d overriding Colors enum created (5 colors) ColorSupplier class - getRandomColor() method created Lottery class with getRandomBall() method created In the main() method 3 balls are created using getRandomBall method of Lottery class with data output to the console
There was a problem hiding this comment.
Great job on implementing the task requirements! 🎉 Your code is functional and meets the core requirements. While there are a couple of suggestions for improvement, such as using a loop to create Ball objects and modifying the Ball constructor to accept a Color type for better type safety, these are minor and do not impact the functionality. Keep up the good work and consider these enhancements for future projects! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| Ball ballOne = lottery.getRandomBall(); | ||
| Ball ballTwo = lottery.getRandomBall(); | ||
| Ball ballThree = lottery.getRandomBall(); |
There was a problem hiding this comment.
Consider using a loop to create the three Ball objects. This approach is recommended in the checklist to avoid repetitive code when creating multiple instances of the same class .
| color = supplier.getRandomColor(); | ||
| index = new Random().nextInt(100); | ||
|
|
||
| Ball ball = new Ball(color, index); |
There was a problem hiding this comment.
Consider changing the Ball constructor to accept a Color type instead of a String for the color parameter. This will improve type safety and align with the use of the Color enum.
…d overriding
Colors enum created (5 colors)
ColorSupplier class - getRandomColor() method created Lottery class with getRandomBall() method created
In the main() method 3 balls are created using getRandomBall method of Lottery class with data output to the console