Unity多模式枪械加载/切换后自动开火问题排查与解决

问题描述 #

我制作了一把具备多种射击模式的枪械，在加载场景时切换至该枪械，或切换其他枪械后切回该枪械时，它会自动发射一发子弹并伴随音效和光效。已尝试在Start方法中增大初始值添加开火延迟、调整auto方法中timeBeforeShooting时长、修改shooting变量状态，但均无效。

问题原因 #

1. 状态残留未重置：当枪械通过SetActive(true)切换激活时，Start方法仅在对象首次加载时执行一次，后续切换不会重新触发。而之前禁用枪械时，timeBeforeShooting会被auto模式的逻辑重置为0（松开鼠标时触发），导致再次激活时，若玩家按住鼠标左键，会直接满足timeBeforeShooting <= 0的开火条件。
2. 初始化逻辑覆盖：首次加载时Start设置的timeBeforeShooting = 1/fireRate，会被后续松开鼠标的操作覆盖为0，切换枪械激活时无法自动恢复初始延迟值。

解决步骤 #

1. 新增激活时状态重置：添加OnEnable方法，每次枪械被激活（切换回来）时，统一重置timeBeforeShooting和shooting状态，避免残留值导致误触发。
2. 移除无效代码：删除Start中无意义的playerView.GetComponent<Camera>()（若playerView已通过Inspector拖入，该行完全多余）。
3. 优化代码冗余：提取通用开火逻辑为独立方法，减少重复代码，便于后续维护。

修改后的完整代码 #

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class AutomaticGun1 : MonoBehaviour
{
    public enum firingMode
    {
        auto = 1,
        semi = 2,
        burst = 3
    }

    [SerializeField] private firingMode currentFiringMode = firingMode.auto;

    [SerializeField] private LayerMask enemyLayer;
    [SerializeField] private float damage;
    [SerializeField] private float fireRate;

    private float timeBeforeShooting;
    private bool shooting;
    private int firingModeID = 1;

    public Camera playerView;
    public ParticleSystem muzzleFlash;
    public AudioSource bang;

    // 首次加载时初始化状态
    void Start()
    {
        ResetGunState();
        // 若playerView未通过Inspector赋值，取消下面注释
        // playerView = GetComponent<Camera>();
    }

    // 每次激活枪械时重置状态（切换枪械触发）
    void OnEnable()
    {
        ResetGunState();
    }

    // 统一重置枪械状态的方法
    void ResetGunState()
    {
        timeBeforeShooting = 1 / fireRate;
        shooting = false;
        // 可选：切换枪械时强制重置为默认射击模式，按需启用
        // currentFiringMode = firingMode.auto;
    }

    void Update()
    {
        if(Input.GetKeyDown(KeyCode.V))
        {
            firingModeID++;
            firingModeID = firingModeID > 3 ? 1 : firingModeID;
            currentFiringMode = (firingMode)firingModeID;
            Debug.Log(firingModeID);
        }

        switch(currentFiringMode)
        {
            case firingMode.auto:
                AutoFire();
                break;

            case firingMode.semi:
                SemiFire();
                break;

            case firingMode.burst:
                BurstFire();
                break;
        }
    }

    void AutoFire()
    {
        if (Input.GetMouseButton(0))
        {
            if (timeBeforeShooting <= 0f)
            {
                FireShot();
                timeBeforeShooting = 1 / fireRate;
            }
            else
            {
                timeBeforeShooting -= Time.deltaTime;
            }
        }
        else
        {
            // 可选：若不想松开鼠标后立即重置延迟，可注释此行
            timeBeforeShooting = 0f;
        }
    }

    void SemiFire()
    {
        if (Input.GetMouseButtonDown(0) && timeBeforeShooting <= 0f)
        {
            FireShot();
        }
    }

    void BurstFire()
    {
        if (Input.GetMouseButtonDown(0) && !shooting)
        {
            StartCoroutine(BurstShots(3));
            shooting = true;
        }
    }

    IEnumerator BurstShots(int timesToShoot)
    {
        for (int i = 1; i <= timesToShoot; i++)
        {
            FireShot();
            yield return new WaitForSeconds(0.5f / fireRate);
        }
        shooting = false;
    }

    // 通用开火逻辑，避免代码重复
    void FireShot()
    {
        muzzleFlash.Play();
        bang.Play();

        Ray gunRay = new Ray(playerView.transform.position, playerView.transform.forward);
        if (Physics.Raycast(gunRay, out RaycastHit hitInfo, 100f, enemyLayer))
        {
            if (hitInfo.collider.gameObject.TryGetComponent(out enemy1 enemyHit))
            {
                enemyHit.SubtractHealth(damage);
                Debug.Log(enemyHit.health);
            }
        }
    }
}

额外说明 #

• 提取FireShot方法统一处理开火逻辑，减少冗余代码，后续修改开火效果只需维护这一处。
• ResetGunState方法确保首次加载和每次切换枪械时，状态都能回到初始值，彻底避免自动开火的触发条件。
• 若不需要松开鼠标后立即重置auto模式的延迟，可注释AutoFire方法中timeBeforeShooting = 0f这一行，玩家松开鼠标后再次按住时，会继续剩余的延迟计时。

内容的提问来源于stack exchange，提问作者Domini