属性を付ける為だけにクラスでまとめるのはあり？

Question

pythonで変数や関数を属性を付ける為だけにクラスでまとめるのはありだと思いますか？
以下一例です。

def read_txt():
    ...

def readlines_txt():
    ...

def creat_txt():
    ...


class txt:
    def read():
        ...

    def readlines():
        ...

    def creat():
        ...

PATH_ONE = "PATH_1"
PATH_TWO = "PATH_2"
PATH_THREE = "PATH_3"


class path:
    ONE = "PATH_1"
    TWO = "PATH_2"
    THREE = "PATH_3"

見てわかるように、インスタンス変数がないので、属性を付ける為だけにクラス化をしていますが、pythonではあまりこういうコードは見かけないような気がするので、好ましくないのかなと思い悩んでいます。競合のしにくさと引き換えに冗長になったり、ネストが深くなるので。

自分だったらする、しないだけでもいいので、意見をお聞きしたいです。

Comments

[quickquip]

上の方のコードですが、passと書いてある部分が「本当は実装が入るのだけどここでは省略」という意図なのか、本当にpassと書いたコードを書いておきたいだけなのかパッと区別がつきません。前者であればEllipsisオブジェクト https://docs.python.jp/3.6/library/stdtypes.html#the-ellipsis-object のリテラル ... を書く方が慣習にあっているかと思います。

[puroko3]

なるほど、pass以外は知りませんでした。修正しておきます。

Answer 1

アリだと思いますが、求められる場面は案外少ないように思います。

• 属性名を付けることにより、意味付けを図りたい場合

namedTupleや、辞書の方が適切です。

• 複数の対等な属性をまとめたい場合

enumの方が適切です。

• 他言語でいう名前空間を利用したい場合

Pythonではモジュールで同様の機構を表現できます。

Comments

[puroko3]

回答ありがとうございます！
純粋に名前空間を使うという意味合いが大きかったのですが、やはりモジュールを使った方が良さげですね...
enumは知らなかったのでとても勉強になりました。
また機会があればお願いします。

Answer 2

クラスではなくtxtやpathモジュールにまとめる方が、よりフラットになるので良い気がします。

Comments

[puroko3]

回答ありがとうございます！
ふむ、やはりパッケージやファイルにまとめた方がよさげですか...
モジュールの方がいいという意見が多いので、モジュール分割中心で考えていきたいと思います。
また機会があればお願いします。

Answer 3

class path:のコードの方だけ！

クラス名と値を見る限りでは外部設定情報(json)などに持つ形の方が良いと思いますが。

クラスで定義すると重複定義ミスをしてもIDEで警告がでるだけですが、
enumのuniqueデコレータを使用すると、実行時例外(TypeError: Attempted to reuse key:)を送出できます。

# -*- coding: utf-8 -*-
from enum import Enum, unique


@unique
class FilePath(Enum):
    ONE = "PATH_1"
    TWO = "PATH_2"
    THREE = "PATH_3"


def main() ->None:
    print(FilePath.ONE.value)
    print(FilePath.TWO.value)
    print(FilePath.THREE.value)


if __name__ == '__main__':
    main()

Comments

[puroko3]

回答ありがとうございます！
定数をまとめる時は、enumの方が良さそうですね。uniqueデコレータも勉強になりました。
次から使っていこうと思います。
また機会があればお願いします。