在Python中构建API调用(Structuring API Calls in Python)

 我很难确定用于构造数据的最佳实践，以调用包含比特币价格的各种API。 我希望能够调用多个API，而无需重复代码。 
 
 我最初的想法是为我要调用的每个API构建类，并将它们的属性（api_id，url和json_tree（我想从中提取数据的json路径）提供给BtcAPI类，然后将它们吐出来。 
 
 *注意，在读取BtcAPI工作之前，Coindesk / Bitstamp类尚未与该类交互。 在我遇到麻烦之前，我想问一下我应该怎么做... * 
 
 现在，我想知道我是否应该制作它们，例如： 
 
coindesk = ['http://www.something.com', 'coindesk', '["time"]["updated"]']
 
 ......并且只是迭代它们中的每一个。 或者说，或者其他各种各样的东西。 这里指出了什么数据结构？ 
 
 我基本上是在寻找一些代码审查（因为这段代码不起作用，我不想将它发送到代码审查堆栈）并了解最佳实践：告诉我你认为我在哪里搞砸可怕的，我能做些什么来更好地构建这些数据？ 我是一个蟒蛇和oop noob。 我可以在程序上做到这一点，但这将是丑陋和多余的。 我想我的课程也有点错误。 见解？ 帮帮我？ 谢谢！ 
 
 谢谢！ 
 
import json
import urllib.request

#The BtcAPI class works well when you feed it static variables. It returns json.

class BtcAPI:

    def __init__(self, url, api_id):
        self.url = url
        self.api_id = api_id

    def btc_api_call(self):

        hdr = { 'User-Agent' : 'Mozilla/5.0 (Windows NT 6.1; Win64; x64)'     }
        req = urllib.request.Request(url, headers=hdr)
        readdata = urllib.request.urlopen(req)
        json_data = readdata.read()

        json_dict = json.loads(json_data)
        return(json_dict)

class Coindesk:


    api_id = 'Coindesk'
    url = 'https://api.coindesk.com/v1/bpi/currentprice.json'               
    json_tree = json_dict['time']['updated']

    def __init__(self):

        self.current_us_price = current_us_price

class Bitstamp:


    api_id = 'Bitstamp'
    url = 'https://www.bitstamp.net/api/ticker/'               
    json_tree = json_dict['last']

    def __init__(self):

        self.current_us_price = current_us_price

coindesk_url = Coindesk()
coindeskoutput = coindesk_url.url
print(coindeskoutput)

I'm having difficulty in determining what the best practices are for structuring data for calling various APIs that contain bitcoin prices. I want to be able to call multiple APIs, without duplicate code.
 
My initial thought was to build classes for each of the APIs I would be calling, and feed their attributes (api_id, url, and json_tree (json path I'd like to pull data from) to the BtcAPI class, and spit them out. 
 
*Note, before reading that while BtcAPI works, the Coindesk/Bitstamp classes do not yet interact with that class. I want to ask about the way I should do this before I go to the trouble...*
 
Now, I'm wondering if I shouldn't make them lists, like:
 
coindesk = ['http://www.something.com', 'coindesk', '["time"]["updated"]']
 
...and just iterate through each of them. Or dicts, or any variety of other thing. What data structure is indicated here?
 
I am basically looking for a bit of a code review (since this code isn't working, I don't want to send it to the code review stack) and an understanding of best practices: tell me where you think I'm screwing up horribly, and what I can do to structure this data better? I'm a python and oop noob. I could do this procedurally, but it would be ugly and superfluous. I imagine I'm using classes a bit wrong, too. Insights? Help? Thank you!
 
Thank you!
 
import json
import urllib.request

#The BtcAPI class works well when you feed it static variables. It returns json.

class BtcAPI:

    def __init__(self, url, api_id):
        self.url = url
        self.api_id = api_id

    def btc_api_call(self):

        hdr = { 'User-Agent' : 'Mozilla/5.0 (Windows NT 6.1; Win64; x64)'     }
        req = urllib.request.Request(url, headers=hdr)
        readdata = urllib.request.urlopen(req)
        json_data = readdata.read()

        json_dict = json.loads(json_data)
        return(json_dict)

class Coindesk:


    api_id = 'Coindesk'
    url = 'https://api.coindesk.com/v1/bpi/currentprice.json'               
    json_tree = json_dict['time']['updated']

    def __init__(self):

        self.current_us_price = current_us_price

class Bitstamp:


    api_id = 'Bitstamp'
    url = 'https://www.bitstamp.net/api/ticker/'               
    json_tree = json_dict['last']

    def __init__(self):

        self.current_us_price = current_us_price

coindesk_url = Coindesk()
coindeskoutput = coindesk_url.url
print(coindeskoutput)

原文：https://stackoverflow.com/questions/48572494