Skip to content
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

Mummies attacking near players #227

Merged
merged 1 commit into from
Feb 23, 2012
Merged

Mummies attacking near players #227

merged 1 commit into from
Feb 23, 2012

Conversation

Ankkah
Copy link
Collaborator

@Ankkah Ankkah commented Feb 23, 2012

Used isTargetBehindWall() to prevent them running into wall.

Todo would be pathfinding like that:
https://chaos.bcit.ca/svn/public/astar/

Correspond to
#210

@Judgy53
Copy link
Collaborator

Judgy53 commented Feb 23, 2012

Can't test it now but +1

@Judgy53 Judgy53 closed this Feb 23, 2012
@Judgy53 Judgy53 reopened this Feb 23, 2012
@Judgy53
Copy link
Collaborator

Judgy53 commented Feb 23, 2012

Fail click ... sorry

@mkalam-alami
Copy link
Collaborator

It seems to work. The isTargetBehindWall() method could probably use some refactoring, but it's fair enough until we have some true pathfinding on tracks.

mkalam-alami pushed a commit that referenced this pull request Feb 23, 2012
Mummies attack near players
@mkalam-alami mkalam-alami merged commit 6c3f44d into Maescool:master Feb 23, 2012
@Ankkah
Copy link
Collaborator Author

Ankkah commented Feb 23, 2012

I took it from turret.java and modified it instead of writing a new one :)

@mristau
Copy link
Contributor

mristau commented Feb 23, 2012

Don't those classes implement/extend from the same class, so a TargetBehindWall() could be written in one place for all mobs

@mkalam-alami
Copy link
Collaborator

Apparently the code is slightly different, it could probably be refactored but it's temporary anyway
(I just opened #249 to start thinking about a long-term solution)

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

Successfully merging this pull request may close these issues.

4 participants