-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
Grade: ✓+
Overall nice job on the assignment, here are some things you could still improve on moving forward:
- NumCredits must be blueprintreadonly not readwrite, now someone might change it in Blueprint and avoiding the ChangeCredits function.
- giving code more whitespace helps with readability especially for others trying to read what is going on in your code.
- should share more functionality between powerups rather than calling visibility/collision/interactable=false for each of your pickups. You can place this in a re-usable function in your pickup base class.
- Make sure you give your bindings easy to understand names like Get_0_Text_0, this will quickly become difficult to remember what that is bound to.
- Using an event would be better for your credits like OnCreditsChanged, this will be more efficient for your widget as well rather than direct binding.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels