Forkwell_ja
Forkwell_ja
2015/12/10 13:53:47 投稿
0

【Refactor Me】split(正規表現)がcontrollerにあるのはなんとなく嫌だな...

リファクタリングのネタが思いつかない方のために、beforeコードを用意しました。

このbeforeコードをリファクタリングしてくれた方にも sushi のチャンスがあります!

beforeコードをコピーして新規投稿を作成してね!


このままでもいいんだけど、もう少し抽象化できそう...

Before

class SushisController
  def neta_list
    @neta_list ||= params[:sushi][:neta].split(/[\s[[:blank:]]]+/)
  end

  def params
    { sushi: { neta: 'アナゴ 赤貝 コハダ' }}
  end
end

require 'minitest/autorun'

class TestSushisController < MiniTest::Test
  def setup
    @sushi_controller = SushisController.new
  end

  def test_neta_list
    assert_equal ['アナゴ', '赤貝', 'コハダ'], @sushi_controller.neta_list
  end
end

After

# your cool code

みんなのコメント