関数をまとめて実行する関数は作るべきか否か

Question

class A():
  @classmethod
  def _a(cls, text):
    return 処理を加えたtext
  @classmethod
  def _b(cls, text):
    return 処理を加えたtext
  @classmethod
  def _c(cls, text):
    return 処理を加えたtext
  @classmethod
  def d(cls, text):
    text = cls._a(text)
    text = cls._b(text)
    text = cls._c(text)
    return text

a,b,cそれぞれに役割があり、それを総括するとdといった関係にあるものです。
例）aは空白除去、bは改行除去、cは記号除去であり、dは不要な文字を除去する

• 上記の関数dのように他の関数をまとめて実行する関数は作るべきでしょうか？作らないべきでしょうか？
• textをメンバ変数にするべきなのでしょうか？
• 外から実行する場合、メンバ変数textに処理したい文字列を渡し、dだけ実行するべきですか？

はたまた、dを呼び出す側に作るべきでしょうか？ どのような作りにしたら良いのでしょうか？

• @classmethodばかりのclassを作るのは不適切ですか？

Answer 1

上記の関数dのように他の関数をまとめて実行する関数は作るべきでしょうか？作らないべきでしょうか？

「他の関数」の再利用性を考えている場合、単に長すぎて分割したい場合など、作っても良いケースはいろいろあると思います。

最初からまとめて書いた方が良いとか、別個に呼び出した方が良いとかも当然ケースに寄ってはあります。

textをメンバ変数にするべきなのでしょうか？

pythonではあまりメンバ変数とは言わない気がしますが、インスタンス変数と解して回答します。

インスタンス化してどういう風になるものなのかわからないので、一概に言えません。

インスタンスの属性にするべきであればインスタンス変数にしていいですし、単にメソッドの引数として渡されるべきでインスタンスが余計な情報を持つべきではないのであれば引数にしてください。

こういうケースでは呼び出される順番によって意味が変わったりするので、うっかりインスタンス変数にしてしまうと問題が起こりやすいかもしれません（変な順番で呼んでもインスタンスに属性が残っていれば実行できてしまう）。

外から実行する場合、メンバ変数textに処理したい文字列を渡し、dだけ実行するべきですか？
はたまた、dを呼び出す側に作るべきでしょうか？ どのような作りにしたら良いのでしょうか？

これは意味がよくわかりません。

@classmethodばかりのclassを作るのは不適切ですか？

不適切とまでは言えませんが、変だと思います。インスタンスの機能を使う気がなさそうに見えるので、こういうケースであれば単にトップレベル関数を定義したモジュールにすれば済むと思います。クラスにしなくて済むものはクラスにしない方が見通しがよくなります。

Answer 2

上記の関数dのように他の関数をまとめて実行する関数は作るべきでしょうか？作らないべきでしょうか？

textをメンバ変数にするべきなのでしょうか？

外から実行する場合、メンバ変数textに処理したい文字列を渡し、dだけ実行するべきですか？
はたまた、dを呼び出す側に作るべきでしょうか？ どのような作りにしたら良いのでしょうか？

ケースバイケースですが、提示されたコードが何かおかしいということはないと思います。

@classmethodばかりのclassを作るのは不適切ですか？

特に問題はないと思います。

Answer 3

上記の関数dのように他の関数をまとめて実行する関数は作るべきでしょうか？

必要と考えたら、作るべきです。
自分的な基準としては、そのような使い方が多いと思えば、作ります。
ただ、細かな関数を多数作ると、メンテナンスが大変そうなので、その辺も考慮します。

この辺は設計の話になるので、どんなシステムを作るか、にも依存するので、簡単には答えられないですが。