Python for文 continueの関数化による出力結果の違い。

Question

Python初学者です。

実現したいこと
１、BitflyerのAPIに接続
２、残高が０でないコインの情報を取得
３、取得した要素をFlaskを使用してHTMLに表示させたいです。

以下のコードで残高が０でない要素のみを正しく取得することが出来ました。

import pybitflyer
import time
from datetime import datetime
from pytz import timezone
from dateutil import parser

BALANCE_KEYS = ["currency_code",
#                 "amount",
                "available"]

API_KEY = ""
API_SECRET = ""



api = pybitflyer.API(api_key = API_KEY, api_secret = API_SECRET)
balances = api.getbalance(product_code="BTC_JPY")

for balance in balances:

    if balance ["available"] == 0:
        continue
    for balance_key in BALANCE_KEYS:
            result =(balance_key + " : " + str(balance[balance_key]))
            print(result +"\n")
```

しかし、上記の処理を関数化しようとして以下のように書き換えたところ、上で出力した返り値の最後の要素のavailableのみしか取得ができません。

```Python
import pybitflyer
from flask import Flask, render_template,request
import time
from datetime import datetime
import json
from dateutil import parser
from pytz import timezone



BASE_URL ="https://api.bitflyer.jp"
url = BASE_URL + "POST /v1/me/sendchildorder"

API_KEY =""
API_SECRET =""
api = pybitflyer.API(API_KEY,API_SECRET)

BALANCE_KEYS = ["currency_code",
                    "available"]
balances = api.getbalance(product_code="BTC_JPY")

def get_balance():
    for balance in balances:
        if balance["available"]==0:
            continue
        for balance_key in BALANCE_KEYS:
            balance_result=(balance_key +" : " +str(balance[balance_key]))
    return balance_result

```
残高が０でない全ての要素を取得、かつ関数化するにはどのようにコードを書き換えれば良いのでしょうか?
returnの書き方に問題があるのでしょうか。
また、なぜ上記２つのコードの出力結果がこんなにも違うのか、理由がわかりません。
なお、APIの情報は個人情報保護のため削除しておりますが、実際のコードには記述があります。

どなたかご存知の方がいらっしゃいましたら、ご教授ください。

Answer 1

2つの処理は明確に異なります。 また、質問の「出力」って何でしょう？

for balance in balances:

    if balance ["available"] == 0:
        continue
    for balance_key in BALANCE_KEYS:
            result =(balance_key + " : " + str(balance[balance_key]))
            print(result +"\n")

これはfor文が回るたびに resultの値を「表示」しています。

def get_balance():
    for balance in balances:
        if balance["available"]==0:
            continue
        for balance_key in BALANCE_KEYS:
            balance_result=(balance_key +" : " +str(balance[balance_key]))
            # ★
    return balance_result

こちらの処理では、for文で回していますが、保存も表示もしていないので、最後に回ったときのbalance_resultの値だけreturnで返しています。
表示するのが目的ｄえあれば、★のところに balance_resultをprintする処理を入れてみればいいでしょう。

結果を全て返したいのであれば、リストにするか辞書にするかしてそれを返すようにすればいいでしょう。

def get_balance():
    all_result = {}
    for balance in balances:
        if balance["available"]==0:
            continue
        for balance_key in BALANCE_KEYS:
            balance_result=(balance_key +" : " +str(balance[balance_key]))
            all_result[balance_key] = balance[balance_key]
    return all_result

表示するにしても返すにしても、これらではちょっと違うように思いますが、得られた結果をどのように使いたいのか示してもらえれば、もうちょっといいアドバイスができるかもしれません。

Comments

[Na3430mls]

ご回答ありがとうございます！なるほど、保存をしていないからループの度に取得した値が消えていたのですね。。。。！

大まかな流れは
１、関数を定義
２、関数内で得られた結果を{currency_code: available}の組み合わせでディクショナリに保存→変数に入れる。
３、変数に保存したものを別のファイルで関数を呼び出すことにより、表示させる

のような感じです。

[Na3430mls]

該当の関数で実現したいのは、
BALANCE_KEYS = ["currency_code","available"]をAPIを使って取得して、関数を実行すれば呼び出しが可能なように変数に入れて保存しておくことです。

出力というのは、実行結果というような意味合いで使っておりましたが使い方が違うかもしれません。紛らわしかっったらすみません。。。

[Na3430mls]

def get_balance():
all_result = {}
for balance in balances:
if balance["available"]==0:
continue
for balance_key in BALANCE_KEYS:
balance_result=(balance_key +" : " +str(balance[balance_key]))
all_result[balance_key] = balance[balance_key]

get_balance()
print(all_result[balance_key])

としたら、print(all_result[balance_key])は定義されていませんと言われてしまいました
こちらに取得した値を保存しているというわけではないのでしょうか？

[TakaiY]

all_resultやbalance_keyなどの関数内で定義した変数は、関数の外では使えません。

all_result は回答にあるように、returnで返していますので、呼び出し側で変数で受けることで使えるようになります。

balance_result = get_balance()
print(balance_result)

このようにすれば表示されるはずです。受ける変数はあえて名前を変えています。

いずれにしても、関数と変数のスコープについては、もうちょっと調べて理解を深めるといいと思います。
balances や BALANCE_KEYS は意図的であれ、関数の外の変数を参照しています。これらは引数で渡してやったほうが明確になると思います。