Tbpgr Blog

Employee Experience Engineer tbpgr(てぃーびー) のブログ

書籍 リファクタリング−プログラマーの体質改善 | メソッド呼び出しの単純化 | メソッド名の変更

内容

適用ケース要約

メソッドの名前がその目的を正しく表現できていない

適用内容要約

メソッド名の変更

適用詳細

適切なリファクタリングを適用していくとどうしてもクラスやメソッドの粒度が小さくなり
数が増えてきます。この際に重要になるが分かりやすいメソッド名が付けられているかどうかです。

サンプル

カンマ区切りの文字列をタブ区切りの文字列に変換する機能を実装します。

サンプルコード

リファクタリング

# encoding: Shift_JIS

def change_data(data)
  return data.gsub(",","\t")
end

data = "データ1,データ2,データ3"
puts "変換前:#{data}"
puts "変換後:#{change_data data}"

リファクタリング

# encoding: Shift_JIS

def comma_to_tab(comma_string)
  return comma_string.gsub(",","\t")
end

comma_string = "データ1,データ2,データ3"
puts "変換前:#{comma_string}"
puts "変換後:#{comma_to_tab comma_string}"

出力(共通)

変換前:データ1,データ2,データ3
変換後:データ1	データ2	データ3