Skip to content

Added Bullet Factory Functionality#2

Open
lexlam1524 wants to merge 10 commits intothoth-tech:mainfrom
lexlam1524:bullet
Open

Added Bullet Factory Functionality#2
lexlam1524 wants to merge 10 commits intothoth-tech:mainfrom
lexlam1524:bullet

Conversation

@lexlam1524
Copy link

Description

BeeFall.2024-09-16.02-57-48.mp4

Implementation of Bullet Factory to handle bullet creation and management.
The Bullet Factory class utilizes the Factory design pattern to provide a centralized method for creating bullets with specified properties. This implementation includes:

  • The SprayProjectiles method, which creates multiple bullets with a specified spread, type, origin, and direction.
  • The CreateBullet method, which creates a single bullet with a specified type, origin, and direction.
  • Support for three types of projectiles: NORMAL, FAST, and EXPLOSIVE.

The SprayProjectiles method calculates the angle step for spreading bullets and rotates the direction vector accordingly. It then creates individual bullets using the CreateBullet method.

The CreateBullet method sets the bullet's position, velocity, and scale based on the specified type and direction.

Type of change

Code

How Has This Been Tested?

Compiled and ran with
skm g++ program.cpp player.cpp obstacle.cpp bullet_factory.cpp -o game.exe

Checklist

My code follows the style guidelines of this project
I have performed a self-review of my own code
I have commented my code in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
I have requested a review from team members on the Pull Request

Copy link

@AmberPotion AmberPotion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix merge conflicts in order for the PR to be approved.

@lexlam1524
Copy link
Author

the conflict has been fixed thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments