Yarn: InfoPesona ->?

Dibuat pada 13 Jan 2020  ·  15Komentar  ·  Sumber: FabricMC/yarn

InfoEnchantment bukanlah sebuah enchantment yang disebut "Info", tetapi sebuah objek data yang menyimpan sebuah enchant bersama dengan levelnya.

Nama yang disarankan:

  • EnchantmentState

    • > Ini juga bukan negara. Sebuah negara terdengar seperti Petayang terikat ke tumpukan item untuk saya.

      - @liach

  • EnchantmentEntry

    • IMO yang agak kabur, dan tidak benar-benar menggambarkan fungsinya.

  • EnchantmentInfo

    • Bagi saya, itu terdengar seperti kelas yang menyimpan deskripsi ("info") dari sebuah pesona.

discussion refactor

Komentar yang paling membantu

Bagaimana dengan mengganti nama Enchantment menjadi EnchantmentType dan yang ini menjadi Enchantment ?

EnchantmentType masih cocok dengan yang lain, dan begitu juga Enchantment untuk kelas ini, karena Anda dapat mengatakan bahwa semuanya (tipe + level) adalah pesona, seperti "Ketajaman II" misalnya.

Semua 15 komentar

EnchantmentInfo mungkin yang paling cocok dari ketiganya. Ini tidak sempurna, tetapi Anda dapat memperkirakan info sebagai pasangan dari "tipe pesona" (Pesona) dengan "tingkat pesona" (numerik).

Padahal itu digunakan untuk apa?

Ini digunakan sebagian besar di EnchantmentHelper untuk mewakili pesona yang akan diterapkan ke item.

Jadi itu aplikasi dari pesona? Aplikasi pesona? EnchantOptions atau EnchantAttributes ?

EnchantmentLevelPair ?

Bagaimana dengan EnchantmentInstance ? Sedikit seperti StatusEffectInstance

tidak benar-benar sebuah instance, dan kelas ini sebenarnya sangat jarang digunakan, lebih seperti Map.Entry<Enchantment, Integer>

Nilai Pesona?

Saya tidak mengikuti paruh pertama logika liach, dia tidak benar-benar mendukung klaim "bukan sebuah contoh".

Apa yang benar-benar saya miliki terhadap EnchantmentInstance adalah bahwa itu tidak terlalu deskriptif, semacam StatusEffectInstance juga tidak terlalu deskriptif, dan juga ItemInstance , yang tampaknya merupakan nama Mojang untuk ItemStack sesuai dengan string di pemecah data. Perbedaan antara ketiganya adalah bahwa untuk efek status, sebenarnya tidak ada nama yang lebih baik, dan sulit untuk memasukkan banyak bidangnya ke dalam satu nama. ItemStack adalah alternatif yang baik untuk ItemInstance karena merupakan nama mapan yang juga digunakan wiki, dan masuk akal secara logis.

Saya masih berpikir bahwa EnchantmentLevelPair adalah nama terbaik, karena hanya berisi dua bidang, pesona dan level. Dan penggunaannya juga menunjukkan bahwa pada dasarnya adalah Pair<Enchantment, Integer> , yang digunakan sebagai kelas utilitas di dalam EnchantmentHelper dan iirc di tempat lain.

ItemInstance , yang tampaknya merupakan nama Mojang untuk ItemStack menurut string di datafixer.

Referensi tipe DFU vanilla menggunakan item_stack .

Referensi tipe DFU vanilla menggunakan item_stack .

Mungkin perbaikan merujuk ke nama itu di suatu tempat?

EnchantmentStack

:tiny_potato:

Juga mengapa kita tidak pergi dengan EnchantmentInfo sebelumnya?

Apakah kita memiliki kelas EnchantmentLevel ? Bagaimana dengan EnchantmentLevels (untuk lebih dari sekali)?

Tujuan dari kelas ini adalah untuk menyimpan pesona dan pasangan level, jadi saya sarankan LevelledEnchantment atau EnchantmentLevelPair .

EnchantmentStack tidak berfungsi. Anda hanya dapat memiliki setumpuk benda konkret.

Bagaimana dengan mengganti nama Enchantment menjadi EnchantmentType dan yang ini menjadi Enchantment ?

EnchantmentType masih cocok dengan yang lain, dan begitu juga Enchantment untuk kelas ini, karena Anda dapat mengatakan bahwa semuanya (tipe + level) adalah pesona, seperti "Ketajaman II" misalnya.

Apakah halaman ini membantu?
0 / 5 - 0 peringkat

Masalah terkait

asiekierka picture asiekierka  ·  4Komentar

ChloeDawn picture ChloeDawn  ·  6Komentar

Runemoro picture Runemoro  ·  3Komentar

quat1024 picture quat1024  ·  6Komentar

copygirl picture copygirl  ·  6Komentar