PIE-NU / Expedition-2018

Our game developed as part of the 2018 Expedition!
0 stars 0 forks source link

BasicMovement.cs should be split? #29

Open Grygoree opened 6 years ago

Grygoree commented 6 years ago

Should modularize the portion of code that controls NPC movement so that it's not part of the script that also controls player movement. Either that or completely remove it if it's not used in the future.

That way the file is easier to understand, and the protagonist prefab doesn't have unnecessary functionality in the editor.

DrDoak commented 6 years ago

I originally bunched the two pieces of code together for testing purposes on my own game. The idea was that I could just flip a boolean to take control of any character. Since my game had a lot of precise hitboxes, this was very helpful to test out the attacks of each enemy. I would recommend consulting with the people currently working on Control @angelomatni2018, @dakersnar and @blabadie1 are the ones I believe.