Python 戻り値について

Python 初心者です。

先物自動売買ツールの作成に挑戦しています。

以下のメソッドは売買ロジックのうちの一つです。

あるポジションを持っている状態で、そのポジションの取得価格より、X円安い場合、損切を行います。

また、直近過去の最低値を一度下回り、そのあと、直近過去の最高値をy円上回ったときに、利確を行うというロジックなのですが、

直近過去の最低値を一度下回ったことを確認するpassedvallyをTrueにしたあとの戻り値で落ちてしまいます。

解決方法、または有効なロジックをご教示いただきたいです。

宜しくお願いいたします。

以下、エラーメッセージになります。

Traceback (most recent call last):
File "gajumaru3.py", line 513, in <module>
main()
File "gajumaru3.py", line 467, in main
vallypassed)
File "gajumaru3.py", line 188, in c_logic
return curposition, curcoin, settlementprice, pos_price, pl_state, vallypassed
UnboundLocalError: local variable 'settlementprice' referenced before assignment

Python
1def c_logic(curposition, curprice, curcoin, mountain, vally, pos_price, vallypassed):
2    if curposition == 'C':
3        if curprice <= pos_price - X:
4            settlementprice = settlement_buy(curprice, curcoin);
5            win_lose_flag = win_lose(curposition, curcoin, settlementprice, pos_price);
6            pl_state = calc_pl_state(curposition, curcoin, pos_price, settlementprice);
7            next_position = ctrl_logic(curposition, win_lose_flag);
8            curposition = next_position
9            nextcoin = ctrl_coin_logic(curcoin, win_lose_flag);
10            curcoin = nextcoin
11            pos_price = curprice
12            vallypassed = False
13            return curposition, curcoin, settlementprice, pos_price, pl_state, vallypassed
14        else:
15            pass
16        if vallypassed == False:
17            if curprice <= vally:
18                vallypassed = True
19                settlementprice = 0
20                pl_state = 0
21#ここで落ちる
22            return curposition, curcoin, settlementprice, pos_price, pl_state, vallypassed
23        #利確
24        elif vallypassed == True and curprice >= mountain + y:
25            settlementprice = settlement_buy(curcoin, curprice);
26            win_lose_flag = win_lose(curposition, curcoin, settlementprice, pos_price);
27            pl_state = calc_pl_state(curposition, curcoin, pos_price, settlementprice);
28            nextposition = ctrl_logic(curposition, win_lose_flag);
29            curposition = nextposition
30            nextcoin = ctrl_coin_logic(curcoin, win_lose_flag);
31            curcoin = nextcoin
32            pos_price = curprice
33            vallypassed = False
34            return curposition, curcoin, settlementprice, pos_price, pl_state, vallypassed
35        else:
36            settlementprice = 0
37            pl_state = 0
38            return curposition, curcoin, settlementprice, pos_price, pl_state, vallypassed
39    else:
40        pass

kazto

2018/04/23 02:21

「落ちる」と称する現象の詳細、スタックトレース、エラーメッセージなど追記できますでしょうか。

songyong

2018/04/23 02:26

失礼しました。エラーメッセージを追記しました。戻り値の不足？が原因という認識で、0を代入してみたのですが、上手くいきませんでした。宜しくお願いいたします。

行動規範の内容に同意します

回答2件

質問文のロジックに関していえば、if curposition == 'C':直下でsettlementpriceとpl_stateの初期化すれば、良い気がしますが。

Python
1if curposition == 'C':
2    settlementprice = 0
3    pl_state = 0
4    if curprice <= pos_price - X:
5        settlementprice = settlement_buy(curprice, curcoin);
6    #以下省略

あとif文のelseのpassが多いので、ifの条件を判定させreturnで抜ける形にしたほうifのネストが軽減されます。

投稿2018/04/23 02:47

umyu

総合スコア5846

ベストアンサー

Python
1if vallypassed == False:
2    if curprice <= vally:
3        vallypassed = True
4        settlementprice = 0
5        pl_state = 0

return curposition, curcoin, settlementprice, pos_price, pl_state, vallypassed

curprice <= vallyが成立しない際、settlementpriceが初期化されません。
例えば次のように、条件が偽の場合についても考慮する必要があります。

Python
1if vallypassed == False:
2    if curprice <= vally:
3        vallypassed = True
4        settlementprice = 0
5        pl_state = 0
6    else:
7        settlementpriceなどを適切に初期化
8
9    return curposition, curcoin, settlementprice, pos_price, pl_state, vallypassed