Python標準ライブラリ内メソッドのオーバーライドは非推奨でしょうか

Question

Pythonの標準ライブラリlogging内の
debugメソッドを呼び出す際、
処理が失敗した場合に自動リトライするよう
以下のようにオーバーライドしたいと考えています。

このような使い方は非推奨でしょうか。
よろしくお願いいたします。

◾️デフォルト
def debug(self, msg, *args, **kwargs):
"""
Log 'msg % args' with severity 'DEBUG'.
To pass exception information, use the keyword argument exc_info with
a true value, e.g.
logger.debug("Houston, we have a %s", "thorny problem", exc_info=1)
"""
if self.isEnabledFor(DEBUG):
self._log(DEBUG, msg, args, **kwargs)

◾️オーバーライド後
def debug(self, msg, *args, **kwargs):
"""
略
"""
if self.isEnabledFor(DEBUG):
for i in range(2):
try:
self._log(DEBUG, msg, args, **kwargs)
except:
pass
else:
break
else:
print('logging process has failed')

Comments

[TakaiY]

非推奨かもしれないと思われた理由は何でしょうか？

[kyo_no_gohan]

理由は特にありません。Python初心者のため作法や慣習が分かっておらず、念の為確認させていただきました。きちんと動作するのであれば問題なさそうでしょうか。

Answer 1

オーバーライドということは、派生したクラスを使っているということでしょうから、全く問題はありません。

ローカルの環境にインストールされている標準ライブラリを直接改変(オーバーライドではありませんが)していたとしても問題ありません。

ちゃんと動作していなかったとしても、使いものにはならないでしょうが、他の人に何ら影響を与えていないので、これも問題ありません。

Comments

[kyo_no_gohan]

ありがとうございます！影響範囲を考えれば特に問題ないですね。ご回答、大変助かりました。

Answer 2

なにを聞きたいのかわかっていませんが、
pythonと全く関わりのない関係ないアカの他人のおっさんの意見としては、

ちゃんと動いているならば、まあいいんじゃないでしょうか。