Third Party Best way to keep a weapon drop consistent throughout the room? (Photon Multiplayer)

I'm having an issue right now in my game where sometimes when one player picks up a gun off the ground, it's not removing the gun off the ground allowing the player to essentially duplicate the item.

In my WeaponPickup script I have the following:

Code (CSharp):

1. void Update()
2. {
3.   if(weaponToPickup)
4.   {
5.      SwitchWeapon(weaponToPickup);
6.      weaponToPickup.GetPhotonView().RPC("Remove", PhotonTargets.All); //removes the gun on the ground
7.      GameObject dropWeapon = PhotonNetwork.Instantiate(currentWeapon, transform.position, transform.rotation, 0); //throw your current weapon down
8.      dropWeapon.name = currentWeapon.name;
9.   }
10. }

And then the RPC is:

Code (CSharp):

1.  
2. [RPC]
3. void Remove()
4. {
5.      if(PhotonNetwork.isMasterClient)
6.      {
7.           PhotonNetwork.Destroy(gameObject);
8.      }
9. }
10.  

Is this a terrible way to do this? It seems to work for the most part, but after a while once players start picking up each other's weapons and dropping them it gets a bit confusing and starts doing the whole duplicating thing.

 

It's not a terrible way but as you see it just doesn't work reliably either.
It might not be obvious but RPC "Remove" gets executed immediately on the client that sends it to all. And: It picks up the weapon, no matter if anyone else is picking the weapon up at the same time (which means the other's "Remove" will arrive any moment but didn't yet).

You better send "Pickup" to AllViaServer. Then you can remove the item from the ground and pick it up on the client who sent the "Pickup" first (check "did i send it? if so, pickup" before you remove the item).

You can see an implementation of it in the PUN package's "Pickup, Teams & Score Demo". Update PUN if you don't have it.

 

Thanks for your help! That does make a lot more sense.

 

whatched this