javascript - 契约公约

Question

如果每个方法和变量只会被使用一次，哪种约定更正确？

根据需要创建尽可能多的变量:

var x = GmailApp.getInboxUnreadCount();
var email = GmailApp.getInboxThreads (0, x);

在一行中编写代码:

var email = GmailApp.getInboxThreads (0, GmailApp.getInboxUnreadCount());

Answer 1

后者，在合理范围内。但这主要是简单情况下的风格问题。

这意味着如果您有一个函数调用需要 10 个参数，并且每个参数都来自一个大型函数调用本身，请不要这样做。想一想谁会得到你的代码库。

“这行代码使用较少的局部变量是否可读？或者我是否需要将其分解以更好地说明这行代码在做什么？”是你应该问自己的问题。在这种情况下，后者是完全可读的。

---

事实上，我认为第一个示例由于无用的局部变量名称 x 而可读性较差。如果取而代之的是更好地命名，它可能是一个更可行的选择。

var unreadCount = GmailApp.getInboxUnreadCount();
var email = GmailApp.getInboxThreads (0, unreadCount);

这样更好，但在这种非常简单的情况下仍然没有必要。