class内のメソッドを別のメソッドで使いたいが，インタンス変数は変更したい場合

実現したいこと

以下のサンプルコードのようなことを実現したいです．題名にもあるように，同一クラス内で別のメソッドをインスタンス変数を変えて利用したい場合，どのように書くのが正しいでしょうか．

class名を書くと再利用性が低下するのでなるべく避けたいのですが．

サンプルコード

python
1class Hoge:
2  def __init__( self, x ):
3    self.x = x
4  
5  def times2( self ):
6    return self.x * 2
7  
8  def times2_plus2( self, y ):
9    """
10    times2を利用したいが，インスタンス引数は別の値を使いたい
11    """
12    #NOTE: 本来ここでclass名を書きたくない．self.times2()としたいところだが,
13    return Hoge(y).times2() + 2
14
15Hoge(2).times2_plus2( 9 )
16# 9 * 2 + 2 = 18
17>>> 20

環境

python 3.6.1
MacOS 10.15.3

よろしくお願いします．

行動規範の内容に同意します

回答1件

ベストアンサー

自身のインスタンス変数と関係ない times2_plus2() がクラス Hoge のメソッドとして定義されているのは変な気がします。
フリー関数として定義してはどうでしょうか

python
1def times2_plus2(y):
2    return y.times2() + 2
3
4class Hoge:
5    def __init__(self, x):
6        self.x = x
7
8    def times2(self):
9        return self.x * 2
10
11times2_plus2(Hoge(9))
12# 20

補足

より汎用的なクラスを設計しようと思った場合、2倍にする関数だけでなく、3倍、4倍と任意の値を乗算したい場合も出てくるかもしれないので、演算子オーバロードを使うといいかもしれません。

Python Tips：自作クラスの演算子のふるまいを定義したい - Life with Python

python
1class Hoge:
2    def __init__(self, x):
3        self.x = x
4
5    def __mul__(self, other):
6        return Hoge(self.x * other)
7
8    def __add__(self, other):
9        return Hoge(self.x + other)
10
11    def __imul__(self, other):
12        self.x *= other
13        return self
14
15    def __iadd__(self, other):
16        self.x += other
17        return self
18
19    def __str__(self):
20        return str(self.x)
21
22
23x = Hoge(9)
24y = x * 2
25z = x * 2 + 2
26print(y)
27print(z)
28x += 2
29print(x)  # 11
30x *= 2
31print(x)  # 22