12
0

Add click callback for REntityServer #227

Zusammengeführt
Lixfel hat 9 Commits von REntityCallback nach master 2023-02-05 19:18:53 +01:00 zusammengeführt
Besitzer
Keine Beschreibung angegeben.
YoyoNow hat 1 Commit 2023-02-03 18:00:47 +01:00 hinzugefügt
Add click callback for REntityServer
Einige Prüfungen sind fehlgeschlagen
SteamWarCI Build failed
740ebc0c1d
YoyoNow hat 1 Commit 2023-02-04 14:37:09 +01:00 hinzugefügt
Rebuild
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
495e33b195
YoyoNow hat 1 Commit 2023-02-04 14:38:29 +01:00 hinzugefügt
Merge branch 'master' into REntityCallback
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
efb040c75c
YoyoNow hat 2 Commits 2023-02-04 14:39:39 +01:00 hinzugefügt
YoyoNow hat 1 Commit 2023-02-04 18:20:47 +01:00 hinzugefügt
Add REntity x,y,z getters
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
b1dea856fc
Lixfel hat 2023-02-04 19:40:28 +01:00 Änderungen angefragt
@ -45,0 +49,4 @@
private static final Reflection.FieldAccessor<Integer> useEntityTarget = Reflection.getField(useEntity, int.class, 0);
private static final Reflection.FieldAccessor<Integer> useEntityAction = Reflection.getField(useEntity, int.class, 1);
private final HashMap<Integer, REntity> entityMap = new HashMap<>();
Besitzer

Entweder ConcurrentHashMap oder synchronized-Blöcke (ansonsten werden ConcurrentModificationExceptions fliegen)

Entweder ConcurrentHashMap oder synchronized-Blöcke (ansonsten werden ConcurrentModificationExceptions fliegen)
Lixfel markierte diese Unterhaltung als gelöst
@ -50,0 +60,4 @@
private BiFunction<Player, Object, Object> filter = (player, packet) -> {
if (callback == null) return packet;
int target = useEntityTarget.get(packet);
if (!entityMap.containsKey(target)) return packet;
Besitzer

statt containsKey...get direkt ein get und dann ein Null-Check bitte.

statt containsKey...get direkt ein get und dann ein Null-Check bitte.
Lixfel markierte diese Unterhaltung als gelöst
@ -50,0 +63,4 @@
if (!entityMap.containsKey(target)) return packet;
REntity entity = entityMap.get(target);
int action = useEntityAction.get(packet);
Besitzer

Du könntest schon hier direkt mit trinary eine EntityAction draus machen (und dir dann das if und final sparen)

Du könntest schon hier direkt mit trinary eine EntityAction draus machen (und dir dann das if und final sparen)
Lixfel markierte diese Unterhaltung als gelöst
@ -50,0 +67,4 @@
if (action == 2) action = 0;
int finalAction = action;
Bukkit.getScheduler().runTaskLater(Core.getInstance(), () -> {
Besitzer

Bitte direkt runTask draus machen.

Bitte direkt runTask draus machen.
Lixfel markierte diese Unterhaltung als gelöst
@ -50,2 +75,4 @@
public REntityServer() {
Core.getInstance().getServer().getPluginManager().registerEvents(this, Core.getInstance());
TinyProtocol.instance.addFilter(useEntity, filter);
Besitzer

Ich würde das erst machen, wenn das erste mal das Callback gesetzt wird (null-Check)

Ich würde das erst machen, wenn das erste mal das Callback gesetzt wird (null-Check)
Lixfel markierte diese Unterhaltung als gelöst
@ -64,6 +95,7 @@ public class REntityServer implements Listener {
}
public void close() {
TinyProtocol.instance.removeFilter(useEntity, filter);
Besitzer

Ebenso hier abhängig machen, ob überhaupt ein Filter existiert (performance)

Ebenso hier abhängig machen, ob überhaupt ein Filter existiert (performance)
Lixfel markierte diese Unterhaltung als gelöst
YoyoNow hat 1 Commit 2023-02-04 19:56:11 +01:00 hinzugefügt
Fix pr stuff
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
b5a9b3d068
YoyoNow hat 1 Commit 2023-02-04 20:24:30 +01:00 hinzugefügt
Fix pr stuff
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
c02d7c2c7d
Lixfel hat die Änderungen 2023-02-04 20:51:08 +01:00 genehmigt
Lixfel hat 1 Commit 2023-02-05 19:17:29 +01:00 hinzugefügt
prevent double filter
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
e39e128aa6
Lixfel hat die Änderungen 2023-02-05 19:18:48 +01:00 genehmigt
Lixfel hat Commit 2d70a331cd in master 2023-02-05 19:18:53 +01:00 gemerged
Lixfel löschte die Branch REntityCallback 2023-02-05 19:18:55 +01:00
Anmelden, um an der Diskussion teilzunehmen.
Keine Beschreibung angegeben.