{"id":1179,"date":"2025-05-09T18:37:34","date_gmt":"2025-05-09T18:37:34","guid":{"rendered":"https:\/\/sites.wsagames.com\/et4g23\/?p=1179"},"modified":"2025-05-09T19:44:14","modified_gmt":"2025-05-09T19:44:14","slug":"player-pt-4","status":"publish","type":"post","link":"https:\/\/sites.wsagames.com\/et4g23\/2025\/05\/09\/player-pt-4\/","title":{"rendered":"Player – Pt 4"},"content":{"rendered":"\n

When developing the Player character further, we had noticed a lot of bugs and issues during this process, this post outlines our discussions, decisions and what I had to do in order to fix them.<\/p>\n\n\n\n

Outlining Issues and Changes<\/h2>\n\n\n\n

Firstly, when Ayse, our Mechanics Designer pushed her initial version of the Player with its new attack, I noticed two issues: It was able to move around with the attack hit box activated and The player could hold left-click and the attack hitbox would constantly be active. Which caused me to do the following:<\/p>\n\n\n

\n
\"\"<\/figure><\/div>\n\n\n

However, neither me nor our Mechanics Designer was unable to find a way around these two issues:<\/p>\n\n\n\n

\"\"<\/figure>\n\n\n\n

Which resulted in me suggesting to re-do the player entirely for a few reasons:<\/p>\n\n\n\n

\"\"<\/figure>\n\n\n\n

After explaining my though process, our Mechanics Designer also agreed it would be best for me to re-do the player entirely:<\/p>\n\n\n\n

\"\"<\/figure>\n\n\n\n

Fixing the Player<\/h2>\n\n\n\n

In order to fix the player, I had to re-do it entirely. Therefore, I not only had to recreate some existing features, I also needed to fix issues and bugs that occurred with the previous iteration of the player:<\/p>\n\n\n\n

    \n
  • Reduce the amount of attack directions the player had from 8 to 4\n
      \n
    • (Reduce the workload on our animator)<\/li>\n<\/ul>\n<\/li>\n\n\n\n
    • Make the players’ position locked while attacking\n
        \n
      • (The player was able to move while attacking in the previous version)<\/li>\n<\/ul>\n<\/li>\n\n\n\n
      • Can dash during attack<\/li>\n\n\n\n
      • Play the correct animation when the player attacked<\/li>\n\n\n\n
      • Deactivate the attack hitbox after a duration<\/li>\n<\/ul>\n\n\n
        \n
        \"\"
        My GitHub commit to the lament repository, fixing the player<\/figcaption><\/figure><\/div>\n\n\n

        Attack<\/h2>\n\n\n\n
        \n
        \n

        Firstly, the function of the player that was causing the most issues was its attack<\/strong>, so this was the part I decided to tackle first. To begin, I created four separate hit boxes for its attack, one of which would activate when the player pressed left-click, depending on what direction<\/em><\/strong> the player was facing (Explained in the Movement <\/strong>section below).<\/p>\n<\/div>\n\n\n\n

        \n
        \"\"\n\t\t\t\n\t\t\t\t\n\t\t\t<\/svg>\n\t\t<\/button><\/figure><\/div><\/div>\n<\/div>\n\n\n\n

        To implement this logic, I created a new C# script called AttackNew:<\/p>\n\n\n\n

        \"\"<\/figure>\n\n\n\n

        This script had references to all of the player’s attack hitboxes e.g: public GameObject upMelee;<\/strong><\/em> and other variables for the attack, such as: public float atkDuration = 0.5f;<\/strong><\/em> , public bool isAttacking = false;<\/strong><\/em> , Animator anim<\/strong><\/em> etc.<\/p>\n\n\n\n

        In the Start()<\/strong><\/em> method, some of these references are gotten, like the player’s controller script and animator.<\/p>\n\n\n\n

        The Update()<\/strong><\/em> method is constantly calling the CheckMeleeTimer() <\/strong><\/em>method and checking if the player has pressed left-click, if they have and the player isn’t attacking, the HandleAttack() <\/strong><\/em>method is called.<\/p>\n\n\n\n

        This HandleAttack() <\/em><\/strong>method, sets the isAttacking animator bool <\/strong><\/em>to true, which can then be used in Unity’s animator to play the correct attack animation, it then starts the MeleeActive() coroutine <\/strong><\/em>and sets the isAttacking bool <\/strong><\/em>to true.
        This MeleeActive() coroutine <\/strong><\/em>waits a small duration (For the windup of the attack animation) and then activates one of the melee hitboxes depending on what direction the player is facing. Then, after another small duration, the hitbox is set to inactive and the isAttacking bool <\/strong><\/em>and isAttacking animator bool <\/strong><\/em>is set to false <\/strong><\/em>aswell. To prevent the attack animation from being played and allowing the player to attack again.<\/p>\n\n\n\n

        Movement<\/h2>\n\n\n\n

        Although I created a new C# script called PlayerControllerNew, the majority of it is kept the same from its previous iteration, such as its: Player movement, inputs and dash. However, one of the only differences is that this script stores what direction the player is currently facing:<\/p>\n\n\n\n

        \n
        \n

        In order to do this, I created 4 bools under the heading of “FacingDirection”, only one of these bools would ever be active at once, essentially storing which direction the player was facing.<\/p>\n<\/div>\n\n\n\n

        \n
        \"\"<\/figure>\n<\/div>\n<\/div>\n\n\n\n
        \"\"<\/figure>\n\n\n\n

        In order to set these bools, I created a SetFacingDirection(bool right, bool left, bool up, bool down) <\/strong><\/em>method, which would take in input of 4 separate bools, each corresponding to a direction that the player could be facing.<\/p>\n\n\n\n

        Then, in the ProcessInputs()<\/strong><\/em> method of the PlayerControllerNew script, the method that was responsible for the players’ movement, I created a set of 4 if statements that would check which was the most recent direction the player had moved in, and would set the player to face in that direction using the SetFacingDirection() <\/strong><\/em>method. Therefore, the player would be facing a specific direction whether the player was currently moving in that direction, or had stopped moving entirely.<\/p>\n\n\n\n

        \n
        \n

        Furthermore, I created an Animate() <\/em><\/strong>method that would set an animator float called “isAttacking 1<\/strong>” to a certain value between 0 and 1 depending on what direction it was facing. Therefore if the player was facing right (90o<\/sup>) its isAttacking 1 <\/strong>float would be 0.25f<\/strong> (90\/360).<\/p>\n\n\n\n

        Then, using this “isAttacking 1<\/strong>” float and the “isAttacking<\/strong>” animator bool, the player would be able to play the correct animation for its attack, according to the direction its facing.<\/p>\n<\/div>\n\n\n\n

        \n
        \"\"<\/figure>\n<\/div>\n<\/div>\n\n\n\n

        The one other difference made in the PlayerControllerNew script is preventing the player from moving during their attack, in order to do this I created a reference to the player’s AttackNew script, which was then set in the Start() <\/strong><\/em>method.
        The Update() <\/strong><\/em>method would then perform a check to see if the player was attacking before allowing any inputs to move the player.<\/p>\n\n\n\n

        \"\"<\/figure>\n\n\n\n

        This was the result of the new player:<\/p>\n\n\n\n

        <\/p>\n\n\n\n