Yarn: Pembantu vs Util vs Util

Dibuat pada 9 Feb 2020  ·  24Komentar  ·  Sumber: FabricMC/yarn

Ada 12 kelas yang diakhiri dengan Helper :

net/minecraft/server/rcon/BufferHelper
net/minecraft/server/rcon/DataStreamHelper
net/minecraft/client/gui/DrawableHelper
net/minecraft/client/util/math/Rotation3Helper
net/minecraft/client/util/DefaultSkinHelper
net/minecraft/client/texture/MipmapHelper
net/minecraft/nbt/NbtHelper
net/minecraft/block/RailPlacementHelper
net/minecraft/world/SpawnHelper
net/minecraft/util/JsonHelper
net/minecraft/util/math/MathHelper
net/minecraft/enchantment/EnchantmentHelper

Ada 17 kelas yang diakhiri dengan Util :

net/minecraft/client/sound/AlUtil
net/minecraft/client/util/InputUtil
net/minecraft/client/util/GlfwUtil
net/minecraft/client/util/SmoothUtil
net/minecraft/client/model/ModelUtil
net/minecraft/client/texture/TextureUtil
net/minecraft/util/ChatUtil
net/minecraft/util/MapUtil
net/minecraft/util/FileNameUtil
net/minecraft/util/Util
net/minecraft/entity/ProjectileUtil
net/minecraft/entity/ai/brain/task/LookTargetUtil
net/minecraft/entity/effect/StatusEffectUtil
net/minecraft/entity/DamageUtil
net/minecraft/potion/PotionUtil
net/minecraft/test/StructureTestUtil
net/minecraft/test/TestUtil

Ada 5 kelas yang diakhiri dengan Utils :

net/minecraft/client/util/ScreenshotUtils
net/minecraft/client/util/NetworkUtils
net/minecraft/client/util/GlAllocationUtils
net/minecraft/network/NetworkThreadUtils
net/minecraft/network/NetworkEncryptionUtils

Kita mungkin harus memilih standar dan menaatinya. Saya pribadi mendukung Utils, tetapi saya ingin mendengar pendapat orang lain.

discussion

Komentar yang paling membantu

Saya paling suka Util tunggal. 👍 untuk Util. untuk Pembantu, untuk Util

Semua 24 komentar

Saya paling suka Util tunggal. 👍 untuk Util. untuk Pembantu, untuk Util

tbh saya bisa jalan apa saja, mari kita konsisten.

Juga jika kita memiliki kata kerja kita dapat menggunakan -ing (gerund), misalnya Chatting NetworkThreading , atau kata benda jamak, misalnya Projectiles Screenshots over Helper Util dll.

Itu membuat sulit untuk beberapa hal seperti MathHelper -> Maths ?? Atau Util -> s ??

Saya sudah lama berdebat untuk Maths , mengingat itu adalah singkatan yang valid untuk Mathematics . https://github.com/FabricMC/yarn/issues/249#issuecomment -446102638

Benang menggunakan nama bahasa Inggris Amerika, di mana Maths bukan singkatan yang valid untuk Mathematics .

Dalam hal ini di mana gerund atau jamak tidak berlaku, saya sarankan untuk kembali ke akhiran Helper . Jika tidak, pilih ing atau s , seperti Texts

Juga DrawableHelper bukan kelas utilitas tetapi antarmuka fungsional atau kelas abstrak dengan banyak metode instans yang nyaman

Masalah saya dengan proposal liach dengan -ing dan -s adalah membuatnya sangat sulit untuk membuat sesuatu yang konsisten, sepenuhnya mengalahkan tujuan masalah. Anda bisa membuatnya jadi hitung kelas utilitas kata benda yang diakhiri dengan -s , tetapi kemudian Anda akan memiliki pengecualian: Nbts , RailPlacements , Maps ? Tidak mungkin untuk membuat konsisten.

Kita harus mengadopsi standar dan menaatinya. Melihat suara, itu seharusnya Helper atau Util , tetapi bukan Utils .

Satu argumen untuk Util lebih dari Helper adalah bahwa kelas Util itu sendiri tidak masuk akal untuk disebut Helper . Saya pikir MathHelper dipanggil MathUtil tidak akan terlalu buruk, hanya saja semua orang terbiasa dipanggil MathHelper . Saya telah bekerja di proyek-proyek di mana kelas utilitas matematika telah disebut MathUtil , dan itu baik-baik saja, tidak ada bedanya. Nama MathUtil bahkan tidak dibahas dalam masalah penggantian nama MathHelper .

Ada lebih banyak kelas Util daripada kelas Helper saat ini, tetapi itu perlu ditimbang dengan seberapa sering kelas tersebut benar-benar digunakan, jadi saya akan memeriksanya sekarang dan melaporkan kembali hasilnya .

Tampaknya mayoritas orang menyukai akhiran Helper . Oleh karena itu saya mengusulkan untuk mengganti nama semua kelas saat ini yang diakhiri dengan Util atau Utils (kecuali kelas Util itu sendiri) menjadi Helper . Mengganti nama kelas lain yang diakhiri dengan Helper dapat didiskusikan dalam edisi terpisah.

Tampaknya mayoritas orang menyukai akhiran Helper .

Tidak?

votes

Oh, ups, aku pasti buta. Kemudian saya mengusulkan agar kita mengganti nama semua kelas yang diakhiri dengan Helper menjadi diakhiri dengan Util .

Saya mengusulkan bahwa sufiks adalah pekerjaan iblis dan kami mengganti nama semua kelas yang diakhiri dengan *elperer dan *til agar tidak memiliki sufiks itu.

MathHelper -> Maths .

PiglinHelper -> Piglins

Benang menggunakan nama bahasa Inggris Amerika, di mana Maths bukan singkatan yang valid untuk Mathematics .

Saya pikir Maths adalah istilah eksklusif Amerika?

MathHelper -> Mathinator

Maps ? saya

Perhatikan juga bahwa Guava (saya pikir itu Guava?) menggunakan pola jamak: Maps , Sets , Collections , Streams .

Ini adalah masalah saya dengan menggunakan jamak. Meskipun namanya terlihat bagus, ada terlalu banyak pengecualian. Entah karena kata itu adalah kata benda massal (misalnya "matematika") dan karena itu tidak memiliki jamak, atau karena jamaknya sudah diambil oleh perpustakaan lain (misalnya "peta"). Itu akan membuat terlalu sulit untuk tetap konsisten, mengalahkan titik mengubah mereka di tempat pertama.

Nama-nama yang baik-baik saja dalam bentuk jamak:

  • penyangga
  • DataStreams
  • dapat digambar
  • DefaultSkin
  • peta mip
  • Nbts
  • Penempatan Rel
  • Memunculkan
  • masukan
  • model
  • tekstur
  • Nama File
  • Proyektil
  • LihatTarget
  • StrukturTes
  • tes
  • Tangkapan layar
  • GlAlokasi

Nama-nama yang tidak boleh karena mereka adalah kata benda massal atau bukan kata benda:

  • Jsons
  • Matematika
  • Juga
  • Glfws
  • menghaluskan
  • Kerusakan (bentuk jamak memiliki arti yang berbeda)
  • Jaringan (hanya ada satu jaringan di sini)
  • NetworkThreads (hanya ada satu utas jaringan)
  • Enkripsi Jaringan

Nama yang tidak tepat karena sudah diambil:

  • pesona
  • peta
  • Efek Status
  • ramuan

Nama yang tidak boleh karena berbagai alasan:

  • Rotation3s (nama ini membutuhkan lebih banyak pemikiran)
  • Obrolan (walaupun secara teknis jamak dari obrolan, itu tidak cocok dengan baik sebagai kelas utilitas untuk "obrolan", dengan asumsi itulah gunanya. Jika untuk komponen teks, maka "Teks" akan menjadi nama yang baik)
  • Utils (kami tidak dapat mengganti nama ini menjadi "s" lol, lebih baik disimpan sebagai "Util")

Secara total, 18/34 nama akan baik-baik saja sebagai bentuk jamak, artinya kita harus membuat nama yang lebih baik untuk 16 lainnya. Begitu banyak untuk konsistensi.

Saya akan baik-baik saja dengan menggunakan bentuk jamak di mana itu masuk akal dan kembali ke sufiks Util di mana itu tidak masuk akal. Maka setidaknya kami memiliki aturan untuk diikuti daripada memilih secara acak antara "Pembantu" dan "Util", yang saat ini kami lakukan.

@Earthcomputer NetworkThreads berfungsi dengan baik. Ini adalah utilitas untuk bekerja dengan utas jaringan. Tidak masalah bahwa hanya ada satu dari mereka.

Anda mungkin tidak setuju dengan saya pada satu atau dua poin itu tetapi bukan itu intinya, itu tidak banyak mengubah rasio keseluruhan 18/34.

Saya akan berdebat untuk Util atau Helper , menghapusnya sepenuhnya tidak masuk akal bagi saya.

Omong-omong, beberapa tidak dapat disebutkan namanya Util . Misalnya, DataStreamHelper bukan kelas util, ini adalah objek yang membungkus aliran data.

Apakah halaman ini membantu?
0 / 5 - 0 peringkat

Masalah terkait

asiekierka picture asiekierka  ·  4Komentar

Runemoro picture Runemoro  ·  3Komentar

quat1024 picture quat1024  ·  6Komentar

Juuxel picture Juuxel  ·  5Komentar

Bixilon picture Bixilon  ·  5Komentar