需要在全局范围内还是在本地范围内？

Question

要求节点模块的正确方法是什么？在全局范围内声明模块更准确/易于理解，还是在本地范围内声明模块更准确/易于理解？

例如，以下哪个最有意义：

全球：

let dns = require('dns') // <-- Global scope declaration
function lookup(subdomain, domain){
    let fqdn
    if(subdomain == '@'){
        fqdn = domain
    } else {
        fqdn = `${subdomain}.${domain}`
    }
    dns.resolveTxt(fqdn, (err, records) => {
        if(records){
            console.log(records)
        } else {
            console.log("no recrods")
        }
    })
}

本地：

function lookup(subdomain, domain){
    let dns = require('dns') // <-- Local scope declaration
    let fqdn
    if(subdomain == '@'){
        fqdn = domain
    } else {
        fqdn = `${subdomain}.${domain}`
    }
    dns.resolveTxt(fqdn, (err, records) => {
        if(records){
            console.log(records)
        } else {
            console.log("no recrods")
        }
    })
}

这是一个意见问题吗？如果是这样，我很抱歉，我会删除这个问题。

我希望改进我的代码，以便其他人更容易理解它，我认为这个问题与此目的相关，因此不应将其视为一个固执己见的问题。

Answer 1

首先，一个小的修正。您的第一个示例不是全局范围。那是模块范围。声明位于模块文件开头的模块范围是首选实现，原因如下：

1. 它在启动时加载（请参阅下面的文字了解为什么这样做很好）。
2. 将所有这些内容放在模块文件的开头，以一种易于查看的方式清楚地在代码中说明了您拥有哪些外部模块依赖项，这对于将来返回此代码进行维护的任何人（包括您)。
3. 你在这个模块中只require()这个模块一次。虽然缓存了模块，但最好不要在每次要引用它时都调用require()。加载一次并使用保存的模块引用。

关于上面的第 1 点，require() 是阻塞和同步的，并且涉及访问文件系统（第一次加载该文件时）。您通常希望在服务器启动时完成所有 require() 操作，因此在处理请求时不会发生阻塞操作。而且，如果您从require() 收到任何错误（例如模块安装丢失或版本冲突或错误配置或缺少依赖项或类似的东西），您还希望这些错误发生在启动时，而不是稍后发生的更快的地方更容易诊断。