ruby - 重做条件的正确红宝石方法是什么？

Question

任务是检查联系人页面是否存在并导航到该页面。对于不是英文的网站，该方法会查找英文页面，然后重新启动以检查联系页面。

我的条件工作正常，但我认为必须有更好的方法来做到这一点：

  # First, I set the @url variable during Booleans.   
  # Checks are either to see if a link exists or if a page exists,
  # (aka no 404 error).
  #
  # Here are two examples:

  # Boolean, returns true if contact link is present.
  def contact_link?
    @url = link_with_href('contact')

    !@url.nil?
  end

  # True if contact page '../contact' does NOT get a 404 error.
  def contact_page?
    @url = page.uri.merge('../contact').to_s
    begin
      true if Mechanize.new.get(@url)
    rescue Mechanize::ResponseCodeError
      false
    end
  end

  # #
  # Now go to the correct page, based off of checks.
  #
  def go_to_contact_page
    1.times do
      case # No redo necessary.
      when contact_link? # True if hyperlink exists
        get(@url)
      when contact_page? # False if 404 error
        get(@url)
      else # Redo is now necessary.
        if english_link? # True if hyperlink exists
          get(@url)
          redo
        elsif en_page? # False if 404 error
          get(@url)
          redo
        elsif english_page? # False if 404 error
          redo
        end
      end
    end
  end

有几件事需要引起您的注意：

1. 最好的办法是1.times do做单redo吗？会begin更好吗？

2. 了解到我在每个检查中都设置了变量，条件分支中@url似乎存在冗余。get(@url)有没有更简洁的方法？

3. 我写redo了三遍，这似乎也是多余的。有没有办法调用它一次并仍然设置@url变量？

谢谢您的帮助！

Answer 1

像这样的东西更具可读性和干性

def english_contact_page
  ..
rescue
  nil
end

def contact_page
  ..
rescue
  nil
end

def get_page
  @url = link_with_href('contact')
  return nil if @url.nil?
  contact_page || english_contact_page  # left side is evaluated first
rescue
  nil
end