Networking #2

Zusammengeführt
Lixfel hat 16 Commits von network nach master 2022-05-10 08:53:36 +02:00 zusammengeführt
Besitzer
Keine Beschreibung angegeben.
Chaoscaot hat 5 Commits 2022-04-24 13:32:10 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2022-04-24 13:32:19 +02:00 hinzugefügt
Chaoscaot hat eine neue Abhängigkeit 2022-04-24 13:35:15 +02:00 hinzugefügt
Chaoscaot hat eine Abhängigkeit 2022-04-24 13:35:18 +02:00 entfernt
Chaoscaot hat eine neue Abhängigkeit 2022-04-24 13:35:22 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2022-04-24 20:55:51 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2022-04-24 21:07:46 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2022-04-24 21:20:49 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2022-04-24 21:44:56 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2022-04-24 22:32:23 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2022-04-24 22:39:46 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2022-04-25 14:16:08 +02:00 hinzugefügt
Lixfel hat 2022-04-25 22:04:47 +02:00 Änderungen angefragt
Lixfel hat einen Kommentar hinterlassen
Besitzer

Also, ich weiß jetzt noch nicht, warum das ganze auf dem CommandFramework aufbauen muss, das würde ich mir einmal auf Normalbranch, damit das einfach besser überprüfbar ist, wünschen.

Die Unterscheidung, in welche Richtung das Packet geht finde ich vollkommen überflüssig, das kann raus.

Ebenso bin ich noch nicht von den Handlern begeistert, da es ja eine Funktion geben muss, welche die Pakete deserialisiert (bzw. das aufruft) und dann einen Handler übergibt (also wahrscheinlich sich selbst)! Da fände ich es deutlich eleganter, wenn es einfach einen Handler gibt, welcher eine Map<Class<? extends Packet>, Consumer<? extends Packet>> hat. Du muss ja auch bedenken, dass auf Spigot-Seite nicht nur der SpigotCore Handler registriert, sondern z.B. auch das LobbySystem (FightInfoPacket). Das sehe ich mit deiner derzeitigen Architektur als nicht möglich an.

Also, ich weiß jetzt noch nicht, warum das ganze auf dem CommandFramework aufbauen muss, das würde ich mir einmal auf Normalbranch, damit das einfach besser überprüfbar ist, wünschen. Die Unterscheidung, in welche Richtung das Packet geht finde ich vollkommen überflüssig, das kann raus. Ebenso bin ich noch nicht von den Handlern begeistert, da es ja eine Funktion geben muss, welche die Pakete deserialisiert (bzw. das aufruft) und dann einen Handler übergibt (also wahrscheinlich sich selbst)! Da fände ich es deutlich eleganter, wenn es einfach einen Handler gibt, welcher eine Map<Class<? extends Packet>, Consumer<? extends Packet>> hat. Du muss ja auch bedenken, dass auf Spigot-Seite nicht nur der SpigotCore Handler registriert, sondern z.B. auch das LobbySystem (FightInfoPacket). Das sehe ich mit deiner derzeitigen Architektur als nicht möglich an.
Besitzer

Also, ich weiß jetzt noch nicht, warum das ganze auf dem CommandFramework aufbauen muss, das würde ich mir einmal auf Normalbranch, damit das einfach besser überprüfbar ist, wünschen.

Das geht nicht so einfach, weil ich noch ein wenig für den CommonCore noch eingestellt habe, worauf chaos hier dependet d.h. CommandFramework erst mergen dann den hier.

> Also, ich weiß jetzt noch nicht, warum das ganze auf dem CommandFramework aufbauen muss, das würde ich mir einmal auf Normalbranch, damit das einfach besser überprüfbar ist, wünschen. Das geht nicht so einfach, weil ich noch ein wenig für den CommonCore noch eingestellt habe, worauf chaos hier dependet d.h. CommandFramework erst mergen dann den hier.
Chaoscaot hat 1 Commit 2022-05-01 21:50:31 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2022-05-01 22:13:41 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2022-05-01 22:46:11 +02:00 hinzugefügt
Chaoscaot hat ein Review von Lixfel 2022-05-02 13:41:02 +02:00 angefragt
Chaoscaot hat 1 Commit 2022-05-03 15:43:19 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2022-05-03 15:51:32 +02:00 hinzugefügt
Chaoscaot hat 1 Commit 2022-05-03 15:55:16 +02:00 hinzugefügt
Lixfel hat 2022-05-09 09:12:25 +02:00 überprüft
Lixfel hat einen Kommentar hinterlassen
Besitzer

Evtl. noch eine SteamWarCI für die Tests?

Evtl. noch eine SteamWarCI für die Tests?
@ -0,0 +23,4 @@
import java.io.*;
@EqualsAndHashCode
Besitzer

Brauchts das ganze EqualsAndHashCode-Zeug tatsächlich?

Brauchts das ganze EqualsAndHashCode-Zeug tatsächlich?
Autor
Besitzer

wegen den Tests

wegen den Tests
Lixfel markierte diese Unterhaltung als gelöst
@ -0,0 +32,4 @@
ObjectOutputStream oos = new ObjectOutputStream(baos);
oos.writeObject(this);
oos.flush();
} catch (Exception e) {
Besitzer

Wären das nicht nur IOExceptions? Ansonsten gäbe es ja auch die ExceptionA | ExceptionB -Syntax.

Wären das nicht nur IOExceptions? Ansonsten gäbe es ja auch die ExceptionA | ExceptionB -Syntax.
Chaoscaot markierte diese Unterhaltung als gelöst
@ -0,0 +47,4 @@
try {
ObjectInputStream ois = new ObjectInputStream(bais);
return (NetworkPacket) ois.readObject();
} catch (Exception e) {
Besitzer

siehe oben.

siehe oben.
Chaoscaot markierte diese Unterhaltung als gelöst
@ -0,0 +27,4 @@
@AllArgsConstructor
@EqualsAndHashCode(callSuper = true)
@Getter
public class InventoryCallbackPacket extends NetworkPacket {
Besitzer

Überlegung (nichts für diesen PR, aber potentiell etwas für einen NachfolgePR): CommonGUI?

Überlegung (nichts für diesen PR, aber potentiell etwas für einen NachfolgePR): CommonGUI?
Autor
Besitzer

Möglich, kann ich mich hiernach mal dran setzen

Möglich, kann ich mich hiernach mal dran setzen
@ -0,0 +35,4 @@
private byte win;
private int blueSchem;
private int redSchem;
private List<Integer> bluePlayers;
Besitzer

Sind ArrayLists Serializable? Ansonsten müsste es auch mit int[] gehen...

Sind ArrayLists Serializable? Ansonsten müsste es auch mit int[] gehen...
Autor
Besitzer

ja

ja
Lixfel markierte diese Unterhaltung als gelöst
@ -0,0 +43,4 @@
private int redLeader;
private int blueSchem;
private int redSchem;
private List<Integer> bluePlayers;
Besitzer

Selbe Frage hier.

Selbe Frage hier.
Lixfel markierte diese Unterhaltung als gelöst
@ -0,0 +37,4 @@
private String title;
private int player;
private int size;
private Map<Integer, String> items;
Besitzer

Serializable?

Serializable?
Autor
Besitzer

Ja

Ja
Lixfel markierte diese Unterhaltung als gelöst
Chaoscaot hat 1 Commit 2022-05-09 17:24:12 +02:00 hinzugefügt
Lixfel hat die Änderungen 2022-05-10 08:53:27 +02:00 genehmigt
Lixfel hat Commit 0bc2987b2b in master 2022-05-10 08:53:36 +02:00 gemerged
Lixfel löschte die Branch network 2022-05-10 08:53:36 +02:00
Anmelden, um an der Diskussion teilzunehmen.
Keine Beschreibung angegeben.