pythonで別クラスのエラーをハンドリングしたい

Question

下記コードを作成しました。
test2.pyでtest1.pyのkeisanクラスをインポートしています。
test1.pyのなかで変数keyは存在しないので、test2.pyを実行したときに"おかしい"と表示させたいのですが、エラーハンドリングの仕方がわかりません。

どなたかご教示お願い致します

##test1.py##
class keisan():
    def tasu(self, x, y):
        kei = x + y
        try:
            print(key)
        except:
            print("おかしい")

##test2.py##
from test1 import keisan

q = keisan()
q.tasu(1,3)

Answer 1

余計なことしなければいいだけかと。

class keisan():
    def tasu(self, x, y):
        kei = x + y
        print(key)

from test1 import keisan
q = keisan()
try:
    q.tasu(1, 3)
except NameError as e:
    print(e)
#name 'key' is not defined

Comments

[DaichiYasuda]

ありがとうございました。

[DaichiYasuda]

この問題に関連する質問をしたので、もしお時間あれば回答願えますでしょうか？
https://teratail.com/questions/124331?modal=q-comp

Answer 2

ご質問のコードで「おかしい」とは表示されると思います。
ご質問の意図は何でしょうか？tasuメソッドが例外を呼び出し元へ発するようにして、それを呼び出した側のtest2.pyでハンドリングすることなら

# test1.py
class keisan():
    def tasu(self, x, y):
        kei = x + y
        print(key)

# test2.py

from test1 import keisan

q = keisan()
try:
    q.tasu(1,3)
except:
    print("おかしい")

という感じです。tasuメソッドにバグがあっても呼び出し元で例外をハンドリングすればよいということです。元のコードでは例外がおきると分かっているのにtasuメソッドが自分自身で例外を「握りつぶしている＝＞なかったことにしている」ので呼び出し元へは例外が通知されません。

もし意図的に「例外が起こったとき、何かの例外処理をするがその例外を呼び出し元へも通知したい」なら以下のようにraiseしなおせばよいです。（あくまで例ですので下記のコードに意味があるとは思えませんが・・・）

class keisan():
    def tasu(self, x, y):
        kei = x + y
        try:
            print(key)
        except Exceptioin as e:
            print('exception: ', e)
            raise e

Comments

[DaichiYasuda]

ご認識あっております
manzyunさんのおっしゃっているように、test2からtest1にエラーを飛ばしたいです

[KSwordOfHaste]

とすれば発生した例外をそのまま上位へ通知しないなら自分の回答の後の例のように、また別の例外を通知したいならmanzyunさんコードのようにするとよさそうです。要するにtry-exceptで例外をハンドリングしたとき、例外を改めてraiseしなおさないと外側に通知されませんよというのがポイントですね。

[DaichiYasuda]

ありがとうございました！

[DaichiYasuda]

この問題に関連する質問をしたので、もしお時間あれば回答願えますでしょうか？
https://teratail.com/questions/124331?modal=q-comp

Answer 3

すっごく雑なやり方かもしれませんが、

        try:
            print(key)
        except:
            print("おかしい")

を

        try:
            print(key)
        except:
            raise ValueError("おかしい")

とやると、test1 の方から ValueError を投げられます。

後は test2 の方で投げられたエラーをどう処理するか書けば解決しそうな気がします。

8. エラーと例外 — Python 3.6.5 ドキュメント

Comments

[DaichiYasuda]

ありがとうございました！
manzyunさんの方法で解決できました。

[DaichiYasuda]

この問題に関連する質問をしたので、もしお時間あれば回答願えますでしょうか？
https://teratail.com/questions/124331?modal=q-comp

Answer 4

こんな感じで自作例外を定義しておき、

class InvalidKeyException(Exception):
    def __init__(self, message, key_val):
        self.message = message
        self.key_val = key_val

受ける側はこんな感じです。

try:
    raise InvalidKeyException("おかしい", -1)
except InvalidKeyException as e:
    print(e) # => ('おかしい', -1) ※tupleとして表示されるのはpython（というか継承元のException）の機能
    print(e.message) # => おかしい
    print(e.key_val) # => -1

keisan.tasuの中でraiseしてあげて、test2.pyでキャッチしてあげれば良い訳ですね。

Comments

[DaichiYasuda]

ありがとうございました！

[DaichiYasuda]

この問題に関連する質問をしたので、もしお時間あれば回答願えますでしょうか？
https://teratail.com/questions/124331?modal=q-comp