Yarn: Entity # isInWater, Entity # isTouchingWater dan Entity # isInsideWater adalah nama yang membingungkan.

Dibuat pada 12 Nov 2019 · 8Komentar · Sumber: FabricMC/yarn

Metode isInWater dan isInsideWater saat ini sangat membingungkan: metode memiliki nama yang sangat mirip tetapi tidak mengembalikan hasil yang sama sama sekali.

isInWater mengembalikan true hanya jika hitbox lengkap ada di dalam air, dan harus diganti namanya menjadi isSubmergedInWater . Dan isInWater mengembalikan nilai bidang inWater yang ditetapkan oleh isSubmergedIn(FluidTags.WATER, true) .
Mengganti nama field inWater juga harus dilakukan.

isInsideWater mengembalikan nilai true ketika hitbox menyentuh air, metode ini harus diganti namanya menjadi isTouchingWater . Bidang terkait insideWater juga harus diganti namanya.

Karena isTouchingWater sudah ada, metode ini juga harus diganti namanya.
Nama isWet muncul di beberapa pikiran tetapi mungkin bertentangan dengan WolfEntity#isWet yang hanya untuk sisi klien.

bug

Sumber

LambdAurora

👍6

Komentar yang paling membantu

Masalah ini telah diperbaiki dengan # 1058 sekarang.

LambdAurora pada 19 Jan 2020

❤3

Semua 8 komentar

Mungkin kita bisa mengganti nama WolfEntity # isWet menjadi isWolfWet ?

Pyrofab pada 12 Nov 2019

Masalah saya dengan mengganti nama isTouchingWater menjadi isWet adalah tidak sepenuhnya benar dari sudut pandang bahasa. Anda bisa basah untuk waktu tertentu setelah menyentuh air. Saya memikirkan isMakingContactWithWater tetapi itu terlalu lama.

LambdAurora pada 12 Nov 2019

Bisa selalu isTouchingWaterBlock dan isTouchingWater . Ini tidak ideal tetapi harus cukup jelas.

Pyrofab pada 16 Nov 2019

👍2

Karena metode isWet adalah metode terpisah di WolfEntity (daripada hanya menggunakan Entity.isWet , atau menimpanya), saya menduga bahwa nama Mojang mungkin sesuatu seperti shouldDoXyz , di mana "xyz" adalah sesuatu yang dilakukan serigala hanya saat basah.

Runemoro pada 19 Nov 2019

👍2

Tergantung. Serigala mengguncang tubuh mereka saat basah, dan jika mereka basah, partikel air juga akan menempel pada mereka. shouldDoXyz mungkin juga tidak cocok untuk kasus render partikel.