mauricioaniche / sem-pacman-python

PacMan game used in the Software Engineering Methods (CS minor, TU Delft)
12 stars 22 forks source link

If ghost plans to move to your location in next move, you die #2

Open mdjong1 opened 6 years ago

mdjong1 commented 6 years ago

When the player moves away from a ghost that is beside him, he will die. Probably related to https://github.com/mauricioaniche/sem-pacman-python/commit/ae43d2522221dd8bffbfd8650fbdc3d8b6b9ca27 in which ghosts get to move first. Below are images attached of two situations in which this happened. (Note: these are for a randomly generated map which may appear in a later PR).

Behavior could be explained by ghost moving faster than pacman (which it is in the real game, I believe). However, then Pacman should disappear off the board (and perhaps reason for losing should be printed (you're not fast enough)).

image image

iamlucasvieira commented 4 years ago

Do you know if this also happens when using the AsciiArtUI() for the UI? I noticed that, when using AsciiArtUI, the ghosts that killed you is removed from the board. Maybe the ghost that moved faster than you also disappear, so perhaps the user can visualize what happened.

iamlucasvieira commented 4 years ago

Just found a case with the AsciiArtUI where the ghost is not removed from the board when you're killed. I think that to the user it might be a bit confusing how the death happened.

image

mauricioaniche commented 4 years ago

Yeah, if you both move to the same place, the app doesn’t really show a clash and the player may feel confused. That’s a good feature improvement!

On Fri, 25 Sep 2020 at 09:27, Lucas dos Santos notifications@github.com wrote:

Just found a case with the AsciiArtUI where the ghost is not removed from the board when you're killed.

I think that to the user it might be a bit confusing how the death happened.

[image: image] https://user-images.githubusercontent.com/34292412/94238449-b33e8500-ff10-11ea-838c-d7c8fd608179.png

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/mauricioaniche/sem-pacman-python/issues/2#issuecomment-698770645, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAYTTC6INCPPEKCW4SSGS3SHRA4JANCNFSM4FZVHCEA .

--

Maurício Aniche Delft University of Technology http://www.mauricioaniche.com