[Suggestion] Object Visibility

To set custom visibility in UNet you must create a class that manipulates the "observers" list for the NetworkIdentity Component of a GameObject. But the problems is that it's oriented towards the non-players instead of the actual players e.g:

If you have 30 enemies on the map that must be synchronized to 2 players, then you will have to call RebuildObservers for all of these 30 enemies and during the OnRebuildObservers event that occurs on each enemy you have to add those 2 players NetworkConnection to its "observers" list.

The way it should be done is to have a list of relevant NetworkIdentity inside of the NetworkIdentity class instead of the current "observers" list. So we would only have to maintain the lists for those 2 players.

The current implementation:

Code (csharp):

1.  
2.     public sealed class NetworkIdentity : MonoBehaviour
3.     {
4.         ...STUFF...
5.         public ReadOnlyCollection<NetworkConnection> observers { get; }
6.         ...STUFF...
7.  
8.         public void RebuildObservers(bool initialize);
9.     }
10.  

So this is like:

Enemy1 observers list
------Player1
------Player2
Enemy2 observers list
------Player1
------Player2
Enemy3 observers list
------Player1
------Player2
...
...
...
Enemy30 observers list
------Player1
------Player2
Player1 observers list
------Player2
Player2 observers list
------Player1

How it should be implemented:

Code (csharp):

1.  
2.     public sealed class NetworkIdentity : MonoBehaviour
3.     {
4.         ...STUFF...
5.         public ReadOnlyCollection<NetworkIdentity> relevantNetworkIdenties { get; }
6.         ...STUFF...
7.  
8.         public void RebuildRelevantNetworkIdenties(bool initialize);
9.     }
10.  

So this would be like:

Player1 relevantNetworkIdenties list
------Enemy1
------Enemy2
------Enemy3
...
...
...
------Enemy30
------Player2

Player2 relevantNetworkIdenties list
------Enemy1
------Enemy2
------Enemy3
...
...
...
------Enemy30
------Player1

 

You can iterate over each player's relevantNetworkIdentities (which store all NetworkIdentity that is relevant to this player) to check if the relevant object's state has changed and send the state update message to this player.