Yarn: ResourceReloadListener#reload -> createReloadTask ou algo assim
ResourceReloadListener#reload não deve realizar nenhuma recarga por conta própria. Em vez disso, seu objetivo é retornar um CompletableFuture que, quando avaliado, deve enviar as tarefas de recarregamento reais para vários Executor s.
Se você tentar enganar o sistema colocando código diretamente em reload e retornando algum CompletableFuture fictício que não faz nada, o jogo trava ao recarregar, pois o futuro retornado por Synchronizer#whenPrepared nunca fica chamado.
Nomeá-lo como createReloadTask parece uma maneira de desencorajar esse caminho e esclarecer que nenhum recarregamento deve ser feito diretamente nesse método.
quat1024
Todos 3 comentários
imo, ResourceReloadListener deve ser renomeado para ResourceReloader , pois são eles que realmente realizam o recarregamento (compare com ResourceReloadHandler ). O nome reload está bom, ou podemos considerar um esquema de nomenclatura consistente para todos os nomes relacionados CompletableFuture .
liach
em 22 out. 2019
Depois de uma olhada, já temos um recarregador de recursos que monitora o recarregamento.
Agora, estou um pouco confuso sobre como isso deve ser nomeado, ou se deve ser renomeado ou não.
liach
em 10 mar. 2021
Hmm, talvez ResourceReloader -> SimpleResourceReload , ResourceReloadListener -> ResourceReloader . Além disso, nomeie ResourceReloadMonitor para ResourceReload e nomeie as implementações de forma correspondente. ResourceReload representa uma única recarga e é descartada imediatamente depois, então esse nome realmente soa bem.
liach
em 10 mar. 2021
Questões relacionadas
quat1024
·
6Comentários
Juuxel
·
6Comentários
Sollace
·
5Comentários
Runemoro
·
4Comentários
liach
·
4Comentários
Comentários muito úteis
imo,
ResourceReloadListenerdeve ser renomeado paraResourceReloader, pois são eles que realmente realizam o recarregamento (compare comResourceReloadHandler). O nomereloadestá bom, ou podemos considerar um esquema de nomenclatura consistente para todos os nomes relacionadosCompletableFuture.