2018-11-19 22:10
浏览 160


First of all, this is my first non-dummy program using Go. Any recommendation will be appreciated.

Code description:

I want to retrieve all the information from an API where the information is being paginated. So I want to iterate through all the pages in order to get all the information.

This is what I did so far:

I have the these two functions:

func request(requestData *RequestData) []*ProjectsResponse {

    client := &http.Client{
        Timeout: time.Second * 10,

    projects := []*ProjectsResponse{}
    innerRequest(client, requestData.URL, projects)

    return projects

func innerRequest(client *http.Client, URL string, projects []*ProjectsResponse) {

    if URL == "" {

    req, err := http.NewRequest("GET", URL, nil)
    if err != nil {
        log.Printf("Request creation failed with error %s
", err)
    req.Header.Add("Private-Token", os.Getenv("API_KEY"))

    res, err := client.Do(req)
    log.Printf("Executing request: %s", req.URL)

    if err != nil {
        log.Printf("The HTTP request failed with error %s
", err)
    data, _ := ioutil.ReadAll(res.Body)
    var response ProjectsResponse
    err = json.Unmarshal(data, &response)

    if err != nil {
        log.Printf("Unmarshalling failed with error %s
", err)

    projects = append(projects, &response)
    pagingData := getPageInformation(res)
    innerRequest(client, pagingData.NextLink, projects)

Undesired behavior:

The values in the projects []*ProjectsResponse array are being appended on each iteration of the recursion, but when the recursion ends I get an empty array list. So, somehow after the innerRequests ends, in the project variable inside the request method I get nothing.

Hope somebody and spot my mistake. Thanks in advance.

  • 写回答
  • 好问题 提建议
  • 追加酬金
  • 关注问题
  • 邀请回答

2条回答 默认 最新

  • doushou9028 2018-11-19 23:53

    I'm guessing that all of your project objects are scoped to the function so they no longer exist when the function ends. I don't think you need your projects to exist before you call innerRequest, so you should probably just have that method return the projects. I think something like this should work...

    func innerRequest(client *http.Client, URL string) []*ProjectsResponse {
    if URL == "" {
        return nil
    //More code...
    pagingData := getPageInformation(res)
    return append([]*ProjectsResponse{&response}, innerRequest(client, pagingData.NextLink)...)
    解决 无用
    打赏 举报