Need help with this simple AI script

Unity Engine
1

Hey guys. So I wrote this AI script for a monster for my 2d game and for some reason _isGrounded bool is always on even when the character isn’t touching the ground. Also when the monster is against the wall it just flies up. Why would it fly up since in order to jump there are 3 criteria ( if (_jumpToAvoid && _canJump && _isGrounded)). Also if you think its messy code let me know how to optimize it. Thanks a ton.

using UnityEngine;
using System.Collections;

public class EnemyMovement : MonoBehaviour
{
    public Transform sightEnd, ground, sightBegin, jumpEnd;
    public float speed,jumpPower;
    private Rigidbody2D _rb2d;
    private bool _avoid, _isGrounded, _isFacingRight, _jumpToAvoid, _canJump;

    void Awake()
    {
        _rb2d = GetComponent<Rigidbody2D>();
    }

    void Update()
    {
        int mask1 = 1 << LayerMask.NameToLayer("Ground");
        int mask2 = 1 << LayerMask.NameToLayer("Obstacle");
        int mask3 = 1 << LayerMask.NameToLayer("Player");
        int combinedMask = mask1 | mask2 | mask3;

        Debug.DrawLine(transform.position, sightEnd.position, Color.red);
        _isGrounded = Physics2D.Linecast(transform.position, ground.position, mask1 | mask2);
        _avoid = Physics2D.Linecast(sightBegin.position, sightEnd.position, mask3);
        _jumpToAvoid = Physics2D.Linecast(sightBegin.position, jumpEnd.position, mask1 | mask2);
    }

    void FixedUpdate()
    {
        if (_isGrounded)
        {              
            _rb2d.velocity = new Vector2(speed, _rb2d.velocity.y);
            _canJump = true;

            if (_avoid)
            {
                speed = -speed;
                _avoid = false;
            }
        }
        //====================================
        if (speed < 0 && !_isFacingRight)
            Flip();
        else if (speed > 0 && _isFacingRight)
            Flip();
        //====================================
        if (_jumpToAvoid && _canJump && _isGrounded)
        {
            _rb2d.AddForce(new Vector2(0, jumpPower));
            _canJump = false;
        }
    }

    void Flip()
    {
        _isFacingRight = !_isFacingRight;
        Vector3 scale = transform.localScale;
        scale.x *= -1;
        transform.localScale = scale;
    }
}
2

I can’t test it but I would suggest changing this line:

_isGrounded = Physics2D.Linecast(transform.position, ground.position, mask1 | mask2);

and testing it with a “<” or “>” instead of “=” using the fraction property.

3

Thanks. I’ve figured out the problem. The players collider was overlapped with the linecast. Thats why _isGrounded was always true.