綺麗なコードを教えてほしいです

Question

こんばんは、プログラミング初心者です、pythonを使い ゲームのAPIを使用してゲームのサーバー,プレイヤー情報を取得しています、コードは書けたのですが、この書き方以外に 他に方法があるのかなと興味があり書き込みました、短く書けたりもっとわかりやすく取得できるコードがあれば教えてほしいです！ 関数名もこれでいいのか教えてほしいです！
コメントはここでしか使ってないです！

import requests

# ゲームのAPI 自分のuidなので自由に使ってください
# https://api.battlemetrics.com/players/1002045653?include=server

# "rust"というゲームのサーバーだけを取得
# プレイヤーが今参加中のサーバーの名前を取得
def getOnlineServer(uid):
    r = requests.get("https://api.battlemetrics.com/players/"+str(uid)+"?include=server")
    if r.status_code != 200:
        return None
    included = r.json()["included"]
    for server in included:
        if server["relationships"]["game"]["data"]["id"] == "rust": # rustのサーバーか
            if server["meta"]["online"]: # サーバーに参加しているか
                return server["attributes"]["name"] # していたら サーバーの名前を返す
    return "offline"

print(getOnlineServer("1002045653"))

Comments

[fiveHundred]

自分で取得したからといって、uidは公開していいものではありません。
例えば、悪意のある人がこれを使って、サーバーに攻撃でもしたら、あなたの責任になりますよ。

teratailの質問は編集後も履歴に残るので、削除依頼してください。

[CHERRY]

ドキュメント( https://www.battlemetrics.com/developers/documentation#link-GET-player-/players/{(%23%2Fdefinitions%2Fplayer%2Fdefinitions%2Fidentity)} ) を読んだ感じでは、このエンドポイントは、認証が不要のようですね。

また、質問のコードで uid と書かれている部分は、ドキュメントには、player_id とあるので、「認証用のトークン」等では無く、https://www.battlemetrics.com/players 等の公式ページで公開されている プレーヤーID みたいですので、質問者が自分のプレーヤーIDを公開しているだけであれば、質問を削除するほどのことではないと思います。

[fiveHundred]

ご指摘通り、認証するためのものと勘違いしてました。
失礼しました。

[azarasi]

返信遅れました、勘違いさせてしまいすみません、ありがとうございます。

Answer 1

他のpythonプログラマが読むことを考えると、以下の点は変更した方が良いかもしれません。

• 関数名はcamelCaseではなくsnake_caseに
• uidは数値ではなく元から文字列で渡しているので、str()は不要

以下の点を使って短かさ優先で書くこともできますが、読みづらい/デバッグが困難になるのでオススメしません。

• 明示的にreturnしないとNoneが返るのでif.r.status_code != 200:の部分は省略可能
• next()を使用して、for部分を短かく

import requests

def get_online_server(uid):
    r = requests.get(f"https://api.battlemetrics.com/players/{uid}?include=server")
    if r.status_code == 200:
        return next((server["attributes"]["name"] for server in r.json()["included"]
                     if server["relationships"]["game"]["data"]["id"] == "rust"
                     and server["meta"]["online"]), "offline")

print(get_online_server("1002045653"))