Fixed two NPE's
This commit is contained in:
parent
4f75801ed8
commit
650493f74e
@ -6,7 +6,11 @@ namespace Network {
|
|||||||
public class ConnectionList {
|
public class ConnectionList {
|
||||||
|
|
||||||
private Peer Peer;
|
private Peer Peer;
|
||||||
private List<Connection> Connections;
|
private List<Connection> Connections = new List<Connection>();
|
||||||
|
|
||||||
|
public ConnectionList(Peer peer) {
|
||||||
|
Peer = peer;
|
||||||
|
}
|
||||||
|
|
||||||
public int Length { get { return Connections.Count; } }
|
public int Length { get { return Connections.Count; } }
|
||||||
public Connection this[int i] {
|
public Connection this[int i] {
|
||||||
@ -25,7 +29,8 @@ namespace Network {
|
|||||||
}
|
}
|
||||||
|
|
||||||
public bool Contains(Connection conn) {
|
public bool Contains(Connection conn) {
|
||||||
foreach (Connection inList in Connections) {
|
for (int i = 0; i < Connections.Count; i++) {
|
||||||
|
Connection inList = Connections[i];
|
||||||
if (inList.Equals(conn)) {
|
if (inList.Equals(conn)) {
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
@ -13,10 +13,11 @@ namespace Network {
|
|||||||
|
|
||||||
private static byte[] ConfirmationBytes = { 154, 211 };
|
private static byte[] ConfirmationBytes = { 154, 211 };
|
||||||
|
|
||||||
public ConnectionList ConnectionList = new ConnectionList();
|
public ConnectionList ConnectionList;
|
||||||
|
|
||||||
public Peer(PacketPeerUDP packetPeer) {
|
public Peer(PacketPeerUDP packetPeer) {
|
||||||
PacketPeer = packetPeer;
|
PacketPeer = packetPeer;
|
||||||
|
ConnectionList = new ConnectionList(this);
|
||||||
}
|
}
|
||||||
|
|
||||||
public abstract void Initialize(string address, int port);
|
public abstract void Initialize(string address, int port);
|
||||||
@ -65,7 +66,6 @@ namespace Network {
|
|||||||
int Port = PacketPeer.GetPacketPort();
|
int Port = PacketPeer.GetPacketPort();
|
||||||
|
|
||||||
PacketBuffer PB = PacketBuffer.FromByteBuffer(Buffer);
|
PacketBuffer PB = PacketBuffer.FromByteBuffer(Buffer);
|
||||||
|
|
||||||
if (PB.Length > ConfirmationBytes.Length) {
|
if (PB.Length > ConfirmationBytes.Length) {
|
||||||
bool Confirmed = true;
|
bool Confirmed = true;
|
||||||
foreach (byte B in ConfirmationBytes) {
|
foreach (byte B in ConfirmationBytes) {
|
||||||
@ -75,6 +75,7 @@ namespace Network {
|
|||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
GD.print("Wow nice");
|
||||||
if (Confirmed) {
|
if (Confirmed) {
|
||||||
Connection conn = new Connection(Address, Port);
|
Connection conn = new Connection(Address, Port);
|
||||||
Singleton.ConnectionList.AddConnection(conn);
|
Singleton.ConnectionList.AddConnection(conn);
|
||||||
|
Loading…
Reference in New Issue
Block a user