如果可以提高代码的清晰度，那么调用一个没有效果的函数是否更好？

我的程序（iOS应用程序）中有三个视图。它们中只有一个同时处于活动状态，因此我关闭了其中两个的可见性，并在用户按下按钮时切换可见性。视图被初始化为可见，因此我在显示主视图之前在代码中设置了可见性。

我可以

[view1 setAlpha:0.0f];
[view2 setAlpha:0.0f];

对于其中两个视图，但现在未解决第三个视图（应在应用程序开始时显示的视图）。我把

[view3 setAlpha:1.0f];

在前两个视图之后，因为我认为这很清楚，实际上存在三个视图，而不是看到代码时可​​能会想到的两个视图。其他程序员如何做到这一点？纯粹是偏爱还是有一些约定？

如果调用非常繁琐，那么最好不要在不需要时调用它，但是我想知道像我的示例这样的小事情。

您有一个不变式：

只有一个视图（3个视图）处于活动状态（并且可见）。

然后，我建议您提供一个功能，以一次切换所有视图的活动和可见性：

[setActiveView viewID:2]

该功能将：

• 检查视图是否已经处于活动状态，避免不必要的工作
• 将视图设置为活动且可见
• 将其他2个视图设置为不活动和不可见

原始调用有很多优点setVisibility：

• 友好的：不必要地调用它不会造成性能问题
• 防御性的：它的单个参数很难修改，setVisibility但是很难记住值的范围是，0.0f - 1.0f并且必须将其设置为1.0f
• 有弹性：下一个家伙不会偶然忘记其中一种观点
• 适应性强：添加/删除视图不需要仔细检查所有应用程序代码来查找开关在哪里，只需更新一个功能（此功能）

理想情况下，为了帮助强制执行不变式，其他任何函数都不能弄乱此设置...

替代主意：如果您的目标是防止由于人们忘记了三个视图而只对其中两个视图执行某项操作而导致错误发生，而这三个视图实际上应该与所有视图一起使用，那么请创建一个使它无法忘记的函数：

setViewVisibilities(0.0f, 0.0f, 1.0f)

现在您有了更强大的功能- 编译时的保证不会遗忘。如果忘记了一个参数，编译器会大吼大叫。这比注释或不必要的代码有用得多，因为它会创建一个严格的命名协议来强制执行您关心的属性。

对于view3不需要更改可见性的情况，您可以添加一些行为，在这些行为中传递诸如-1.0或的特殊值nil表示“根本不更改视图可见性”。这避免了不必要地设置可见性的问题。

我认为最好添加一条注释来解释该呼叫是不必要的（以及原因）。

^{（也许，不需要呼叫或您需要对此进行注释的事实可能是代码的味道）}

在这种情况下，@ Mattieu M.有正确的解决方案。

在更普遍的情况下，那里是没有类似的变换，你要问自己：是否有任何机会，未来的程序员可能会搞砸？

答案通常是。这意味着，是的，您应该添加呼叫。也许框架的某些将来版本以所有视图关闭而不是打开开始。