Golang JSON-Decodierung kann Schnittstelle nicht decodieren {}

Question

Ich benutze eine Bibliothek (Go-kit), die erfordert, dass ich Funktionen zum Codieren/Decodieren meiner Anfrage und Antworttypen zu/von JSON angeben. Für die Codierung ist es einfach:

Ich übergeben diese Funktion, um den HTTP-Server zu erstellen, und es funktioniert gut. Doch für Anfragen ihr vorgeschlagenen Ansatz ist eine separate Funktion der Form zu machen:

func decodeUppercaseRequest(_ context.Context, r *http.Request) (interface{}, error) { 
    var req UppercaseRequest 
    if err := json.NewDecoder(r.Body).Decode(&req); err != nil { 
     return nil, err 
    } 
    return req, nil 
} 

Für jede einzelne RPC in meiner Anwendung. Ich würde meinen Code DRY gerne behalten und Hunderte von fast identischen Methoden vermeiden. Als solche habe ich versucht, eine Funktion zu schreiben Schließungen zu erzeugen, die die gegebene Anforderungstyp dekodieren:

func DecodeRequest(req interface{}) httptransport.DecodeRequestFunc { 
    return func(_ context.Context, r *http.Request) (interface{}, error) { 
     if err := json.NewDecoder(r.Body).Decode(&req); err != nil { 
      return nil, err 
     } 
     return req, nil 
    } 
} 

Diese Funktion kann wie so aufgerufen werden:

DecodeRequest(UppercaseRequest{}} 

Leider, wenn ich so, die JSON Die Decodierung schlägt fehl, obwohl der Typ von req tatsächlich mypackage.UppercaseRequest ist. Ich bin mir nicht sicher, wohin ich von hier aus gehen soll. Kann ich es vermeiden, eine Methode pro Anfragetyp schreiben zu müssen? Gibt es eine Möglichkeit, wie ich der Decode-Funktion helfen kann zu verstehen, was dieser Typ zur Laufzeit ist? Danke im Voraus!

ist hier ein Spielplatz gehen das Problem demonstriert: https://play.golang.org/p/GgHsLffp1G

Answer 1

Nach dem Stück Code, den Sie zeigen uns, glaube ich Ihnen eine Art Behauptung Problem konfrontiert sind. Ich habe eine playground to show you what I explain below erstellt.

Sie übergeben ein UpperCaseRequest an DecodeRequest func. In diesem func ist das Argument vom Typ interface {} und übergibt einen Zeiger dieses Arguments an den json Decoder. Der Decoder sieht also einen Zeiger auf die Schnittstelle und keinen Zeiger auf UpperCaseRequest.

Deshalb ist es nicht richtig decodiert. Und dann, eine Typprüfung auf es zu versuchen, schlägt fehl, weil zwei verschiedene Typen nicht möglich ist.

Also, in Ihrem Code, würde ich vorschlagen:

func DecodeRequest(req interface{}) httptransport.DecodeRequestFunc { 
    return func(_ context.Context, r *http.Request) (interface{}, error) { 
     // Note the '&' is removed here 
     if err := json.NewDecoder(r.Body).Decode(req); err != nil { 
      return nil, err 
     } 
     return req, nil 
    } 
} 

Und rufen Sie diese Funktion wie folgt aus:

// Note the & is placed here. 
DecodeRequest(&UppercaseRequest{}}