Yarn: net.minecraft.container.Container-> SharedGuiState?

作成日 2018年12月22日  ·  15コメント  ·  ソース: FabricMC/yarn

コンテナは在庫を意味します。 このクラスを在庫のみのものと間違えるのは非常に簡単ですが、次のようになります。

  • 炉や機械の進捗状況など、在庫以外の数値情報を伝達します
  • ビーコン効果など、実際には数値ではない情報を同期するためにも使用されます
  • カスタムパケットで配信される情報を管理するために使用されるsyncIdを保持します
  • EnchantingTableContainerなどのボタンクリックを管理します
  • スロットとボタンにはx / yの位置があります。 これにより、コンテナはGUIの構造記述としても機能します。

したがって、 Containerという名前は私にとって実際には機能しませんでした。 このすべてを念頭に置いて、SharedGuiStateはここでは最良の名前ではないかもしれません。 すべての特定のインスタンスに名前を付けるのは難しいです。 より良いパッケージ名にするために、おそらく代わりにnet.minecraft.guistate.GuiState。 しかし、オブジェクトの目的について十分なコンセンサスを得ることができれば、はるかに適切なものを見つけることができると思います。

最も参考になるコメント

私は、Containerはほとんど意味がなく、voldemapがなければ、Containerという名前にはならないので、置き換えが何であっても、Containerを使用する必要があることに同意します。

Menuは良いと思いますが、GuiStateはあまり良くありませんが、耐えられます。 かなり頻繁に使用されるので、名前は10文字以内にする必要があります。

全てのコメント15件

だからUiDataまたはUserInterfaceData?

SyncedGui

@therealfarfetchdは、通常のGUIではないことが重要であると述べたので、おそらくSyncedInterfaceか何か

19w02aMojangが「createMenu」と言う

不和の#yarnチャネルから:

  • 私たちの何人かは名前メニューが嫌いです、それはノッチの悪臭を放ち、曖昧です
  • ServerUi / ClientUi、Gui / GuiStateのようなものを議論する
  • asieはけいおんに行くことを提案します!

Mojangの名前を知っている場所で使用するというポリシーがあるため、最終的にはMenu/Screenに移動する必要があると思います。

Mojangの名前であっても、「悪い名前を悪い名前に変更しないでください」と言います。

Containerの主な問題は、それがvoldenameであるということです。したがって、問題が発生する可能性があるため、Mojang名よりもIMOが劣っています。

私は、Containerはほとんど意味がなく、voldemapがなければ、Containerという名前にはならないので、置き換えが何であっても、Containerを使用する必要があることに同意します。

Menuは良いと思いますが、GuiStateはあまり良くありませんが、耐えられます。 かなり頻繁に使用されるので、名前は10文字以内にする必要があります。

GuiStateは見た目が良い名前ではないと思いますが、その機能を最もよく表す名前です。 もう1つの同様のオプションはGuiControllerで、一種のMVCパターン(在庫=モデル、Gui =ビュー、コンテナー=コントローラー)として見れば意味がありますが、GuiStateよりも少し長くなります。

Guiの名前が変更されたので、ScreenStateまたはScreenControllerにする必要があります。

それが長いことを除いて、私はそれについてのすべてが大好きです。 最終的にはCartographyTableScreenControllerだけでなく、 DoubleLockableScreenControllerNameableScreenControllerProviderもなります。 しかし、それは非常に明確です。

コントローラーは一般的すぎてゲームエンジンだけでは使用できないと思いますが、この場合は何とも衝突しません。 また、パッケージを使用して、より具体的にすることもできます。 net.minecraft.screen.controllerパッケージに入れてみませんか?

コントローラーは素晴らしいです👍

私はまだControllerが最も正確な名前だと思いますが、#846には非常に説得力のある採用理由があります。

このページは役に立ちましたか?
0 / 5 - 0 評価

関連する問題

Draylar picture Draylar  ·  6コメント

Bixilon picture Bixilon  ·  5コメント

altrisi picture altrisi  ·  4コメント

Awakened-Redstone picture Awakened-Redstone  ·  4コメント

asiekierka picture asiekierka  ·  3コメント