Python OOP: Variable is not defined. What is the scope issue?

Question

The error message is as follows:

Traceback (most recent call last):
  File "./bitcoin_price.py", line 57, in <module>
    Coindesk()
  File "./bitcoin_price.py", line 45, in Coindesk
    coindesk.pass_for_request()
  File "./bitcoin_price.py", line 39, in pass_for_request
    get_price = BtcAPI(url, api_id, json_tree)
NameError: name 'url' is not defined

What am I missing, here? I'm guessing it is the coindesk.pass_for_request(), but I can't sort out exactly why the values don't get passed. Also, what is a good way to debug something like this? I'm having a rough time looking inside of the classes to see what is going on.

class Price:


    def __init__(self, api_id, url, json_tree):

        self.api_id = api_id
        self.url = url
        self.json_tree = json_tree

    def pass_for_request(self):

        get_price = BtcAPI(url, api_id, json_tree)
        get_price.btc_api_call()


def Coindesk():

    coindesk = Price(api_id ="coindesk", url = "https://api.coindesk.com/v1/bpi/currentprice.json", json_tree = "['time']['updated']")
    coindesk.pass_for_request()

Yash Sharma · Accepted Answer · 2018-02-04 06:22:12Z

1

There is no url or api_id variable in the scope of pass_for_request. You probably meant to access self.url and self.api_id, in python you have to use self. to access members unlike in other languages where using this. is optional.

Also another thing I spotted is when you're trying to create a Price in Coindesk you are passing in api_id and the other arguments as if they are arguments with defaults, which they are not. You'd need to call it as: Price("coindesk", "https://mylinktocoindesk", "['time']['updated']")

Here's some reading on OOP in python that you might find helpful: https://docs.python.org/3/tutorial/classes.html https://www.tutorialspoint.com/python/python_classes_objects.htm

answered Feb 4, 2018 at 6:22

Yash Sharma

2931 gold badge4 silver badges14 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

ijkl26 · Accepted Answer · 2018-02-04 06:21:34Z

1

Try to replace

get_price = BtcAPI(url, api_id, json_tree)

with

get_price = BtcAPI(self.url, self.api_id, self.json_tree)

answered Feb 4, 2018 at 6:21

ijkl26

837 bronze badges

Collectives™ on Stack Overflow

Python OOP: Variable is not defined. What is the scope issue?

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related