pythonで定数から値を取得して表示したい

Question

プログラム初心者です。
pythonで定数に一致したメッセージを出力するプログラムを作成しています。
dictのような結果を出力したいのですが、
name 'messages' is not definedが発生してうまくいきません。
対応方法をご教示願います。

class ErrorClass:
    messages = {
        'E400001': '桁数がおかしいです',
        'E400002': '型がおかしいです',
        'E400003': '値が存在しておりません。'
    }

    def set_errorMassage(code): 
        if code in messages:
            self.error_info = {'code': code, 'message': messages['code'],'httpcode': code[1:4]}
        else:
            error_info = {'code': 'E400999', 'message': 'この操作は実行できません。','httpcode': '500'}
        return self.error_info
    
    def get_errorMassage():
        if self.error_info is not null:
            error_info = {'code': '', 'message': '','httpcode': '200'}
        return self.error_info 

if __name__ == "__main__":
    set_errorMassage('E400001')
    print(get_errorMassage)

//set_errorMassage('E400001')
{'code': 'E400001', 'message': '桁数がおかしいです','httpcode': '400'}
//ちなみにset_errorMassageがsetされない場合<エラーがない時>
{'code': '', 'message': '','httpcode': '200'}
//よくわからないエラーがでたとき
{'code': 'E400999', 'message': 'この操作は実行できません。','httpcode': '500'}

Answer 1

関数set_errorMassageにself引数を持たせ、messagesをself.messagesに編集すればよいと思われます。
また、関数名はset_error_messageのほうが良いというか、正しいです。Massageというとマッサージです。

Answer 2

messagesはErrorClassクラスの中に入っているので、ErrorClass経由でしか使うことができません。
なので、messagesと書かれていても、未定義ということになります。

• ErrorClassは不要ではないか？

この処理だけみると、ErrorClassは不要です。 messagesをトップルベルに出せば使えるようになります。

• ErrorClassが必要な場合

このクラスが必要なのであれば、クラス変数としてアクセスする必要があります。
コードの中のmessagesを ErrorClass.messages に置き換えればOKです。

Comments

[pokemonta]

このErrorClassは、共用資産として、他のpythonから使う想定でいます。
他のpythonのエラー発生個所でset_errorMassageを使います。
この場合、Class経由じゃないと無理だと思っていますがいかがでしょうか？
他のpythonで当該classをimportする。

[TakaiY]

クラスでなくても、モジュールとして提供して、定数を定義することは問題なくできますよ。

ただ、改めてコードを見てみると、get_errorMessage は保存してあるエラーメッセージから情報を取り出すことになっているので、何らかの形でメッセージを保存する必要があるので、であれば、クラス化してそのデータを保持するようにしたほうがいいかもしれません。

クラスをインスタンス化して、インスタンス毎にメッセージを持てるようにするか、プロセス毎に1つでよければクラス変数でもいいでしょう。

クラスにするのであれば、set_errorMessage も get_errorMessage もクラスに含めるべきなので、インデントが足りません。
(よくみれば、selfを使っているのでそもそもそこが間違えているのかもしれませんね)

[pokemonta]

set_errorMassage、get_errorMassageは、
ErrorClassのメソッドとして記載したつもりでしたが
インデントのせいでクラス外と判断していたのですね。
ただ、set_errorMassageを右にずらしても(半角4つずらいしても)messageを参照できませんでした。
messageをクラス直下には置けないのでしょうか？

クラス変数にすべきかインスタンス変数にすべきかですが
私の理解では、ErrorClassに対して、様々なプログラムからエラーメッセージを
参照しにきますので、それごとにerror_info の内容が異なると想定しています。
つまりerror_info を共有しないのでインスタンス変数を採用しようと思っています。
（オブジェクト指向の理解があっているか不安ですが。。）

そもそもインスタンスの初期化処理もないので
このような処理でよいのかもわかりません。やりたいことに対して
私のこの発想でよいのでしょうか？一般的にはどうすべきでしょうか？

[TakaiY]

そういうことであれば、_DaRuMa_ さんの回答のように変更する必要があります。
messegesについては、外部から変更することは無いと思いますので、このままでいいとして、error_infoは__init__で初期化する必要があるでしょう。

また、クラスの定義のしかただけでなく、作ったクラスの使い方をふくめて知識が不完全に見うけられるので、そのあたりちゃんと調べなおしてやりなおしてみることをお勧めします。

また、このErrorClassについても、呼び出す側がインスタンス化して使うことになるわけで、使い勝手や必要な機能なども考えて、クラス化する必要があるかどうかを含めて再検討したほうがいいと思います。