[MISC] tried to send the message to the current lobby, didn't work.

Still in progress not functional because of a null exception when receiving the broadcast message.

Client/App.xaml.cs

@@ -1,4 +1,5 @@
-using System;
+using Client.Views;
+using System;
 using System.Collections.Generic;
 using System.Configuration;
 using System.Data;
@@ -17,9 +18,9 @@ namespace Client
         protected override void OnStartup(StartupEventArgs e)
         {
             base.OnStartup(e);
-            MainWindow startWindow = new MainWindow();
-            ViewModel VM = new ViewModel();
-            startWindow.DataContext = VM;
+            LoginScreen startWindow = new LoginScreen();
+            //ViewModel VM = new ViewModel();
+            //startWindow.DataContext = VM;
             startWindow.Show();
         }
 

Client/Client.cs

@@ -16,11 +16,12 @@ namespace Client
         private int totalBufferReceived = 0;
         public int Port = 5555;
         public bool Connected = false;
-        //TODO send login packet to server with ClientServerUtil.createpayload(0x01,dynamic json with username)
-        public string Username { get; }
+        private string username;
+        public Callback OnSuccessfullConnect;
 
-        public Client()
+        public Client(string username)
         {
+            this.username = username;
             this.tcpClient = new TcpClient();
             Debug.WriteLine("Starting connect to server");
             tcpClient.BeginConnect("localhost", Port, new AsyncCallback(OnConnect), null);
@@ -31,6 +32,8 @@ namespace Client
             Debug.Write("finished connecting to server");
             this.tcpClient.EndConnect(ar);
             this.stream = tcpClient.GetStream();
+            OnSuccessfullConnect?.Invoke();
+            SendMessage(JSONConvert.ConstructUsernameMessage(username));
             this.stream.BeginRead(buffer, 0, buffer.Length, new AsyncCallback(OnReadComplete),null);
         }
 
@@ -62,35 +65,38 @@ namespace Client
             }
 
             stream.BeginRead(buffer, 0, buffer.Length, new AsyncCallback(OnReadComplete), null);
-
         }
 
         private void handleData(byte[] message)
         {
-            byte id = message[0];
+            byte id = message[4];
             byte[] payload = new byte[message.Length - 1];
             Array.Copy(message, 1, payload, 0, message.Length - 1);
             switch (id)
             {
-                case 0x01:
+                case JSONConvert.LOGIN:
                     // json log in username data
                     break;
-                case 0x02:
+                case JSONConvert.MESSAGE:
                     // json message data
                     (string, string) combo = JSONConvert.GetUsernameAndMessage(payload);
                     string textUsername = combo.Item1;
                     string textMsg = combo.Item2;
-                    //TODO display username and message in chat window
 
+                    //TODO display username and message in chat window
+                    Debug.WriteLine("[CLIENT] INCOMING MESSAGE!");
+                    Debug.WriteLine("[CLIENT] User name: {0}\t User message: {1}", textUsername, textMsg);
                     break;
 
-                case 0x03:
+                case JSONConvert.LOBBY:
                     // lobby data
                     //TODO fill lobby with the data received
                     break;
-                case 0x04:
+
+                case JSONConvert.CANVAS:
                     // canvas data
                     break;
+
                 default:
                     Debug.WriteLine("[CLIENT] Received weird identifier: " + id);
                     break;
@@ -100,11 +106,13 @@ namespace Client
 
         public void SendMessage(byte[] message)
         {
+            Debug.WriteLine("[CLIENT] sending message " + Encoding.ASCII.GetString(message));
             stream.BeginWrite(message, 0, message.Length, new AsyncCallback(OnWriteComplete), null);
         }
 
         private void OnWriteComplete(IAsyncResult ar)
         {
+            Debug.WriteLine("[CLIENT] finished writing");
             stream.EndWrite(ar);
         }
     }

Client/ClientData.cs

@@ -0,0 +1,72 @@
+using SharedClientServer;
+using System;
+using System.Collections.Generic;
+using System.Data;
+using System.Text;
+
+namespace Client
+{
+    class ClientData
+    {
+        private static ClientData _instance;
+        private static readonly object padlock = new object();
+
+        public static ClientData Instance
+        {
+            get
+            {
+                lock (padlock)
+                {
+                    if (_instance == null)
+                    {
+                        _instance = new ClientData();
+                    }
+                    return _instance;
+                }
+            }
+        }
+
+
+        private User _user;
+        private Client _client;
+        private Lobby _lobby;
+        private string _message;
+
+        private ClientData()
+        {
+
+        }
+
+
+        public User User
+        {
+            get { return _user; }
+            set { _user = value; }
+        }
+
+        public Client Client
+        {
+            get { return _client; }
+            set { _client = value; }
+        }
+
+        public Lobby Lobby
+        {
+            get { return _lobby; }
+            set { _lobby = value; }
+        }
+
+        public String Message
+        {
+            get
+            {
+                return _message;
+            }
+            set
+            {
+                _message = value;
+            }
+        }
+
+    }
+}

Client/Models/Model.cs

@@ -11,6 +11,7 @@ namespace Client
 
         private int _numbers;
         private bool _status;
+        private bool _canStartGame;
 
         //Test code 
         public int Numbers
@@ -37,11 +38,18 @@ namespace Client
             }
         }
 
+        public bool CanStartGame
+        {
+            get { return _canStartGame; }
+            set { _canStartGame = value; }
+        }
+
 
         public Model()
         {
             _status = false;
             _numbers = 0;
+            _canStartGame = true;
         }
 
     }

Client/ViewModels/ViewModel.cs

@@ -5,6 +5,10 @@ using System.ComponentModel;
 using System.Text;
 using System.Windows.Input;
 using SharedClientServer;
+using System.Diagnostics;
+using System.Windows;
+using System.Collections.ObjectModel;
+using Client.Views;
 
 namespace Client
 {
@@ -12,19 +16,38 @@ namespace Client
     {
         public event PropertyChangedEventHandler PropertyChanged;
 
+        public ICommand OnHostButtonClick { get; set; }
+        public ICommand JoinSelectedLobby { get; set; }
+
+        public Lobby SelectedLobby { get; set; }
+
         public ViewModel()
         {
             _model = new Model();
-            ButtonCommand = new RelayCommand(() =>
-            {
-                Client client = new Client();
-            });
-
-            _lobbies = new List<Lobby>();
+            _lobbies = new ObservableCollection<Lobby>();
 
             _lobbies.Add(new Lobby(50, 3, 8));
             _lobbies.Add(new Lobby(69, 1, 9));
             _lobbies.Add(new Lobby(420, 7, 7));
+
+            OnHostButtonClick = new RelayCommand(() => 
+            {
+                Debug.WriteLine("Host button clicked");
+            });
+
+            JoinSelectedLobby = new RelayCommand(startGameInLobby, true);
+        }
+
+        private void startGameInLobby()
+        {
+            if (SelectedLobby != null)
+            {
+                ClientData.Instance.Lobby = SelectedLobby;
+                
+                _model.CanStartGame = false;
+                GameWindow window = new GameWindow();
+                window.Show();
+            }
         }
 
         private void ClickCheck()
@@ -35,17 +58,12 @@ namespace Client
             _model.Numbers = _model.Numbers + 5;
         }
 
-        public ICommand ButtonCommand { get; set; }
-
 
         private Model _model;
         public Model Model
         {
             get
             {
-                if (_model == null)
-                    _model = new Model();
-
                 return _model;
             }
 
@@ -55,8 +73,8 @@ namespace Client
             }
         }
 
-        private List<Lobby> _lobbies;
-        public List<Lobby> Lobbies
+        private ObservableCollection<Lobby> _lobbies;
+        public ObservableCollection<Lobby> Lobbies
         {
             get { return _lobbies; }
             set { _lobbies = value; }

Client/ViewModels/ViewModelGame.cs

@@ -1,10 +1,71 @@
-using System.Collections.ObjectModel;
+using GalaSoft.MvvmLight.Command;
+using SharedClientServer;
+using System.Collections.ObjectModel;
 using System.ComponentModel;
+using System.Diagnostics;
+using System.Windows.Input;
 
 namespace Client.ViewModels
 {
     class ViewModelGame : INotifyPropertyChanged
     {
+        ClientData data = ClientData.Instance;
         public event PropertyChangedEventHandler PropertyChanged;
+        public ObservableCollection<string> Messages { get; } = new ObservableCollection<string>();
+
+        private dynamic _payload;
+
+        private string _username;
+
+        private string _message;
+        public string Message
+        {
+            get
+            {
+                return _message;
+            }
+            set
+            {
+                _message = value;
+            }
+        }
+        public ICommand OnKeyDown { get; set; }
+
+        public ViewModelGame()
+        {
+            if (_payload == null)
+            {
+                _message = "";
+               
+            }
+            else
+            {
+                //_message = data.Message;
+                //_username = data.User.Username;
+                //Messages.Add($"{data.User.Username}: {Message}");
+            }
+            OnKeyDown = new RelayCommand(ChatBox_KeyDown);
+        }
+
+        private void ChatBox_KeyDown()
+        {
+            //if enter then clear textbox and send message.
+            if (Message != string.Empty) AddMessage(Message);
+            Message = string.Empty;
+        }
+
+        internal void AddMessage(string message)
+        {
+            Messages.Add($"{data.User.Username}: {message}");
+
+            _payload = new
+            {
+                username = data.User.Username,
+                message = message
+            };
+
+            //Broadcast the message after adding it to the list!
+            data.Client.SendMessage(JSONConvert.GetMessageToSend(JSONConvert.MESSAGE, _payload));
+        }
     }
 }

Client/Views/GameWindow.xaml

@@ -48,8 +48,13 @@
         </Border>
 
         <Grid Grid.Column="2" Grid.Row="1">
-            <TextBox Name="SentMessage" IsReadOnly="True"/>
-            <TextBox x:Name="ChatBox" Keyboard.KeyDown="ChatBox_KeyDown" Width="200" Height="50" ToolTip="Message goes here" ToolTipService.IsEnabled="True" VerticalAlignment="Bottom" HorizontalAlignment="Left"/>
+            <ListBox Name ="TextBox" ItemsSource="{Binding Path=Messages}" Margin="0,0,0,69"/>
+
+            <TextBox Name="ChatBox" Text="{Binding Message, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" Margin="0,465,0,0">
+                <TextBox.InputBindings>
+                    <KeyBinding Key="Return" Command="{Binding OnKeyDown}"/>
+                </TextBox.InputBindings>
+            </TextBox>
         </Grid>
 
     </Grid>

Client/Views/GameWindow.xaml.cs

@@ -18,7 +18,7 @@ namespace Client.Views
     /// </summary>
     public partial class GameWindow : Window
     {
-
+       
         public GameWindow()
         {
             DataContext = new ViewModelGame();
@@ -82,7 +82,6 @@ namespace Client.Views
             //    var deepCopy = System.Windows.Markup.XamlReader.Parse(xaml) as UIElement;
             //    TEST.Children.Add(deepCopy);
             //}
-
         }
 
         private void ClrPcker_Background_SelectedColorChanged_1(object sender, RoutedPropertyChangedEventArgs<Color?> e)
@@ -95,23 +94,14 @@ namespace Client.Views
             color = colorSelected;
         }
 
-        private void ChatBox_KeyDown(object sender, KeyEventArgs e)
-        {
-            //if enter then clear textbox and send message.
-            if (e.Key.Equals(Key.Enter))
-            {
-                WriteToChat(ChatBox.Text);
-                ChatBox.Clear();
-            }
-        }
-
-        /*
-         * Writes the current client's message to the chatbox.
-         */
-        private void WriteToChat(string message)
-        {
-            string user = "Monkey"; 
-            SentMessage.AppendText($"{user}: {message}\n");
-        }
+        ///*
+        // * Writes the current client's message to the chatbox.
+        // */
+        //private void WriteToChat(string message)
+        //{
+        //    string user = data.User.Username;
+        //    SentMessage.AppendText($"{user}: {message}\n");
+        //    data.User.Message = message;
+        //}
     }
 }

Client/Views/LoginScreen.xaml

@@ -0,0 +1,29 @@
+<Window x:Class="Client.Views.LoginScreen"
+        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
+        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
+        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
+        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
+        xmlns:local="clr-namespace:Client.Views"
+        mc:Ignorable="d"
+        Title="LoginScreen" Height="450" Width="800">
+    <Grid>
+        <Grid.RowDefinitions>
+            <RowDefinition Height="60"/>
+            <RowDefinition Height="50"/>
+            <RowDefinition Height="30"/>
+        </Grid.RowDefinitions>
+
+        <Grid.ColumnDefinitions>
+            <ColumnDefinition />
+            <ColumnDefinition />
+        </Grid.ColumnDefinitions>
+
+        <Label Grid.Row="0" FontSize="40" Content="Welcome to Scrubl.io"/>
+        <Label Grid.Row="1" FontSize="30" Content="Enter a username:"/>
+        <Label Grid.Row="2" FontSize="15" Content="(max amount of characters for the username is 10)"/>
+
+        <TextBox Name="usernameTextbox" Grid.Row="1" Grid.Column="1" MaxLength="10" FontSize="30" VerticalAlignment="Center" HorizontalAlignment="Left" Width="250"/>
+        <Button Content="ENTER" Grid.Column="1" Grid.Row="1" VerticalAlignment="Center" HorizontalAlignment="Right" Width="100" Height="40" Click="Button_EnterUsername"/>
+
+    </Grid>
+</Window>

Client/Views/LoginScreen.xaml.cs

@@ -0,0 +1,45 @@
+using SharedClientServer;
+using System;
+using System.Collections.Generic;
+using System.Text;
+using System.Windows;
+using System.Windows.Controls;
+using System.Windows.Data;
+using System.Windows.Documents;
+using System.Windows.Input;
+using System.Windows.Media;
+using System.Windows.Media.Imaging;
+using System.Windows.Shapes;
+
+namespace Client.Views
+{
+    /// <summary>
+    /// Interaction logic for LoginScreen.xaml
+    /// </summary>
+    public partial class LoginScreen : Window
+    {
+        ClientData data = ClientData.Instance;
+        public LoginScreen()
+        {
+            InitializeComponent();
+        }
+
+        private void Button_EnterUsername(object sender, RoutedEventArgs e)
+        {
+            User user = new User(usernameTextbox.Text);
+            Client client = new Client(user.Username);
+            client.OnSuccessfullConnect = () =>
+            {
+                // because we need to start the main window on a UI thread, we need to let the dispatcher handle it, which will execute the code on the ui thread
+                Application.Current.Dispatcher.Invoke(delegate {
+                    data.User = user;
+                    data.Client = client;
+                    MainWindow startWindow = new MainWindow();
+                    startWindow.Show();
+                    this.Close();
+                });
+            };
+
+        }
+    }
+}

Client/Views/MainWindow.xaml

@@ -12,7 +12,7 @@
             <RowDefinition/>
         </Grid.RowDefinitions>
 
-        
+
         <Grid.ColumnDefinitions>
             <ColumnDefinition />
             <ColumnDefinition/>
@@ -34,9 +34,8 @@
             <Label Grid.Row="0" Content="This client information:" FontSize="17"/>
 
             <Label Grid.Row="1" Grid.Column="0" Content="Your username:" FontSize="15" VerticalAlignment="Center"/>
-            <TextBox Name="usernameTextbox" Grid.Row="1" Grid.Column="1" MaxLength="10" FontSize="15" VerticalAlignment="Center"/>
 
-            <Label Grid.Row="2" Grid.Column="0" Content="Which color you want to be:" FontSize="15" VerticalAlignment="Center"/>
+            <Label Grid.Row="2" Grid.Column="0" Content="Select your color:" FontSize="15" VerticalAlignment="Center"/>
             <ComboBox Name="colorSelection" Grid.Row="2" Grid.Column="1" VerticalAlignment="Center" FontSize="15">
                 <ComboBoxItem Content="BLUE"/>
                 <ComboBoxItem Content="RED"/>
@@ -48,11 +47,12 @@
             </ComboBox>
 
             <Label Grid.Row="3" Name="testLabel" FontSize="15" VerticalAlignment="Center"/>
+            <Label Name="usernameLabel" Content="place username here" Grid.Column="1" HorizontalAlignment="Center" Margin="0,12,0,0" VerticalAlignment="Top" Grid.Row="1"/>
 
         </Grid>
 
 
-        <ListView Name="LobbyList" Grid.Row="1" Grid.Column="0" Margin="10, 10, 10, 10" ItemsSource="{Binding Path=Lobbies}">
+        <ListView Name="LobbyList" Grid.Row="1" Grid.Column="0" SelectedItem="{Binding SelectedLobby}" Margin="10, 10, 10, 10" ItemsSource="{Binding Path=Lobbies}">
             <ListView.View>
                 <GridView x:Name="grdList">
                     <GridViewColumn Header="Lobby ID" DisplayMemberBinding="{Binding ID}" Width="70"/>
@@ -67,8 +67,8 @@
                 <RowDefinition Height="50"/>
                 <RowDefinition Height="50"/>
             </Grid.RowDefinitions>
-            <Button Name="joinButton" Grid.Row="0" Content="join a selected lobby" Click="Button_Click" Width="200" Height="40" HorizontalAlignment="Left" Margin="10, 0, 0, 0"/>
-            <Button Name="hostButton" Grid.Row="1" Content="host a new lobby" Command="{Binding ...}" Width="200" Height="40" HorizontalAlignment="left" Margin="10, 0, 0, 0"/>
+            <Button Name="joinButton" Grid.Row="0" Content="join a selected lobby" Command="{Binding JoinSelectedLobby}" IsEnabled="{Binding Model.CanStartGame}" Width="200" Height="40" HorizontalAlignment="Left" Margin="10, 0, 0, 0"/>
+            <Button Name="hostButton" Grid.Row="1" Content="host a new lobby" Command="{Binding OnHostButtonClick}" IsEnabled="{Binding Model.CanStartGame}" Width="200" Height="40" HorizontalAlignment="left" Margin="10, 0, 0, 0"/>
         </Grid>
 
     </Grid>

Client/Views/MainWindow.xaml.cs

@@ -21,25 +21,28 @@ namespace Client
     /// </summary>
     public partial class MainWindow : Window
     {
+        ClientData data = ClientData.Instance;
         public MainWindow()
         {
+            this.DataContext = new ViewModel();
             InitializeComponent();
+
+            usernameLabel.Content = data.User.Username;
         }
 
         private void Button_Click(object sender, RoutedEventArgs e)
         {
-
             Lobby lobbySelected = LobbyList.SelectedItem as Lobby;
             if(lobbySelected != null)
             {
                 testLabel.Content = lobbySelected.ID;
-                usernameTextbox.IsEnabled = false;
                 colorSelection.IsEnabled = false;
                 joinButton.IsEnabled = false;
                 hostButton.IsEnabled = false;
 
                 GameWindow window = new GameWindow();
                 window.Show();
+                Close();
             }
         }
     }

Server/Models/ServerClient.cs

@@ -1,4 +1,5 @@
 
+using Newtonsoft.Json.Linq;
 using SharedClientServer;
 using System;
 using System.Collections.Generic;
@@ -10,12 +11,13 @@ namespace Server.Models
 {
     class ServerClient : ObservableObject
     {
-        public string Username { get; set; }
         private TcpClient tcpClient;
         private NetworkStream stream;
         private byte[] buffer = new byte[1024];
         private byte[] totalBuffer = new byte[1024];
         private int totalBufferReceived = 0;
+        public User User { get; set; }
+        private ServerCommunication serverCom = ServerCommunication.INSTANCE;
         
 
         /// <summary>
@@ -24,8 +26,10 @@ namespace Server.Models
         /// <param name="client">the TcpClient object to use</param>
         public ServerClient(TcpClient client)
         {
+            Debug.WriteLine("[SERVERCLIENT] making new instance and starting");
             tcpClient = client;
             stream = tcpClient.GetStream();
+            Debug.WriteLine("[SERVERCLIENT] starting read");
             stream.BeginRead(buffer, 0, buffer.Length, new AsyncCallback(OnRead), null);
         }
 
@@ -35,6 +39,10 @@ namespace Server.Models
         /// <param name="ar">the async result status</param>
         private void OnRead(IAsyncResult ar)
         {
+            if (ar == null || (!ar.IsCompleted) || (!this.stream.CanRead) || !this.tcpClient.Client.Connected)
+                return;
+
+
             int bytesReceived = this.stream.EndRead(ar);
 
             if (totalBufferReceived + bytesReceived > 1024)
@@ -84,36 +92,43 @@ namespace Server.Models
         /// <param name="message">the incoming message</param>
         private void HandleIncomingMessage(byte[] message)
         {
-            Debug.WriteLine($"Got message from {Username} : {message}");
-            byte id = message[0];
-            byte[] payload = new byte[message.Length - 1];
-            Array.Copy(message,1,payload,0,message.Length-1);
+            Debug.WriteLine($"Got message : {Encoding.ASCII.GetString(message)}");
+            byte id = message[4];
+            byte[] payload = new byte[message.Length - 5];
+            Array.Copy(message,5,payload,0,message.Length-5);
+            Debug.WriteLine("[SERVERCLIENT] GOT STRING" + Encoding.ASCII.GetString(payload));
             switch(id)
             {
                 
-                case 0x01:
+                case JSONConvert.LOGIN:
                     // json log in username data
-                    string uName = JSONConvert.GetUsernameLogin(message);
+                    string uName = JSONConvert.GetUsernameLogin(payload);
+                    
                     if (uName != null)
                     {
-                        Username = uName;
-                        Debug.WriteLine("[SERVERCLIENT] set username to " + Username);
+                        User = new User(uName);
+                        User.Username = uName;
+                        Debug.WriteLine("[SERVERCLIENT] set username to " + uName);
                         
                     }
                     break;
-                case 0x02:
+                case JSONConvert.MESSAGE:
                     // json message data
                     (string, string) combo = JSONConvert.GetUsernameAndMessage(payload);
                     string textUsername = combo.Item1;
                     string textMsg = combo.Item2;
 
+                    Debug.WriteLine("[SERVERCLIENT] User name: {0}\t User message: {1}", textUsername, textMsg);
+
                     // todo handle sending to all except this user the username and message to display in chat
+                    serverCom.SendToLobby(User.Lobby,payload);
+                    Debug.WriteLine("Payload has been sent!");
                     break;
 
-                case 0x03:
+                case JSONConvert.LOBBY:
                     // lobby data
                     break;
-                case 0x04:
+                case JSONConvert.CANVAS:
                     // canvas data
                     // todo send canvas data to all other serverclients in lobby
                     break;

Server/Models/ServerCommunication.cs

@@ -1,5 +1,4 @@
-
-using Client;
+using Client;
 using SharedClientServer;
 using System;
 using System.Collections.Generic;
@@ -18,6 +17,7 @@ namespace Server.Models
         public List<Lobby> lobbies;
         private Dictionary<Lobby, List<ServerClient>> serverClientsInlobbies;
         public Action newClientAction;
+        
 
         /// <summary>
         /// use a padlock object to make sure the singleton is thread-safe
@@ -98,7 +98,7 @@ namespace Server.Models
         {
             foreach (ServerClient sc in serverClients)
             {
-                if (sc.Username != username) sc.sendMessage(message);
+                if (sc.User.Username != username) sc.sendMessage(message);
             }
         }
 
@@ -117,14 +117,14 @@ namespace Server.Models
             }
         }
 
-        public void AddToLobby(Lobby lobby, string username)
+        public void AddToLobby(Lobby lobby, User user)
         {
             foreach (Lobby l in lobbies)
             {
                 if (l == lobby)
                 {
                     bool succ;
-                    l.AddUsername(username, out succ);
+                    l.AddUser(user, out succ);
                     if (!succ)
                     {
                         // TODO send lobby full message
@@ -132,7 +132,7 @@ namespace Server.Models
                     {
                         foreach(ServerClient sc in serverClients)
                         {
-                            if (sc.Username == username)
+                            if (sc.User.Username == user.Username)
                             {
                                 serverClientsInlobbies[l].Add(sc);
                                 break;

SharedClientServer/ClientServerUtil.cs

@@ -6,6 +6,7 @@ using System.Text;
 
 namespace SharedClientServer
 {
+    public delegate void Callback();
     class ClientServerUtil
     {
         // creates a message array to send to the server or to clients
@@ -17,6 +18,5 @@ namespace SharedClientServer
             Array.Copy(stringAsBytes, 0, res, 1, stringAsBytes.Length);
             return res;
         }
-
     }
 }

SharedClientServer/JSONConvert.cs

@@ -1,24 +1,73 @@
-using Newtonsoft.Json;
+using Client;
+using Newtonsoft.Json;
 using System;
 using System.Collections.Generic;
+using System.Diagnostics;
 using System.Text;
 
 namespace SharedClientServer
 {
     class JSONConvert
     {
+        public const byte LOGIN = 0x01;
+        public const byte MESSAGE = 0x02;
+        public const byte LOBBY = 0x03;
+        public const byte CANVAS = 0x04;
+
+        enum LobbyIdentifier
+        {
+            HOST,
+            ADD,
+            LEAVE,
+            REQUEST
+        }
         public static (string,string) GetUsernameAndMessage(byte[] json)
         {
             string msg = Encoding.ASCII.GetString(json);
             dynamic payload = JsonConvert.DeserializeObject(msg);
+            
             return (payload.username, payload.message);
         }
 
         public static string GetUsernameLogin(byte[] json)
         {
-            string msg = Encoding.ASCII.GetString(json);
-            dynamic payload = JsonConvert.DeserializeObject(msg);
+            dynamic payload = JsonConvert.DeserializeObject(Encoding.ASCII.GetString(json));
             return payload.username;
         }
+
+        public static byte[] ConstructUsernameMessage(string uName)
+        {
+            return GetMessageToSend(LOGIN, new
+            {
+                username = uName
+            });
+        }
+
+        public static byte[] ConstructLobbyDataMessage(Lobby lobby)
+        {
+            return null;
+        }
+
+
+        /// <summary>
+        /// constructs a message that can be sent to the clients or server
+        /// </summary>
+        /// <param name="identifier">the identifier for what kind of message it is</param>
+        /// <param name="payload">the json payload</param>
+        /// <returns>a byte array containing a message that can be sent to clients or server</returns>
+        public static byte[] GetMessageToSend(byte identifier, dynamic payload)
+        {
+            // convert the dynamic to bytes
+            byte[] payloadBytes = Encoding.ASCII.GetBytes(JsonConvert.SerializeObject(payload));
+            // make the array that holds the message and copy the payload into it with the first spot containing the identifier
+            byte[] res = new byte[payloadBytes.Length + 5];
+            // put the payload in the res array
+            Array.Copy(payloadBytes, 0, res, 5, payloadBytes.Length);
+            // put the identifier at the start of the payload part
+            res[4] = identifier;
+            // put the length of the payload at the start of the res array
+            Array.Copy(BitConverter.GetBytes(payloadBytes.Length+5),0,res,0,4);
+            return res;
+        }
     }
 }

SharedClientServer/Lobby.cs

@@ -1,56 +1,79 @@
-using System;
-using System.Collections.Generic;
-using System.ComponentModel;
-using System.Text;
-
-namespace Client
-{
-    class Lobby : INotifyPropertyChanged
-    {
-        public event PropertyChangedEventHandler PropertyChanged;
-
-
-        private int _id;
-        private int _playersIn;
-        private int _maxPlayers;
-        private List<string> _usernames;
-
-        public void AddUsername(string username, out bool success)
-        {
-            success = false;
-            if (_usernames.Count < _maxPlayers)
-            {
-                _usernames.Add(username);
-                success = true;
-            }
-        }
-
-        public Lobby(int id, int playersIn, int maxPlayers)
-        {
-            _id = id;
-            _playersIn = playersIn;
-            _maxPlayers = maxPlayers;
-            _usernames = new List<string>();
-        }
-
-        public int ID
-        {
-            get { return _id; }
-            set { _id = value; }
-        }
-
-        public int PlayersIn
-        {
-            get { return _playersIn; }
-            set { _playersIn = value; }
-        }
-
-        public int MaxPlayers
-        {
-            get { return _maxPlayers; }
-            set { _maxPlayers = value; }
-        }
-
-
-    }
-}
+using SharedClientServer;
+using System;
+using System.Collections.Generic;
+using System.ComponentModel;
+using System.Text;
+
+namespace Client
+{
+    class Lobby : INotifyPropertyChanged
+    {
+        public event PropertyChangedEventHandler PropertyChanged;
+
+
+        private int _id;
+        private int _playersIn;
+        private int _maxPlayers;
+        //private List<string> _usernames;
+        private List<User> _users;
+
+        //public void AddUsername(string username, out bool success)
+        //{
+        //    success = false;
+        //    if (_usernames.Count < _maxPlayers)
+        //    {
+        //        _usernames.Add(username);
+        //        success = true;
+        //    }
+        //}
+
+        public Lobby(int id, int playersIn, int maxPlayers)
+        {
+            _id = id;
+            _playersIn = playersIn;
+            _maxPlayers = maxPlayers;
+            //_usernames = new List<string>();
+            _users = new List<User>();
+        }
+
+        public void AddUser(string username, out bool succes)
+        {
+            succes = false;
+            if (_users.Count < _maxPlayers)
+            {
+                _users.Add(new User(username, 0, false));
+                succes = true;
+            }
+        }
+
+        public void AddUser(User user, out bool succes)
+        {
+            succes = false;
+            if (_users.Count < _maxPlayers)
+            {
+                _users.Add(user);
+                succes = true;
+            }
+        }
+
+        public int ID
+        {
+            get { return _id; }
+            set { _id = value; }
+        }
+
+        public int PlayersIn
+        {
+            get { return _playersIn; }
+            set { _playersIn = value; }
+        }
+
+        public int MaxPlayers
+        {
+            get { return _maxPlayers; }
+            set { _maxPlayers = value; }
+        }
+
+
+    }
+}

SharedClientServer/SharedClientServer.projitems

@@ -1,17 +1,18 @@
-<?xml version="1.0" encoding="utf-8"?>
-<Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
-  <PropertyGroup>
-    <MSBuildAllProjects>$(MSBuildAllProjects);$(MSBuildThisFileFullPath)</MSBuildAllProjects>
-    <HasSharedItems>true</HasSharedItems>
-    <SharedGUID>6d26f969-9cb1-414f-ac3e-7253d449ac5a</SharedGUID>
-  </PropertyGroup>
-  <PropertyGroup Label="Configuration">
-    <Import_RootNamespace>SharedClientServer</Import_RootNamespace>
-  </PropertyGroup>
-  <ItemGroup>
-    <Compile Include="$(MSBuildThisFileDirectory)ClientServerUtil.cs" />
-    <Compile Include="$(MSBuildThisFileDirectory)JSONConvert.cs" />
-    <Compile Include="$(MSBuildThisFileDirectory)Lobby.cs" />
-    <Compile Include="$(MSBuildThisFileDirectory)ObservableObject.cs" />
-  </ItemGroup>
+<?xml version="1.0" encoding="utf-8"?>
+<Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
+  <PropertyGroup>
+    <MSBuildAllProjects>$(MSBuildAllProjects);$(MSBuildThisFileFullPath)</MSBuildAllProjects>
+    <HasSharedItems>true</HasSharedItems>
+    <SharedGUID>6d26f969-9cb1-414f-ac3e-7253d449ac5a</SharedGUID>
+  </PropertyGroup>
+  <PropertyGroup Label="Configuration">
+    <Import_RootNamespace>SharedClientServer</Import_RootNamespace>
+  </PropertyGroup>
+  <ItemGroup>
+    <Compile Include="$(MSBuildThisFileDirectory)ClientServerUtil.cs" />
+    <Compile Include="$(MSBuildThisFileDirectory)JSONConvert.cs" />
+    <Compile Include="$(MSBuildThisFileDirectory)Lobby.cs" />
+    <Compile Include="$(MSBuildThisFileDirectory)ObservableObject.cs" />
+    <Compile Include="$(MSBuildThisFileDirectory)User.cs" />
+  </ItemGroup>
 </Project>

SharedClientServer/User.cs

@@ -0,0 +1,46 @@
+using System;
+using System.Collections.Generic;
+using System.Text;
+
+namespace SharedClientServer
+{
+    class User
+    {
+        private string _username;
+        private int _score;
+        private bool _host;
+        private string _message;
+
+        public User(string username, int score, bool host)
+        {
+            _username = username;
+            _score = score;
+            _host = host;
+        }
+
+        public User(string username)
+        {
+            _username = username;
+            _score = 0;
+            _host = false;
+        }
+
+        public string Username
+        {
+            get { return _username; }
+            set { _username = value; }
+        }
+
+        public int Score
+        {
+            get { return _score; }
+            set { _score = value; }
+        }
+
+        public bool Host
+        {
+            get { return _host; }
+            set { _host = value; }
+        }
+    }
+}