Ruby +递归函数+定义全局变量

Question

我正在使用Ruby来提取bitbucket repo列表.来自bitbucket的响应将只包含10个存储库和下一页的标记,其中将有另外10个存储库等等...(他们称之为分页)

所以,我编写了一个递归函数,如果存在下一个页面标记,它会调用自身.这将一直持续到最后一页.

这是我的代码:

#!/usr/local/bin/ruby
require 'net/http'
require 'json'
require 'awesome_print'

@repos = Array.new

def recursive(url)

    ### here goes my net/http code which connects to bitbucket and pulls back the response in a JSON as request.body 
    ### hence, removing this code for brevity

    hash = JSON.parse(response.body)

    hash["values"].each do |x|
        @repos << x["links"]["self"]["href"]
    end

    if hash["next"]
        puts "next page exists"
        puts "calling recusrisve with: #{hash["next"]}"
        recursive(hash["next"])
    else
        puts "this is the last page. No more recursions"
    end
end

repo_list = recursive('https://my_bitbucket_url')
@repos.each {|x| puts x}

现在,我的代码工作正常,它列出了所有的回购.

问题: 我是Ruby新手,所以我不确定@repos = Array.new上面使用全局变量的方式.如果我在函数中定义数组,那么每次调用该函数都会创建一个新数组,覆盖之前调用的内容.

那么,在这种情况下,Ruby程序员如何使用全局符号.我的代码是否遵守Ruby道德规范,或者它是非常业余的东西(但是正确的,因为它有效)这样做的方式.

Answer 1

共识是尽可能避免全局变量.

我会像这样递归地构建集合:

def recursive(url)
  ### ...

  result = [] 

  hash["values"].each do |x|
    result << x["links"]["self"]["href"]
  end

  if hash["next"]
    result += recursive(hash["next"])
  end
  result
end

或将集合移交给函数:

def recursive(url, result = [])
  ### ...

  hash["values"].each do |x|
    result << x["links"]["self"]["href"]
  end

  if hash["next"]
    recursive(hash["next"], result)
  end
  result
end

无论哪种方式,您都可以调用该功能

repo_list = recursive(url)

我会这样写:

def recursive(url)
  # ...

  result = hash["values"].map { |x| x["links"]["self"]["href"] }
  result += recursive(hash["next"]) if hash["next"]
  result
end