Global name not defined error in Python

Question

I am implementing my first class in Python and am struggling to make it work. I started with a very simple example:

#!/usr/bin/env python
"""

"""
import fetcher as fetcher
import fetchQueue as fetchQueue

    def __init__(self, seed = "a string"):
        self.seed = seed
        myFetchQueue = fetchQueue.FETCHQueue()

    def test(self):
        print "test"
        myFetchQueue.push(seed)
        myFetchQueue.pop()


#Entrance of this script, just like the "main()" function in C.
if __name__ == "__main__":
    import sys
    myGraphBuilder = GRAPHBuilder()
    myGraphBuilder.test()

and this class should call a method of another class I defined in a very similar way.

#!/usr/bin/env python
"""

"""

from collections import defaultdict
from Queue import Queue

class FETCHQueue():

    linkQueue = Queue(maxsize=0)
    visitedLinkDictionary = defaultdict(int)

    #Push a list of links in the QUEUE
    def push( linkList ):
        print linkList

    #Pop the next link to be fetched
    def pop():
        print "pop"

However when I run the code I get this output:

test Traceback (most recent call last):   File "buildWebGraph.py", line 40, in <module>
    myGraphBuilder.test()   File "buildWebGraph.py", line 32, in test
    myFetchQueue.push(seed) NameError: global name 'myFetchQueue' is not defined

So I guess that the construction of the object of class GRAPHBuilder and FETCHQueue is working, otherwise I would get an error before the string test gets outputed, but something else is going wrong. Can you help me?

thefourtheye · Accepted Answer · 2014-01-18 06:40:16Z

1

def __init__(self, seed = "a string"):
    self.seed = seed
    myFetchQueue = fetchQueue.FETCHQueue()

Here, myFetchQueue is a local variable to the __init__ function. So, it will not available to other functions in the class. You might want to add it to the current instance, like this

    self.myFetchQueue = fetchQueue.FETCHQueue()

Same way, when you are accessing it, you have to access it with the corresponding instance, like this

    self.myFetchQueue.push(self.seed)
    self.myFetchQueue.pop()

edited Jan 18, 2014 at 6:40

answered Jan 18, 2014 at 6:31

thefourtheye

241k53 gold badges466 silver badges505 bronze badges

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

2 Comments

Matteo Over a year ago

and I guess also seed in self.myFetchQueue.push(seed) should be self.seed if I understood correctly... ;D

Matteo Over a year ago

Yep sorry for that...it completely ran out of my mind! I would have figured out eventually ;D

Collectives™ on Stack Overflow

Global name not defined error in Python

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related