-
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 4 Levels - General Levels Updates #200
Conversation
WaveManager WaveService WaveClass Further implemtation required
Explains new functionality about spawning waves
…s remaining) Need to set what happens when the level is completed, as game currently crashes due to index out of bounds error.
…aveClass and WaveFactory
…into Team-4---Waves
…eTask to return the sounds Array.
…eCount in the WaveTask.java when a new wave is started. This will be used hopefully for some UI showing the player what wave they are in.
…ves-max # Conflicts: # source/core/src/main/com/csse3200/game/components/tasks/waves/WaveTask.java # source/core/src/main/com/csse3200/game/services/WaveService.java
… lane the next mob will spawn in
… lane the next mob will spawn in
…to aid in testing of the waves feature.
…ave been added to wave related classes.
…ll be spawned in WaveService.java. Also made the wave start timestamp update in LevelWaves.java (the first time waves are made) & in WaveTask.java (For when a wave ends and a new one starts).
# Conflicts: # source/core/src/main/com/csse3200/game/components/tasks/waves/LevelWaves.java
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.
LGTM !
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!
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.
There appears to be no function for pausing the wave timer mid-game, only startWaveTimer() and stopWaveTimer() which appear to only work at the setup and cleanup of the stage.
I'd like for WaveTask to have some method of pausing the in-between waves timer that can I can access with the pause menu entity.
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.
There are some minor javadoc errors in getNextWaveTime() and getWaveCount() methods in WaveService. Other than that, everything else looks good!
The pull request include several features of levels that have been created such as: