Yarn: Параметры IBlockState: состояние или блок?

Созданный на 28 окт. 2016  ·  6Комментарии  ·  Источник: FabricMC/yarn

Во многих местах IBlockState s используются там, где раньше находился Block . Я почти хочу вызвать параметры этого типа block поскольку это имеет для меня больше смысла. Что вы думаете?

Кроме того, возникает вопрос, нужно ли просто переименовать IBlockState в IBlock , а Block в BlockType или что-то в этом роде? Хотя, может быть, это слишком рано - стоит подождать, что Mojang собирается сделать с блоками, по крайней мере ..?

discussion

Самый полезный комментарий

Почему бы не использовать однозначные параметры вроде itemStack , blockState ?

Все 6 Комментарий

state .

И IBlockState должен остаться как есть, ИМО.

Аналогично ItemStack . Я думаю об этом как о экземпляре item больше, чем stack который в любом случае вступает в игру только для штабелируемых элементов. Я тоже думаю о IBlockState как об экземпляре блока в мире, когда он передается в такие методы.

Почему бы не использовать однозначные параметры вроде itemStack , blockState ?

@mezz Потому что они длинные и на самом деле ничего не добавляют. В идеале параметр должен быть назван не по типу, а по его использованию в коде. И при использовании я часто думаю об этом параметре как об «элементе», над которым работает метод ».

Это может сбивать с толку, потому что у нас есть класс Item но мы переименовали его в ItemType , что имеет для меня больше смысла, поскольку экземпляр Item не является экземпляром элемента, а описывает тип предмета, это устранит эту путаницу.

Из того, что обсуждалось в irc несколько раз, похоже, что они идут с state если в контексте нет нескольких видов состояний, таких как более 1 BlockState или BlockState и FluidState.

да.

Была ли эта страница полезной?
0 / 5 - 0 рейтинги

Смежные вопросы

enbrain picture enbrain  ·  4Комментарии

Juuxel picture Juuxel  ·  6Комментарии

asiekierka picture asiekierka  ·  4Комментарии

Juuxel picture Juuxel  ·  5Комментарии

Boundarybreaker picture Boundarybreaker  ·  3Комментарии