Go 中显式范围块的目的？

Question

我正在阅读 MicroMDM SCEP 存储库中的此源代码，https://github.com/micromdm/scep/blob/1e0c4b782f3f2e1e6f81da5f82444a6cedc89df3/cmd/scepclient/scepclient.go#L54-L65：

func run(cfg runCfg) error {
    ctx := context.Background()
    var logger log.Logger
    {
        if strings.ToLower(cfg.logfmt) == "json" {
            logger = log.NewJSONLogger(os.Stderr)
        } else {
            logger = log.NewLogfmtLogger(os.Stderr)
        }
        stdlog.SetOutput(log.NewStdlibAdapter(logger))
        logger = log.With(logger, "ts", log.DefaultTimestampUTC)
        if !cfg.debug {
            logger = level.NewFilter(logger, level.AllowInfo())
        }
    }
    lginfo := level.Info(logger)

我想知道显式块（外部{ ... }）的目的是什么？这段代码会不会和它们被删除时不完全一样，比如

func run(cfg runCfg) error {
    ctx := context.Background()
    var logger log.Logger
    if strings.ToLower(cfg.logfmt) == "json" {
        logger = log.NewJSONLogger(os.Stderr)
    } else {
        logger = log.NewLogfmtLogger(os.Stderr)
    }
    stdlog.SetOutput(log.NewStdlibAdapter(logger))
    logger = log.With(logger, "ts", log.DefaultTimestampUTC)
    if !cfg.debug {
        logger = level.NewFilter(logger, level.AllowInfo())
    }
    lginfo := level.Info(logger)

也许显式块只是为了提高可读性？

Answer 1

在这种情况下，额外的块似乎没有任何意义。块内没有声明任何变量。它不会增加清晰度，反而会让您感到困惑。

如果需要清晰，您可以将该代码提取到一个新函数中以初始化记录器。

func initLogger(cfg runCfg) log.Logger {
    var logger log.Logger

    if strings.ToLower(cfg.logfmt) == "json" {
        logger = log.NewJSONLogger(os.Stderr)
    } else {
        logger = log.NewLogfmtLogger(os.Stderr)
    }
    stdlog.SetOutput(log.NewStdlibAdapter(logger))
    logger = log.With(logger, "ts", log.DefaultTimestampUTC)
    if !cfg.debug {
        logger = level.NewFilter(logger, level.AllowInfo())
    }

    return logger
}

func run(cfg runCfg) error {
    ctx := context.Background()
    logger := initLogger(cfg)
    lginfo := level.Info(logger)
    ...

我最好的猜测是这个块在过去曾有过某种用途，而更改代码的人并没有将其删除，也可能不确定是否仍然有用途。翻看这个函数的责备日志或许会给你一个答案。

Answer 2

花点时间阅读其余代码，这些块是all over the code。但是，在这些情况下，声明了 短期变量，这是一种在不再需要它们后将其丢弃的方法。两种最有可能的情况是记录器可能有一些，或者它只是开发人员选择的特定样式。实际上，两者都可能是真的。

这对 Go 来说有点奇怪，但它可以有效地明确告诉垃圾收集器以及开发人员这些已经超出范围。然而，Go 编译器和 GC 在撰写本文时已经足够先进，可以知道何时可以丢弃它们，因此除了整理当前作用域的命名空间之外，程序本身几乎没有什么好处。

虽然我同意@Schwern，但它增加了清晰度并实现了相同的结果，将这些重构为自己的功能。如果有人花时间明确声明作用域块，那么为什么不利用这段时间来使它们发挥作用呢？

链接代码块：

var svc scepserver.Service // scep service
{
    svcOptions := []scepserver.ServiceOption{
        scepserver.ChallengePassword(*flChallengePassword),
        scepserver.WithCSRVerifier(csrVerifier),
        scepserver.CAKeyPassword([]byte(*flCAPass)),
        scepserver.ClientValidity(clientValidity),
        scepserver.AllowRenewal(allowRenewal),
        scepserver.WithLogger(logger),
    }
    svc, err = scepserver.NewService(depot, svcOptions...)
    if err != nil {
        lginfo.Log("err", err)
        os.Exit(1)
    }
    svc = scepserver.NewLoggingService(log.With(lginfo, "component", "scep_service"), svc)
}

var h http.Handler // http handler
{
    e := scepserver.MakeServerEndpoints(svc)
    e.GetEndpoint = scepserver.EndpointLoggingMiddleware(lginfo)(e.GetEndpoint)
    e.PostEndpoint = scepserver.EndpointLoggingMiddleware(lginfo)(e.PostEndpoint)
    h = scepserver.MakeHTTPHandler(e, svc, log.With(lginfo, "component", "http"))
}