Skip to content

Commit

Permalink
Kicking player out if they have a duplicate name
Browse files Browse the repository at this point in the history
Prob wont happen in demo but dont wanna leave an error case in that would break the entire game
  • Loading branch information
mvorsteg committed May 6, 2023
1 parent 9d08604 commit 72c632e
Show file tree
Hide file tree
Showing 3 changed files with 43 additions and 2 deletions.
10 changes: 10 additions & 0 deletions Assets/Code/Logic/HostEngine.cs
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,16 @@ public bool AddPlayer(int playerID, string name, out CharacterType assignedChara
assignedCharacter = CharacterType.Mustard;
return false;
}
// check if name already taken
foreach (PlayerState player in players.Values)
{
if (player.playerName == name)
{
assignedCharacter = CharacterType.Mustard;
return false;
}
}

// find available character to assign
Dictionary<CharacterType, bool> availableChars = new Dictionary<CharacterType, bool>();
foreach (CharacterType character in Enum.GetValues(typeof(CharacterType)))
Expand Down
24 changes: 24 additions & 0 deletions Assets/Scenes/SampleScene.unity
Original file line number Diff line number Diff line change
Expand Up @@ -45974,6 +45974,30 @@ MonoBehaviour:
m_StringArgument:
m_BoolArgument: 0
m_CallState: 2
- m_Target: {fileID: 1616721088}
m_TargetAssemblyTypeName: UnityEngine.GameObject, UnityEngine
m_MethodName: SetActive
m_Mode: 6
m_Arguments:
m_ObjectArgument: {fileID: 0}
m_ObjectArgumentAssemblyTypeName: UnityEngine.Object, UnityEngine
m_IntArgument: 0
m_FloatArgument: 0
m_StringArgument:
m_BoolArgument: 0
m_CallState: 2
- m_Target: {fileID: 679888795}
m_TargetAssemblyTypeName: GuestEngine, Assembly-CSharp
m_MethodName: SetReadyForReveal
m_Mode: 6
m_Arguments:
m_ObjectArgument: {fileID: 0}
m_ObjectArgumentAssemblyTypeName: UnityEngine.Object, UnityEngine
m_IntArgument: 0
m_FloatArgument: 0
m_StringArgument:
m_BoolArgument: 1
m_CallState: 2
--- !u!114 &1207409703
MonoBehaviour:
m_ObjectHideFlags: 0
Expand Down
11 changes: 9 additions & 2 deletions ProjectSettings/ProjectSettings.asset
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ PlayerSettings:
16:10: 1
16:9: 1
Others: 1
bundleVersion: 0.3.1
bundleVersion: 0.3.2
preloadedAssets: []
metroInputSource: 0
wsaTransparentSwapchain: 0
Expand Down Expand Up @@ -268,7 +268,14 @@ PlayerSettings:
AndroidMinifyDebug: 0
AndroidValidateAppBundleSize: 1
AndroidAppBundleSizeToValidate: 150
m_BuildTargetIcons: []
m_BuildTargetIcons:
- m_BuildTarget:
m_Icons:
- serializedVersion: 2
m_Icon: {fileID: 2800000, guid: 5053824c923d8493c8614cc21914823e, type: 3}
m_Width: 128
m_Height: 128
m_Kind: 0
m_BuildTargetPlatformIcons: []
m_BuildTargetBatching:
- m_BuildTarget: Standalone
Expand Down

0 comments on commit 72c632e

Please sign in to comment.