-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Team 1 Quality of Life changes to mobs #285
Conversation
…om/UQcsse3200/2023-studio-3 into Team-1--Jono-True-Love
@@ -30,6 +32,7 @@ public class DodgingComponent extends Component { | |||
private float dodgeSpeed = 1.75f; | |||
private float originalSpeed; // Original entity vertical speed | |||
private PhysicsEngine physics; | |||
private Random random = new Random(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Random keeps popping up as an issue on SonarCloud, is there anything else that could be used instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MiniSoda17 You can use RandUtils to generate random numbers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks for doing some cleanup of FGA!
…erCountDisplay is updated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good to me!
…b.com> Co-authored-by: gregchan550 <[email protected]>
Updates