1

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

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

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

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

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

感谢您的回答！

4

1 回答 1

5

问题

通常，当某些事情感觉不太对时，那是因为它不是。

虽然您的代码可以工作，但将来维护起来并不容易，而且很容易出错。即使您从不打算再次查看此代码，练习良好的编码约定总是好的。

我建议阅读The Pragmatic Programmer，其中包含许多关于如何编写更好代码的提示。其中一个技巧是最小化模块之间的耦合。现在你的 RootViewController 知道你的 Singleton 并且你的 Singleton 知道你的 RootViewController 所以它们都相互耦合。将来，当您更改一个时，您可能也必须更改另一个。

如果要添加另一个标签怎么办？ 您必须在 RootViewController 中添加标签，然后更改您的单例以更新该标签。

如果您完全删除标签怎么办？假设你在几年后回到这个并删除标签，突然有另一个类没有编译，你可能不记得为什么。您在应用程序的一个完全独立的部分中破坏了代码。

至于遵循MVC，这违反了这一点。您的视图正在被单独的东西修改。如果 MVC 设置正确，您应该能够放置任意数量的视图，而无需更改任何控制数据的代码，在这种情况下是您的计时器。

现在至于内存管理，你的 RootViewController 现在被这个 Singleton 保留了。单例在应用程序的整个生命周期中都存在。您是对的，您在技术上没有保留周期，但是您的 RootViewController 现在永远不会被释放。在大多数应用程序中，这无关紧要，因为 RootViewController 始终保留在堆栈的底部，但这是巧合，因此无法可靠地对其进行编程。这实际上是 The Pragmatic Programmer 的另一个技巧，不要巧合编程。

解决方案

如果您确实需要像您说的那样的全局计时器，更好的解决方案是使用通知（另一个故事是您的全局计时器单例听起来不是一个好主意）。每当您从单例中更新标签时，您都可以触发通知。您的 RootViewController 将收到此通知并可以更新其标签。如果需要，您可以在通知中传递数据。现在，如果您对视图进行更改并想要更新其他内容，您只需在一个位置（RootViewController）更改代码。

于 2012-11-08T23:46:33.320 回答