Yarn: InfoEnchantment ->?

Creado en 13 ene. 2020  ·  15Comentarios  ·  Fuente: FabricMC/yarn

InfoEnchantment no es un encantamiento llamado "Información", sino un objeto de datos que almacena un encantamiento junto con su nivel.

Nombres sugeridos:

  • EnchantmentState

    • > Tampoco es un estado. Un estado suena como un mapaque está vinculado a una pila de elementos para mí.

      - @liach

  • EnchantmentEntry

    • En mi opinión, un poco vago, y realmente no describe lo que hace.

  • EnchantmentInfo

    • Para mí, eso suena como una clase que almacena la descripción ("información") de un encantamiento.

discussion refactor

Comentario más útil

¿Qué hay de cambiar el nombre de Enchantment a EnchantmentType y este a Enchantment ?

EnchantmentType todavía encaja con el otro, y también Enchantment para esta clase, ya que se podría decir que todo (tipo + nivel) es un encantamiento, como "Nitidez II", por ejemplo.

Todos 15 comentarios

EnchantmentInfo es probablemente el que mejor encaja de esos tres. No es perfecto, pero podría extrapolar la información como si fuera el emparejamiento de un "tipo de encantamiento" (encantamiento) con "nivel de encantamiento" (numérico).

Sin embargo, ¿para qué se utiliza?

Se usa principalmente en EnchantmentHelper para representar un encantamiento que se aplicará a un elemento.

¿Entonces es una aplicación de un encantamiento? ¿Aplicación de encantamiento? EnchantOptions o EnchantAttributes ?

EnchantmentLevelPair ?

¿Qué pasa con EnchantmentInstance ? Un poco como StatusEffectInstance

no es realmente una instancia, y esta clase en realidad se usa muy raramente, más como un Map.Entry<Enchantment, Integer>

EnchantmentValue?

No estoy siguiendo la primera mitad de la lógica de Liach, él realmente no respalda su afirmación de que "no es realmente una instancia".

Lo que realmente tengo contra EnchantmentInstance es que no es muy descriptivo, algo así como StatusEffectInstance tampoco es muy descriptivo, y tampoco lo es ItemInstance , que parece ser el nombre de Mojang por ItemStack acuerdo con las cadenas en los fijadores de datos. La diferencia entre los tres es que para los efectos de estado, realmente no existe un nombre mejor y es difícil agrupar sus muchos campos en un solo nombre. ItemStack es una buena alternativa a ItemInstance ya que es un nombre bien establecido que la wiki también usa y tiene un sentido lógico.

Sigo pensando que EnchantmentLevelPair es el mejor nombre, ya que contiene solo dos campos, encantamiento y nivel. Y su uso también grita que es básicamente un Pair<Enchantment, Integer> , que se usa como una clase de utilidad dentro de EnchantmentHelper y iirc en ningún otro lugar.

ItemInstance , que parece ser el nombre de Mojang para ItemStack según las cadenas en los fijadores de datos.

Las referencias de tipo vanilla DFU usan item_stack .

Las referencias de tipo vanilla DFU usan item_stack .

¿Quizás una solución se refiere a ese nombre en alguna parte?

EnchantmentStack

: pequeña_patata:

Además, ¿por qué no elegimos EnchantmentInfo antes?

¿Tenemos una clase EnchantmentLevel ? ¿Qué pasa con EnchantmentLevels (por más de una vez)?

El propósito de esta clase es almacenar un encantamiento y un par de niveles, por lo que sugiero LevelledEnchantment o EnchantmentLevelPair .

EnchantmentStack no funciona. Solo puedes tener una pila de objetos concretos.

¿Qué hay de cambiar el nombre de Enchantment a EnchantmentType y este a Enchantment ?

EnchantmentType todavía encaja con el otro, y también Enchantment para esta clase, ya que se podría decir que todo (tipo + nivel) es un encantamiento, como "Nitidez II", por ejemplo.

¿Fue útil esta página
0 / 5 - 0 calificaciones

Temas relacionados

Runemoro picture Runemoro  ·  3Comentarios

Sollace picture Sollace  ·  5Comentarios

Runemoro picture Runemoro  ·  4Comentarios

liach picture liach  ·  4Comentarios

Runemoro picture Runemoro  ·  4Comentarios