统一网络 |权限问题 - 玩家可以移动但不能独立射击

Question

我正在使用 Unet 制作多人游戏。目前，玩家可以独立移动（即，如果用户一按 d，则角色一将向右移动，如果用户二按 d，则角色二将向右移动） - 但是，两个玩家都“穿过”主机。

我的移动代码在 PlayerUnit 脚本中。它在顶部有一个!isLocalPlayer 检查，在Update() 方法中。此检查在脚本检测到用户输入（即 w、s、mouse1 等）之前执行。

当按下正确的键时，一切正常；执行了适当的操作。 例如：

if (Input.GetKeyDown(up))
    Jump();

拍摄的工作方式略有不同，因为它涉及不同的脚本。

当 mouse1 被按下时，PlayerUnit 中的Shoot() 方法被调用。这反过来又在 PlayerUnit 脚本中使用 Command 在单独的 WeaponMaster 脚本中调用 ClientRpc，该脚本调用 Fire() 协程。

在 PlayerUnit 中：

private void Shoot ()
{   
    //WeaponMaster.wM.Shoot();
    CmdShoot();

}

[Command]
void CmdShoot ()
{
    WeaponMaster.wM.RpcShoot();
}

在武器大师中：

[ClientRpc]
public void RpcShoot ()
{
    if (ammo != 0)
    {
        if (isAutomatic == true)
        {
            keepFiring = true;
            StartCoroutine(Fire());
        }

        if (isAutomatic == false && noQuickFire == true)
           StartCoroutine(Fire());
    }
}

WeaponMaster 也有一个 !isLocalPlayer 检查，同样是在 Update() 方法中。

调试时，我知道playerTwo的（客户端）Shoot()方法正在被调用，但是playerOne的RpcShoot正在被执行。当playerOne 射击时一切正常 - 因此两个玩家都“穿过”主机射击的问题。

所有脚本都在同一个对象上，带有一个NetworkIdentity 组件。

我确信问题只是缺少权限检查 - 但我尝试在各种方法中添加 !isLocalPlayer 测试无济于事。

Answer 1

正如我所说的，我的猜测是 WeaponMaster.wM 中的单例模式在这里不是一个好方法，因为你得到了错误的引用，并且因为无论谁调用 CmdShoot，一切都通过服务器可能总是得到相同的引用。

---

您应该存储每个玩家自己的 WeaponMaster 参考并像在PlayerUnit 中一样浏览它

// Already reference this via the inspector (drag&drop)
[SerializeField] private WeaponMaster weaponMaster;

// or as fallback get it on runtime
private void Awake()
{
    if(!weaponMaster) weaponMaster = GetComponent<WeaponMaster>();
}

public void Shoot ()
{  
    if(!isLocalPlayer) return;

    CmdShoot();
}

[Command]
private void CmdShoot ()
{
    RpcShoot();
}

// Personally in general I would leave any RPC
// as close to the method calling it
// as possible for better maintainability
[ClientRpc]
private void RpcShoot ()
{
    weaponMaster.Shoot();
}

相应地在WeaponMaster

public void Shoot()
{
    if (ammo == 0) return;

    if (isAutomatic)
    {
        keepFiring = true;
        StartCoroutine(Fire());
    }
    else if (!isAutomatic && noQuickFire)
        StartCoroutine(Fire());
}