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

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

ただいまの
回答率

90.50%

  • Python

    8002questions

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

  • Python 3.x

    6415questions

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

Python 戻り値について

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 213

songyong

score 7

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

def c_logic(curposition, curprice, curcoin, mountain, vally, pos_price, vallypassed):
    if curposition == 'C':
        if curprice <= pos_price - X:
            settlementprice = settlement_buy(curprice, curcoin);
            win_lose_flag = win_lose(curposition, curcoin, settlementprice, pos_price);
            pl_state = calc_pl_state(curposition, curcoin, pos_price, settlementprice);
            next_position = ctrl_logic(curposition, win_lose_flag);
            curposition = next_position
            nextcoin = ctrl_coin_logic(curcoin, win_lose_flag);
            curcoin = nextcoin
            pos_price = curprice
            vallypassed = False
            return curposition, curcoin, settlementprice, pos_price, pl_state, vallypassed
        else:
            pass
        if vallypassed == False:
            if curprice <= vally:
                vallypassed = True
                settlementprice = 0
                pl_state = 0
#ここで落ちる
            return curposition, curcoin, settlementprice, pos_price, pl_state, vallypassed
        #利確
        elif vallypassed == True and curprice >= mountain + y:
            settlementprice = settlement_buy(curcoin, curprice);
            win_lose_flag = win_lose(curposition, curcoin, settlementprice, pos_price);
            pl_state = calc_pl_state(curposition, curcoin, pos_price, settlementprice);
            nextposition = ctrl_logic(curposition, win_lose_flag);
            curposition = nextposition
            nextcoin = ctrl_coin_logic(curcoin, win_lose_flag);
            curcoin = nextcoin
            pos_price = curprice
            vallypassed = False
            return curposition, curcoin, settlementprice, pos_price, pl_state, vallypassed
        else:
            settlementprice = 0
            pl_state = 0
            return curposition, curcoin, settlementprice, pos_price, pl_state, vallypassed
    else:
        pass
  • 気になる質問をクリップする

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • kazto

    2018/04/23 11:21

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

    キャンセル

  • songyong

    2018/04/23 11:26

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

    キャンセル

回答 2

+3

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

if curposition == 'C':
    settlementprice = 0
    pl_state = 0
    if curprice <= pos_price - X:
        settlementprice = settlement_buy(curprice, curcoin);
    #以下省略

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

checkベストアンサー

+2

if vallypassed == False:
    if curprice <= vally:
        vallypassed = True
        settlementprice = 0
        pl_state = 0

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

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

if vallypassed == False:
    if curprice <= vally:
        vallypassed = True
        settlementprice = 0
        pl_state = 0
    else:
        settlementpriceなどを適切に初期化

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/04/23 11:39

    いつも迅速な回答、ありがとうございます。
    偽の条件が抜けていたのですね。。


    ありがとうございます。

    今後ともご指導宜しくお願い致します。

    キャンセル

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

  • ただいまの回答率 90.50%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

  • 解決済

    Pythonのreturn falseと、return not の違い

    Pythonで、特定の処理が偽場合にreturn falseとするのが通常ですが、return not というreturnの方法もあると知りました。 これは、return f

  • 解決済

    python を使った会話AIの実現

    前提・実現したいこと 前提 docomo apiからRepl-AIに変更し、1度返事を得ることができた。 実現したいこと 連続した会話をしたい。 ※初心者で的を射ない質問で

  • 解決済

    Tkinter で動作をイベントとして登録する方法

    プログラミング初学者です 音楽プレーヤーをtkinterを使って製作しているのですが ボタン押すと新しい子ウィンドウが作成され、子ウィンドウが出ている間はボタンをDISABLE

  • 受付中

    間違えたif文とフラグの使いかた

    お世話になります。 5つの分岐をして、結果的に条件に合わない場合は、成功と考えて arbitrage_decision = 1  # 停止がない為、裁定取引判定処理を実施(一番

  • 解決済

    can't multiply sequence by non-int of type 'float'...

     前提・実現したいこと 閲覧ありがとうございますm(_ _)m プログラミングは全くの初心者で、本を読みながらpythonの勉強をしているのですが、本の通りにコーディングしたはずな

  • 解決済

    pythonで乱数生成による無限ループ脱出

     前提・実現したいこと 初めての質問です。よろしくお願いします pythonでヌメロンっていうゲームを作ろうと思った際に行き詰まりました 簡単にいうと2人で対戦し、お互い3桁の数字

  • 解決済

    メンバ変数をBaseball_Teamに持たせる 方法

     前提・実現したいこと 2017年セリーグの勝ち負け数を管理するためのクラスを作成して、インスタンスを6チーム分生成した後に、各チームの勝ち負け数を表示するプログラムを作成しようと

  • 解決済

    アウトプット方法がわからない

    □アウトプット team     win lose draw rate のように表示させたいですが、記載方法がわかりません。どのようにおこなえばよろしいでしょうか。 print

同じタグがついた質問を見る

  • Python

    8002questions

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

  • Python 3.x

    6415questions

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