-
Notifications
You must be signed in to change notification settings - Fork 193
Open
Description
Line 21 in e385c76
currentDessertIndex = nextDessertIndex, |
While reviewing and referencing the codebase, I noticed that the variable currentDessertIndex (located at DessertViewModel.kt:21) doesn't seem necessary for the app's current functionality.
The app runs smoothly without it if the initial values of dessertImageId, dessertPrice, and dessertStartProductionAmount are set from the first item of the desserts list during initialization.
This simplification could improve readability and reduce the dependency on redundant state.
Enhancements:
- Remove currentDessertIndex
- Directly initialize required values from desserts[0]
- Monitor any regression during dessert upgrades
Let me know if you'd like me to provide a PR for this. Thanks for the awesome learning resource!
Metadata
Metadata
Assignees
Labels
No labels