메서드 추출(Extract Method)

목적

기본 작업 사이클

메소드 추출(Extract Method)이란?

포인트

예

def scraiping
  # ログインする
  driver.navigate.to('https://サイト')
  driver.find_element(:id, 'id').send_keys('neko')
  driver.find_element(:id, 'password').send_keys('inu')
  driver.find_element(:css, 'li.loginBt > a').click

  # ダウンロードする
  driver.find_element(:css, 'input[value="ダウンロード（CSV）"]').click

  # ログアウトする
  driver.find_elements(:css, 'a').find { |element| element.text == 'ログアウト' }.click
end

def scraiping
  login
  download
  logout
end

def login
  driver.navigate.to('https://サイト')
  driver.find_element(:id, 'id').send_keys('neko')
  driver.find_element(:id, 'password').send_keys('inu')
  driver.find_element(:css, 'li.loginBt > a').click
end

def download
  driver.find_element(:css, 'input[value="ダウンロード（CSV）"]').click
end

def logout
  driver.find_elements(:css, 'a').find { |element| element.text == 'ログアウト' }.click
end

def scraiping
  # ログインする
  url = 'https://サイト'
  driver.navigate.to(url)
  driver.find_element(:id, 'id').send_keys('neko')
  driver.find_element(:id, 'password').send_keys('inu')
  driver.find_element(:css, 'li.loginBt > a').click

  download
  logout
end

def scraiping
  url = 'https://ふつうのサイト'
  # 単に抽出しない
  login(url)
  download
  logout
end

def login(url)
  # 仮に抽出元で後続で変数を使用していなければ引数なしで一時変数としておけば良い
  # url = 'https://サイト'

  driver.navigate.to(url)
  driver.find_element(:id, 'id').send_keys('neko')
  driver.find_element(:id, 'password').send_keys('inu')
  driver.find_element(:css, 'li.loginBt > a').click

  # 仮に抽出元で変数を書き換えたものを後続で使用する場合は引数を返す必要がある
  # url = 'https://サイト/1' if (driver.find_element(:id, 'next'))
end