Yarn: MinecraftClient.openScreen -> setScreen

创建于 2020-07-21  ·  10评论  ·  资料来源: FabricMC/yarn

它首先关闭当前屏幕。 openScreen听起来好像前一个屏幕保持打开状态,而setScreen(null)openScreen(null)更有意义。

refactor

最有用的评论

setScreen 让它听起来像一个简单的 setter

setter 而不仅仅是字段的全部意义在于在设置字段时运行额外的逻辑(无论额外的逻辑是否已经存在,或者您希望将来有可能在不破坏设置字段的代码的情况下添加它)。

所有10条评论

是的,对我来说很有意义。 永远只有其中之一。

是的,它也可以交换屏幕,所以setScreen确实更好

真的不喜欢这种变化。 setScreen 让它听起来像一个简单的 setter,但如果一致同意的话......好吧

还是present / display / show

如果我们听从 liach 的建议,那感觉就像是为了改名而改名

大声笑,否则你需要得到一个让 shartte 满意的

setScreen 让它听起来像一个简单的 setter

setter 而不仅仅是字段的全部意义在于在设置字段时运行额外的逻辑(无论额外的逻辑是否已经存在,或者您希望将来有可能在不破坏设置字段的代码的情况下添加它)。

@liach Nah,不需要取悦我。 我不认为这_特别_重要,并且认为当前名称很好,但是将其重命名为 setScreen 并不是世界末日:-D

另一个建议: switchScreen

switchScreen(null)仍然非常有意义,它没有看起来像一个 setter 或可逆操作的问题。

如果我们听从 liach 的建议,那感觉就像是为了改名而改名

我以为是这里的问题? 我发现openScreen非常混乱,我花了五分钟的时间寻找closeScreenlastScreenpopScreen 。 具有open至少应该具有close

present / display / show可能有同样的问题,尽管它们的逆并不是普遍的( destroyconceal , hide )。

setScreen很好,因为 setter 可以有更多的代码,而不仅仅是field = param 。 Yarn 已经调用了许多更长的 setter setX即使它们不仅仅是设置字段的值。

此页面是否有帮助?
0 / 5 - 0 等级

相关问题

Sollace picture Sollace  ·  5评论

copygirl picture copygirl  ·  6评论

Juuxel picture Juuxel  ·  5评论

ChloeDawn picture ChloeDawn  ·  6评论

quat1024 picture quat1024  ·  6评论