Yarn: Parameter IBlockState: status atau blok?

Dibuat pada 28 Okt 2016  ·  6Komentar  ·  Sumber: FabricMC/yarn

Di banyak tempat, IBlockState s digunakan di tempat-tempat di mana Block seharusnya berada di masa lalu. Saya hampir ingin memanggil parameter jenis itu block karena lebih masuk akal bagi saya. Bagaimana menurutmu?

Selain itu, ini membuka pertanyaan apakah IBlockState hanya perlu diganti namanya menjadi IBlock , dan Block menjadi BlockType , atau apa? Meskipun mungkin itu terlalu dini - setidaknya harus menunggu apa yang akan dilakukan Mojang terhadap Blocks..?

discussion

Komentar yang paling membantu

Mengapa tidak parameter yang tidak ambigu seperti itemStack , blockState ?

Semua 6 komentar

state .

Dan IBlockState harus tetap seperti itu, IMO.

Mirip dengan ItemStack . Saya menganggapnya sebagai contoh dari item lebih dari stack yang hanya berperan untuk item yang dapat ditumpuk. Saya menganggap IBlockState sebagai turunan dari blok di dunia juga, ketika diteruskan ke metode seperti ini.

Mengapa tidak parameter yang tidak ambigu seperti itemStack , blockState ?

@mezz Karena panjang dan tidak benar-benar menambahkan apa pun. Idealnya parameter tidak boleh dinamai menurut jenisnya tetapi penggunaannya dalam kode. Dan dalam penggunaan, saya sering menganggap parameter tersebut sebagai "'item' metode ini beroperasi".

Mungkin membingungkan karena kami memiliki kelas Item tetapi katakanlah kami mengganti namanya menjadi ItemType , yang lebih masuk akal bagi saya karena instance Item bukan instance item tetapi menjelaskan jenis item, itu akan menghilangkan kebingungan ini.

Dari apa yang telah dibahas di irc beberapa kali, sepertinya akan menggunakan state kecuali ada beberapa jenis status seperti lebih dari 1 BlockState atau BlockState dan FluidState dalam konteksnya.

Ya.

Apakah halaman ini membantu?
0 / 5 - 0 peringkat

Masalah terkait

ChloeDawn picture ChloeDawn  ·  6Komentar

quat1024 picture quat1024  ·  6Komentar

Runemoro picture Runemoro  ·  4Komentar

enbrain picture enbrain  ·  4Komentar

Juuxel picture Juuxel  ·  5Komentar