Yarn: ResourceReloadListener#reload -> createReloadTask ou quelque chose comme ça
ResourceReloadListener#reload n'est pas censé effectuer de rechargement par lui-même. Au lieu de cela, son objectif est de renvoyer un CompletableFuture qui, une fois évalué, est censé soumettre les tâches de rechargement réelles à divers Executor s.
Si vous essayez de tromper le système en mettant du code directement dans reload et en renvoyant un faux CompletableFuture qui ne fait rien, le jeu se bloque lors du rechargement, car le futur renvoyé par Synchronizer#whenPrepared n'obtient jamais appelé.
Le nommer quelque chose comme createReloadTask semble être un moyen de décourager ce chemin et de préciser qu'aucun rechargement ne doit être effectué directement dans cette méthode.
quat1024
Tous les 3 commentaires
imo, ResourceReloadListener devrait être renommé en ResourceReloader car ce sont eux qui effectuent réellement le rechargement (comparé à ResourceReloadHandler ). Le nom reload est bien imo, ou nous pouvons envisager un schéma de nommage cohérent pour tous les noms liés à CompletableFuture .
liach
le 22 oct. 2019
Après un coup d'œil, nous avons déjà un chargeur de ressources qui surveille le rechargement.
Maintenant, je suis un peu confus sur ce que cela devrait être nommé, ou s'il doit être renommé ou non.
liach
le 10 mars 2021
Hmm, peut-être ResourceReloader -> SimpleResourceReload , ResourceReloadListener -> ResourceReloader . De plus, nommez ResourceReloadMonitor à ResourceReload et nommez les implémentations en conséquence. ResourceReload représente un seul rechargement et est supprimé immédiatement après, donc ce nom sonne vraiment bien.
liach
le 10 mars 2021
Questions connexes
ChloeDawn
·
6Commentaires
liach
·
4Commentaires
quat1024
·
6Commentaires
copygirl
·
6Commentaires
Runemoro
·
4Commentaires
Commentaire le plus utile
imo,
ResourceReloadListenerdevrait être renommé enResourceReloadercar ce sont eux qui effectuent réellement le rechargement (comparé àResourceReloadHandler). Le nomreloadest bien imo, ou nous pouvons envisager un schéma de nommage cohérent pour tous les noms liés àCompletableFuture.