Pythonを例に デフォルト引数の上手な使い方

Question

デフォルト引数を使っているときに、呼び出しがネストしすぎると逆にデフォルト引数が鬱陶しくなったり、
内部で引数の指定し忘れが起こったりしそうになります。
Pythonの場合で例をあげます。

class TestA:
    def __init__(self, num):
        self._num = num

    def calc_testA(self, n=1):
        return self._num * self._private_calc(n=n)  # この部分(n=n)が冗長かつミスが起こりやすい

    def _private_calc(self, n=1):   # つまりはここのデフォルト引数を普通の引数に直す
        return 100*n

class TestB:
    def __init__(self, num):
        self._num = num

    def calc_testB(self, num_a, n=1):
        test_a = TestA(num_a)
        return test_a.calc_testA(n=3)*self._num

def main()
    test_b = TestB(7)
    res = test_b.calc_testB(4, n=20)

このように関数が２個ネストするような場合にデフォルト引数を連続させてしまうのは悪いと考えているのですが
そういう認識で合っていますでしょうか？また、むしろデフォルト引数でも良いという考え方があるならば教えて欲しいです。

Comments

[tiitoi]

関数定義でデフォルトの値を設定するのは「デフォルト引数」といい、キーワード引数は関数を呼び出す際の指定方法の1つですが、それを混同されていませんか？

[_Victorique__]

まさにその通りです！修正します！

Answer 1

事前に確認しておきますが、
『デフォルト値の有無』と『キーワード引数の強制』は関係ないです。

>>> def func(a=0): pass
...
>>> func(1)
>>> func(a=1)
>>>

>>> def func(*, a): pass
...
>>> func(1)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: func() takes 0 positional arguments but 1 was given
>>> func(a=1)
>>>

>>> def func(*, a=0): pass
...
>>> func(1)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: func() takes 0 positional arguments but 1 was given
>>> func(a=1)
>>>

####キーワード引数が鬱陶しい件
実引数の命名が適切ならば、位置引数として取り渡せば良いかと。

####デフォルト値が鬱陶しい件
それぞれの関数／メソッドについて、デフォルト値の採択が適当か再検討すべきでしょう。
具体的な状況に依ります。

Comments

[_Victorique__]

回答有難う御座います。
def func(*, a): pass
これに対して
func(*[1], 1)
でエラーが出る理由が分かりません。このアスタリスクはどのように解釈すれば良いですか？

[LouiS0616]

関数のパラメータリストに単に * がある場合、それ以降の引数をキーワード付きで引き渡すことを強制できます。
アンパックの * とは関係ないです。

[_Victorique__]

はーなるほどです。
つまりは
def func(*, a=1):pass
の場合は
func()
とすることもできるんですね！
勉強になりました！有り難うございます！

Answer 2

鬱陶しければデフォルト値を使わないのはありだと思いますが、

n=nのような記述は（冗長とはいえ）意図するところがわかりやすく、これをやめるとかえってミスりそうなので、キーワード引数にはしておいた方が個人的には好みです。

Answer 3

関数を２つ作ったほうが明確になる気がします。

class TestA:
    def __init__(self, num):
        self._num = num

    def calc_testA(self, n=1):
        return self._num * self._private_calc(n)

    def _private_calc(self, n):
        return 100*n

    def _private_calc_default(self): # ところでこれを呼び出すサンプルケースがそもそもないのだが…
        return self._private_calc(1)

Answer 4

self._num * self._private_calc(n=n) # この部分(n=n)が冗長かつミスが起こりやすい

引数名の命名が不適切なだけではないでしょうか?

後は、LouiS0616さんの書かれたように、 ディフォルト値の採択が適切かどうか。

Comments

[_Victorique__]

確かに命名が不適切かもしれませんが、これは単純な例です。
デフォルト引数で次から次に渡していく過程が無駄？なように思えたのでこういう質問になりました。
結局はユーザーがいじれる（publicな）部分だけデフォルト引数にしてしまっても良いのではないか？という疑問が生じました。内部でもデフォルト引数で扱う必要性はあるでしょうか？位置引数で取り扱っても良いのではないでしょうか？もちろん引数名が適切であるのが大前提ですが。