app2641
2015/11/24 15:41:45 投稿
0

より正確な名前付け

<p>before では refactoring という漠然とした名前の変数を使用していたが、someone_refactroing と名前を変えて誰のリファクタリングが分かるように名前付けをした <img alt="sushi" src="/assets/emoji/unicode/1f363-831097849a370c02369283b17255c6e4f0ffe9e0a1269610119e49b26401f621.png" style="vertical-align:middle" width="20" height="20" /></p>

Before

describe 'refactoring test' do
  let(:my_refactoring) { build_stubbed :refactoring, user: user }
  let(:refactoring) { build_stubbed :refactoring }

  # do something....
end

After

describe 'refactoring test' do
  let(:my_refactoring) { build_stubbed :refactoring, user: user }
  let(:someone_refactoring) { build_stubbed :refactoring, user: (build_stubbed :user) }

  # do something....
end

みんなのコメント

fujimura
fujimura
2015/11/25 11:23:34 投稿
<p><code>someones_refactoring</code> にして <code>my</code> と格を揃えるとより良いかと。</p>
app2641
app2641
2015/11/25 12:03:00 投稿
<p>なるほど、確かにそちらのほうが良さそうですね!<br> ありがとうございます <img alt="sushi" src="/assets/emoji/unicode/1f363-831097849a370c02369283b17255c6e4f0ffe9e0a1269610119e49b26401f621.png" style="vertical-align:middle" width="20" height="20" /></p>