Yarn: ResourceReloadListener#reload -> createReloadTask oder so ähnlich
ResourceReloadListener#reload soll selbst kein Nachladen durchführen. Stattdessen ist es sein Ziel, ein CompletableFuture zurückzugeben, das, wenn es ausgewertet wird, die eigentlichen Nachladeaufgaben an verschiedene Executor s senden soll.
Wenn Sie versuchen, das System zu betrügen, indem Sie den Code direkt in reload einfügen und einen Dummy CompletableFuture zurückgeben, der nichts tut, hängt das Spiel beim Neuladen, da die von Synchronizer#whenPrepared zurückgegebene Zukunft niemals ankommt namens.
Etwas wie createReloadTask zu benennen, scheint eine Möglichkeit zu sein, diesen Pfad zu entmutigen und klarzustellen, dass bei dieser Methode kein direktes Neuladen erfolgen sollte.
quat1024
Alle 3 Kommentare
imo, ResourceReloadListener sollte in ResourceReloader umbenannt werden, da sie diejenigen sind, die das Neuladen tatsächlich durchführen (im Vergleich zu ResourceReloadHandler ). Der Name reload ist meiner Meinung nach in Ordnung, oder wir können ein konsistentes Namensschema für alle CompletableFuture verwandten Namen in Betracht ziehen.
liach
am 22. Okt. 2019
Nach einem Blick haben wir bereits einen Ressourcen-Reloader, der das Neuladen überwacht.
Jetzt bin ich etwas verwirrt darüber, wie dies heißen soll oder ob es umbenannt werden soll oder nicht.
liach
am 10. März 2021
Hmm, vielleicht ResourceReloader -> SimpleResourceReload , ResourceReloadListener -> ResourceReloader . Benennen Sie zusätzlich ResourceReloadMonitor bis ResourceReload und benennen Sie die Implementierungen entsprechend. ResourceReload steht für ein einzelnes Neuladen und wird sofort danach verworfen, also klingt dieser Name eigentlich gut.
liach
am 10. März 2021
Verwandte Themen
Runemoro
·
4Kommentare
liach
·
4Kommentare
altrisi
·
4Kommentare
asiekierka
·
3Kommentare
Bixilon
·
5Kommentare
Hilfreichster Kommentar
imo,
ResourceReloadListenersollte inResourceReloaderumbenannt werden, da sie diejenigen sind, die das Neuladen tatsächlich durchführen (im Vergleich zuResourceReloadHandler). Der Namereloadist meiner Meinung nach in Ordnung, oder wir können ein konsistentes Namensschema für alleCompletableFutureverwandten Namen in Betracht ziehen.