iphone - 将指针从单例设置到 viewController 并更新 GUI 违反 MVC

Question

我想问一个关于良好代码实践和 MVC 模型的理论问题，情况如下:

该设计包含一个 RootViewController 和一个单例类。单例作为一个 Controller ，带有一个应该在整个应用程序中可用的计时器对象。计时器对象由 UISwitch 和 NSTimer 组成，它们都归单例 Controller 所有，因此可以根据需要将 UISwitch 添加到新的 viewController 中，并且可以在整个应用程序中访问计时器对象。

为了使用当前计时器的计数更新 RootViewController 的屏幕，我在单例中创建了一个指向 RootViewController 的指针，并将 RootViewController 设置为该指针，类似于 [singeltonOBject setDelegate:self]。
使用这种设计，可以在计时器滴答时从单例访问和更新 RootViewController 的 UILabel。我选择这样做是因为我没有时间实现常规委托(delegate)。

请注意，RootViewController 归 AppDelegate 所有，而单例类显然不归 RootViewController 所有。因此没有保留周期。

问题:
设置从单例到 RootViewController 的指针并从单例 Controller 更新 UILable 是否违反了良好的编码习惯，是否有任何基本原则未实现？

感谢您的回答!

Answer 1

问题
通常，当某些事情感觉不太对时，那是因为它不是。
虽然您的代码可以工作，但将来维护起来并不容易，而且很容易出错。即使您从不打算再次查看此代码，练习良好的编码约定总是好的。
我推荐阅读The Pragmatic Programmer其中包含许多关于如何编写更好代码的提示。其中一个提示是 最大限度地减少模块之间的耦合 .现在你的 RootViewController 知道你的 Singleton 并且你的 Singleton 知道你的 RootViewController 所以它们都相互耦合。将来，当您更改一个时，您可能也必须更改另一个。
如果要添加另一个标签怎么办？
您必须在 RootViewController 中添加标签，然后更改您的单例以更新该标签。
如果您完全删除标签怎么办？ 假设你在几年后回到这个并删除标签，突然有另一个类没有编译，你可能不记得为什么。您在应用程序的一个完全独立的部分中破坏了代码。
至于遵循MVC，这违反了这一点。您的 View 正在被单独的东西修改。如果 MVC 设置正确，您应该能够放置任意数量的 View ，而无需更改任何控制数据的代码，在这种情况下是您的计时器。
现在至于内存管理，你的 RootViewController 现在被这个 Singleton 保留了。单例在应用程序的整个生命周期中都存在。您是对的，您在技术上没有保留周期，但是您的 RootViewController 现在永远不会被释放。在大多数应用程序中，这无关紧要，因为 RootViewController 始终位于堆栈底部，但这是巧合，因此无法可靠地对其进行编程。这实际上是 The Pragmatic Programmer 的另一个提示，不要巧合 .
解决方案
如果您确实需要像您说的那样的全局计时器，更好的解决方案是使用通知(另一个故事是您的全局计时器单例听起来不是一个好主意)。每当您从单例中更新标签时，您都可以触发通知。您的 RootViewController 将收到此通知并可以更新其标签。如果需要，您可以在通知中传递数据。现在，如果您对 View 进行更改并想要更新其他内容，您只需在一个位置(RootViewController)更改代码。