Los métodos isInWater
y isInsideWater
son actualmente muy confusos: los métodos tienen nombres muy similares pero no devuelven el mismo resultado en absoluto.
isInWater
devuelve verdadero solo cuando el hitbox completo está en el agua, y debe cambiarse el nombre a isSubmergedInWater
. Y isInWater
devuelve el valor del campo inWater
que está establecido por isSubmergedIn(FluidTags.WATER, true)
.
Entonces también se debe cambiar el nombre del campo inWater
.
isInsideWater
devuelve verdadero cuando el hitbox golpea el agua, el método debe cambiarse de nombre a isTouchingWater
. También se debe cambiar el nombre del campo asociado insideWater
.
Como isTouchingWater
ya existe, el método también debe cambiarse de nombre.
El nombre isWet
vino a algunas mentes, pero podría entrar en conflicto con WolfEntity#isWet
que es solo del lado del cliente.
¿Quizás podríamos cambiar el nombre de WolfEntity # isWet a isWolfWet
?
Mi problema al cambiar el nombre de isTouchingWater
a isWet
es que no es del todo correcto desde el punto de vista del idioma. Puede estar mojado durante cierto tiempo después de tocar el agua. Pensé en isMakingContactWithWater
pero es demasiado largo.
Siempre puede ser isTouchingWaterBlock
y isTouchingWater
. No es ideal, pero debería ser lo suficientemente claro.
Dado que el método isWet
es un método separado en WolfEntity
(en lugar de simplemente usar Entity.isWet
, o anularlo), supongo que el nombre de Mojang probablemente sea algo así como shouldDoXyz
, donde "xyz" es algo que los lobos hacen solo cuando están mojados.
Depende. Los lobos sacuden su cuerpo cuando están mojados y, si están mojados, también se forman partículas de agua sobre ellos. shouldDoXyz
puede no ajustarse tan bien al caso de renderizado de partículas.
@Runemoro Estoy de acuerdo con eso. Podría ser isFurWet
o isASoggyBoi
.
La situación podría ser que el método en lobos sea anterior a los métodos en Entity.
+1 por Wolf#isFurWet
Este problema se solucionó con el número 1058 ahora.
Comentario más útil
Este problema se solucionó con el número 1058 ahora.