Pour moi c’est plus un exemple qu’un tutoriel.
Par contre, il y a des truc qui ne servent a rien comme l’a dit Robin.
private void getEntitiesNearby(World world, double currentPosX, double currentPosY, double currentPosZ)
{
currentPosX = this.posX;
currentPosY = this.posY;
currentPosZ = this.posZ;
List entitiesNearby = world.getEntitiesWithinAABB(VotreSousFifre.class, AxisAlignedBB.getBoundingBox(currentPosX, currentPosY, currentPosZ, currentPosX + 32, currentPosY + 32, currentPosZ + 32));
}
Ça typiquement ça ne sert a rien, ça a l’air d’être faux et c’est une erreur de conception orientée objet.
C’est pas propre du tout, ça duplique des variables, ça rend le code illisible, en tout cas pas clair du tout.
Normalement le code de ta fonction devrais juste être :
private List <entity>getEntitiesNearby()
{
return this.worldObj.getEntitiesWithinAABB(VotreSousFifre.class, AxisAlignedBB.getBoundingBox(this.posX, this.posY, this.posZ, this.posX + 32, this.posY + 32, this.posZ + 32));
}
Mais vu que tu n’as qu’une seule instruction dans ta fonction et que la visibilité de celle-ci est privée, ça ne sert a rien de faire une fonction pour ça. Autant directement écrire l’instruction dans [size=xx-smallonLivingUpdate()]
Edit : je n’avais pas vu que tu avais mis le heal dans la fonction getEntitiesNearby(), c’est encore pire du coup puisque la fonction n’a pas un nom correct. Déjà faire un assesseur qui retourne void c’est louche, mais getEntitiesNearby() veut bien dire “obtenir les entités à proximités” et pas faire le calcul des entités puis soigner au cas ou. Il faut mettre ton code au propre.</entity>
