Yarn: BlockPos -> Position

Créé le 20 févr. 2019  ·  6Commentaires  ·  Source: FabricMC/yarn

Billet de rétroaction pour une proposition controversée. Les raisons pour et contre cela sont très appréciées. " BlockPos est le nom Mojang connu" n'est pas un argument valable à mon avis. De nombreuses classes ont des noms Mojang connus qui ne sont pas utilisés. Je soutiens que BlockPos est activement utilisé pour plus que des blocs. Position est une représentation plus générique d'un vecteur de position basé sur int.

Commentaire le plus utile

La position est plus adaptée aux vecteurs à virgule flottante.

Tous les 6 commentaires

La "bonne" chose à propos BlockPos est qu'elle indique qu'elle est fixée à la grille de coordonnées. Position semble plus vague (plus adapté pour Vec3f ) et ne prend pas vraiment en compte les méthodes spécifiques à la grille de coordonnées que vous avez (comme le décalage par une direction).

La position est plus adaptée aux vecteurs à virgule flottante.

Oui, je vais dire que si nous changeons BlockPos , nous devons toujours souligner que c'est un entier.

Les commentaires sont appréciés et je suis d'accord que ce nom serait plus adapté à un vecteur à virgule flottante.

Si vous vous ennuyez trop, vous pouvez l'appeler IntegerLattice la place

vous pouvez l'appeler IntegerLattice à la place

Le réseau est l'ensemble de tous les points. Il faudrait que ce soit IntegerLatticePoint/IntegerLatticePosition

Cette page vous a été utile?
0 / 5 - 0 notes

Questions connexes

quat1024 picture quat1024  ·  6Commentaires

Juuxel picture Juuxel  ·  5Commentaires

Awakened-Redstone picture Awakened-Redstone  ·  4Commentaires

haykam821 picture haykam821  ·  4Commentaires

Sollace picture Sollace  ·  5Commentaires