LLAPI Issue, Player Updating?

Unity Engine
1

So im using LLAPI to manage my server and client for a game. Im currently getting this issue however.

This is printed in the console when it should be spitting out constant updated coordinates for players…

Here is the code for client thus far… I really cannot see the issue

using System.Collections;
using System.Collections.Generic;
using System.Text;
using UnityEngine;
using UnityEngine.Networking;
using UnityEngine.UI;

public class Client : MonoBehaviour {

    public class Player
    {
        public string playerName;
        public GameObject avatar;
        public int connectionId;
    }

    private const int MAX_CONNECTION = 100;

    private int port = 5701;

    private int hostId;
    private int webHostId;

    private int ourClientId;
    private int connectionId;

    private int reliableChannel;
    private int unreliablechannel;

    private float connectionTime;
    private bool isStarted = false;
    private byte error;
    private bool isConnected = false;

    private string playerName;

    public GameObject playerPrefab;
    public Dictionary<int,Player> players = new Dictionary<int, Player>();

    public void Connect()
    {
        //Does the player have a name
        string pName = GameObject.Find("NameInput").GetComponent<InputField>().text;
        if (pName == "")
        {
            Debug.Log("You must set a name");
            return;
        }

        playerName = pName;

        NetworkTransport.Init();
        ConnectionConfig cc = new ConnectionConfig();

        reliableChannel = cc.AddChannel(QosType.Reliable);
        unreliablechannel = cc.AddChannel(QosType.Unreliable);

        HostTopology topo = new HostTopology(cc, MAX_CONNECTION);

        hostId = NetworkTransport.AddHost(topo, 0);
        connectionId = NetworkTransport.Connect(hostId, "127.0.0.1", port, 0, out error);

        connectionTime = Time.time;
        isConnected = true;
    }

    private void Update()
    {
        if (!isConnected)
            return;

        int recHostId;
        int connectionId;
        int channelId;
        byte[] recBuffer = new byte[1024];
        int bufferSize = 1024;
        int dataSize;
        byte error;
        NetworkEventType recData = NetworkTransport.Receive(out recHostId, out connectionId, out channelId, recBuffer, bufferSize, out dataSize, out error);
        switch (recData)
        {
            case NetworkEventType.DataEvent:           //3
                string msg = Encoding.Unicode.GetString(recBuffer, 0, dataSize);
                Debug.Log("Receiving : " + msg);
                string[] splitData = msg.Split('|');

                switch (splitData[0])
                {
                    case "ASKNAME":
                        OnAskName(splitData);
                        break;

                    case "CNN":
                        SpawnPlayer(splitData[1], int.Parse(splitData[2]));
                    break;

                    case "DC":
                        PlayerDisconnected(int.Parse(splitData[1]));
                break;

                    case "ASKPOSITION":
                        OnAskPosition(splitData);
                        break;

                    default:
                        Debug.Log("Invalid message : " + msg);
                        break;
                }

                break;

        }
    }

    private void OnAskName(string[] data)
    {
        //Set this clients ID
        ourClientId = int.Parse(data[1]);

        // Send our name to the server
        Send("NAMEIS|" + playerName, reliableChannel);

        // Create all the other players
        for (int i = 2; i < data.Length - 1; i++)
        {
            string[] d = data[i].Split('%');
            SpawnPlayer(d[0], int.Parse(d[1]));
        }
    }

    private void OnAskPosition(string[] data)
    {
        if (!isStarted)
            return;
        //update everyone else
        for (int i = 1; i <= data.Length-1; i++)
        {


            string[] d = data[i].Split('%');

            // prevent the server from updating us
            if (ourClientId != int.Parse(d[0]))
            {
            Vector3 position = Vector3.zero;
            position.x = float.Parse(d[1]);
            position.y = float.Parse(d[2]);
            players[int.Parse(d[0])].avatar.transform.position = position;
            }
        }

        //Send own position
        Vector3 myPosition = players[ourClientId].avatar.transform.position;
        string m = "MYPOSITION|" + myPosition.x.ToString() + '|' + myPosition.y.ToString();
        Send(m, unreliablechannel);

    }

    private void SpawnPlayer(string playerName, int cnnId)
    {
        GameObject go = Instantiate(playerPrefab) as GameObject;

        //Is this ours?
        if (cnnId == ourClientId)
        {
            //add mobility
            go.AddComponent<PlayerCon>();
            GameObject.Find("Canvas").SetActive(false);
            isStarted = true;
        }

        Player p = new Player();
        p.avatar = go;
        p.playerName = playerName;
        p.connectionId = cnnId;
        players.Add(cnnId, p);
    }

    private void PlayerDisconnected(int cnnId)
    {
        Destroy(players[cnnId].avatar);
        players.Remove(cnnId);
    }

    private void Send(string message, int channelId)
    {
        Debug.Log("Sending : " + message);
        byte[] msg = Encoding.Unicode.GetBytes(message);

NetworkTransport.Send(hostId,connectionId, channelId, msg, message.Length * sizeof(char), out error);
       
    }
}
2

You should REALLY not use unicode encoded strings like that for your messages. You probably want to use the NetworkWriter & NetworkReader,

In addition, why is your server even asking for the position? Why doesnt the client just send it x times per second.

As for the issue. The message recieved is missing a | symbol.

1 Like