内部関数から外部関数の値を参照するのは安全ですか？

Question

私はNode.jsからGoに移動します。ノードで使用する構造体がGoで安全であるかどうか、同じことを達成するためにもっと慣用的な方法があるかどうかについては懸念しています。私はEchoフレームワークを使用しており、コンテキストオブジェクト内で使用できるルート固有の構造体を設定したいと考えています。私はミドルウェア内のすべての呼び出しのための構造体を生成することができますが、それを行うには高価です。代わりに、構造体を外側のfuncに1回セットし、外側のfuncの構造体を参照する内側のfuncを返します。私の希望は、一度だけ発電コストを負担してから、すべての通話に対して私のルートに関連した正しい構造を持つことです。

e.POST(path, POST.GenericPostHandler, func(next echo.HandlerFunc) echo.HandlerFunc { 
    operation := getOperationMap(path) 
    return func(c echo.Context) error { 
     c.Set("op", operation) 
     return next(c) 
    } 
}) 

このコードには何か問題がありますか？それはGCに問題を引き起こすでしょうか？同じことを達成するより効率的な方法はありますか？私は、ミドルウェアが呼び出されるたびに構造体のコピーが作成されたと仮定します。

Answer 1

operationMapは、初期化後に変化したことがない場合は、次のようなシングルトンインスタンスとしてoperationMapを宣言することができます。

package main 

import (
    "fmt" 
    "sync" 
) 

var (
    operationMapInst map[string]string // I don't know the exact type of map, so you should change the type. 
    operationMapOnce sync.Once 
) 

func getOperationMap() map[string]string { 
    // operationMapOnce.Do() runs only once 
    // when the first time getOperationMap() is called. 
    operationMapOnce.Do(func() { 
     // Initialize operationMapInst. 
     operationMapInst = map[string]string{"/": "root", "/ver": "version"} 
     fmt.Println("operaionMap has initialized!") 
    }) 

    return operationMapInst 
} 

func main() { 
    // The initialization logic runs only once. 
    // Because getOperationMap() returns map, 
    // syntax for the value for a path should be getOperationMap()[path], 
    // not getOperationMap(path). 
    rootOp, ok := getOperationMap()["/"] 
    fmt.Println(rootOp, ok) 

    // repetition 
    rootOp, ok = getOperationMap()["/"] 
    fmt.Println(rootOp, ok) 
    verOp, ok := getOperationMap()["/ver"] 
    fmt.Println(verOp, ok) 
    verOp, ok = getOperationMap()["/ver"] 
    fmt.Println(verOp, ok) 
} 

をあなたはこのコードhereを実行することができます。

Goのシングルトンパターンを理解するためにhttp://marcio.io/2015/07/singleton-pattern-in-go/をお勧めします。

Answer 2

このコードは安全でGCに問題はなく、Goで使用できる良い慣用パターンです。

この例では、operationが1つだけ作成され、ヒープに移動され、Echoで処理されるごとに要求ごとに共有されます。

すべてのリクエストを処理する際に使用される高価な構造体を初期化する必要がある場合は、このパターンを自分で使用することがよくあります。