《36》リファクタリング:異なるエラー処理〈Python 2.x 版〉
実録:はじめてのプログラミング《記事一覧》
《36》リファクタリング:異なるエラー処理
《著》小粒ちゃん+α《監修》小泉ひよ子とタマゴ倶楽部
2009年3月4日(水)
関連記事
今日の進捗
- Language Reference: Numeric and Mathematical Modules; functools — Higher order functions and operations on callable objects
- Python.use(better) -- セミナー研修テキスト
- 連結リスト課題を「続・ひよ子のきもち」で公開
Comment | |
---|---|
本人:野中 | リファクタリングを繰り返すうちに全体が把握できるようになって慣れかもしれませんね。 |
担当:伊藤/本間 | 。(^^) |
何が問題か:異なるエラー処理
(前述した)2つのメソッド __delitem__/remove では、
def __delitem__(self, key): ... while node: ... else: s = self.__class__.__name__ raise IndexError,"%s assignment index out of range"%s def remove(self, value): ... while node: ... else: s = self.__class__.__name__ raise ValueError,"%s.remove(x): x not in %s"%(s,s)
while に続く else ブロックで発生させる例外(IndexError/ValueError)が異なります。そこで、リファクタリングを実施して、これらの異なるコードの断片を抽出して、独立したメソッドを用意します。
リファクタリング:異なるエラー処理
新たなメソッドを抽出するときには、もとの局所変数を、新たに引数として再定義するのが基本です。ここでは、1つの局所変数 s が対象になります。すると、
def _removeNode_delitem_error(self, s): raise IndexError,"%s assignment index out of range"%s def _removeNode_remove_error(self, s): raise ValueError,"%s.remove(x): x not in %s"%(s,s)
各メソッドに固有の例外とメッセージの内容だけを記述すればいいのが分かります。すると、
class myList(object): def removeNode(self, value, tag): ... while node: ... else: s = self.__class__.__name__ eval("self._removeNode_%s_error(s)"%tag)
新たな引数 tag を導入するだけで、メソッドの名前をその状況に合わせて変化させられるようになります。つまり、呼び出すメソッドを実行時に確定させるのです。