質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
Python 3.x

Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

Q&A

解決済

1回答

2030閲覧

pythonのretryingデコレータのパラメータにコンストラクタで取得した値を設定したい

tomy-c

総合スコア179

Python 3.x

Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

0グッド

1クリップ

投稿2019/12/09 04:23

前提・実現したいこと

現在GoogleCloudTranslateAPIを使用して大量の言葉を翻訳するシステムを作成しています。
その過程でGoogleAPIの割り当てと制限等の影響で翻訳機能を使った際の動作が不安定だということが判明し、
retryingモジュールを使用して、翻訳部分について設定ファイルより取得した値だけリトライを行う処理を検討しており、その過程で外部サイトを参考にテストコードを作成しました。

該当のソースコード

参考サイト
参考サイト2
上記を元にコンストラクタで設定したクラス変数をretryデコレータにセットしようとすると静的解析でエラーが発生します。
最終的には独立したクラスモジュールに組み込みたいため、上記の**(コンストラクタでデコレータのパラメータをセットする)**という機能は必須になります。

python

1# 指定したURLに接続しレスポンスが帰ってくるまでリトライするコード 2 3#!/usr/bin/env python 4# -*- coding: utf-8 -*- 5 6import logging 7import requests 8import sys 9import time 10from retrying import retry, RetryError 11from requests.exceptions import HTTPError 12 13logging.basicConfig(level=logging.INFO) 14 15 16class TestRunner(object): 17 __maxdelay = 0 18 __retrycount = 0 19 20 def __init__(self): 21 self.__maxdelay = 10000 22 self.__retrycount = 3 23 24 # 例外時3回,合計10秒(10,000ミリ秒)リトライを行う(例外内容をRetryErrorにラップする) 25 # ※(問題箇所)マジックナンバー行けるが、クラス変数はNG 26 @retry(stop_max_delay=self.__maxdelay, stop_max_attempt_number=self.__retrycount, wrap_exception=True) 27 def get(self, url): 28 print("call function") 29 logging.info(time.time()) 30 response = requests.get(url) 31 if (response.status_code != requests.codes.ok): 32 raise response.raise_for_status() 33 return response 34 35if __name__ == "__main__": 36 # 強制的に例外引き起こすためのパラメータ 37 url = "https://dev.classmethod.jp/server-side/python/python-retrying-le/" 38 39 runner = TestRunner() 40 try: 41 response = runner.get(url) 42 print(response.headers) 43 except HTTPError as e: 44 print("ERROR:HTTP") 45 print(e) 46 except RetryError as e: 47 # パラメータでRetryErrorにラップさせているため、例外発生時はここを通ってからリトライをする 48 print("ERROR:RETRY") 49 print(e) 50 except Exception as e: 51 print("ERROR:OTHER") 52 print(e)

発生している問題・エラーメッセージ

NameError: name 'self' is not defined

selfは使えないと言われています。
試しにselfを消して記述しても当然ながらコンストラクタでセットしたパラメータは適応されていません(初期値がセットされます)

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

自己解決

根本の問題を探っていくとデコレータへのパラメータ渡しがインスタンス生成前に走っていたのが問題だったため、retryingを適用させたいクラス関数を関数内関数にし、関数内関数に対してデコレータを与えることで、クラス変数をデコレータにセットできました。(この作り方がいいのかはわかりませんが・・・)
また、この方法で実装すると、正常時に実態があるのにオブジェクト内の変数に直接アクセスしてprintができなくなりました。(一旦別の変数にセットすれば普通にprint出来ました。)
どうも不可解な動きを部分的にしていますが、概ね望んでいた動作をしたのでこれを解決方法にします。
(※もっとマトモでスマートな方法がある場合遠慮なく回答を記載していただくと助かります)

python

1# 指定したURLに接続しレスポンスが帰ってくるまでリトライするコード 2 3#!/usr/bin/env python 4# -*- coding: utf-8 -*- 5 6import logging 7import requests 8import sys 9import time 10from retrying import retry, RetryError 11from requests.exceptions import HTTPError 12 13logging.basicConfig(level=logging.INFO) 14 15 16class TestRunner(object): 17 __maxdelay = 0 18 __retrycount = 0 19 20 def __init__(self): 21 self.__maxdelay = 10000 22 self.__retrycount = 10 23 24 def get(self, url): 25 # 関数内関数ならクラス変数をデコレータにセット可能 26 # 例外時10回,合計10秒(10,000ミリ秒)リトライを行う(例外内容をRetryErrorにラップする) 27 @retry(stop_max_delay=self.__maxdelay, stop_max_attempt_number=self.__retrycount, wrap_exception=True) 28 def decoget(*args, **kwargs): 29 print("call function") 30 logging.info(time.time()) 31 response = requests.get(url) 32 if (response.status_code != requests.codes.ok): 33 raise response.raise_for_status() 34 return response 35 res = decoget(self, url) 36 return res 37 38if __name__ == "__main__": 39 # 正常なデータ 40 url = "https://dev.classmethod.jp/server-side/python/python-retrying-module/" 41 42 # 強制的に例外を引き起こすデータ 43 # url = "https://dev.classmethod.jp/server-side/python/python-retrying-le/" 44 45 runner = TestRunner() 46 try: 47 response = runner.get(url) 48 # ↓関数内関数でデコったあと以下の呼び出しだと実体あるのに'Response' object has no attribute 'header' 49 # print(response.header) 50 51 # こっちだと普通に通る(Why??????????????) 52 r = response.headers 53 print(r) 54 except HTTPError as e: 55 print("ERROR:HTTP") 56 print(e) 57 except RetryError as e: 58 print("ERROR:RETRY") 59 print(e) 60 except Exception as e: 61 print("ERROR:OTHER") 62 print(e) 63

投稿2019/12/09 07:19

tomy-c

総合スコア179

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問