如何讓此Go代碼更幹？

Question

我正在實現一個REST API的Go包裝器。它基本上解析JSON並且應該返回適​​當的結構類型。我發現自己做了很多這樣的：

// GetBlueprintDetails returns details about a blueprint 
func (c *Client) GetBlueprintDetails(projectID string, blueprintID string) (*BlueprintDetails, *APIError) { 
    path := fmt.Sprintf("projects/%s/blueprints/%s", projectID, blueprintID) 
    res, err := c.Request("GET", path, nil, nil) 
    if err != nil { 
     return nil, err 
    } 
    var ret BlueprintDetails 
    e := json.Unmarshal(res.Body, &ret) 
    if e != nil { 
     return nil, &APIError{Error: &e} 
    } 
    return &ret, nil 
} 

// GetProjects returns a list of projects for the user 
func (c *Client) GetProjects() (*[]Project, *APIError) { 
    res, err := c.Request("GET", "projects", nil, nil) 
    if err != nil { 
     return nil, err 
    } 
    var ret []Project 
    e := json.Unmarshal(res.Body, &ret) 
    if e != nil { 
     return nil, &APIError{Error: &e} 
    } 
    return &ret, nil 
} 

兩個功能之間的唯一區別是取消封送的結構基本類型。我知道在Go中沒有泛型，但必須有一種模式才能使其更加乾爽。

任何想法？

Answer 1

您可以創建一個MakeRequest函數，它的HTTP請求部分，並解組的JSON爲結構

這裏是你如何可以做到這一點，看看在MakeRequest功能

// GetBlueprintDetails returns details about a blueprint 
func (c *Client) GetBlueprintDetails(projectID string, blueprintID string) (*BlueprintDetails, *APIError) { 
    path := fmt.Sprintf("projects/%s/blueprints/%s", projectID, blueprintID) 
    bluePrintDetails = new(BlueprintDetails) 
    err := c.MakeRequest("GET", path, bluePrintDetails) 
    return bluePrintDetails, err 
} 

// GetProjects returns a list of projects for the user 
func (c *Client) GetProjects() (*[]Project, *APIError) { 
    projects = make([]Project, 0) 
    err := c.MakeRequest("GET", "project", &projects) 
    return &projects, err 
} 

func (c *Client) MakeRequest(method string, path string, response interface{}) *APIError { 
    res, err := c.Request(method, path, nil, nil) 
    if err != nil { 
     return nil, err 
    } 
    e := json.Unmarshal(res.Body, response) 
    if e != nil { 
     return &APIError{Error: &e} 
    } 
    return nil 
}