python foutcode

[arneschoonvliet]

beste tweakers,

dit is de code

Python: volume balk.py

def voorbereiding():
        while 1:
            stoppen = raw_input("wil je stoppen?(y/n) ")
            if stoppen == 'y':break
            eenheid = raw_input("geef je eenheid in ")
            
            berekening()
    
    
def berekening():
    b = input('geef de breedte ')
    print str(b) + str(eenheid)
    l = input('geef de lengte ')
    print str(l) + str(eenheid)
    h = input('geef de hoogte ')
    print str(h) + str(eenheid)
    uitkomst = b*l*h
    print "\nHet volume van de balk is " + str(uitkomst)

voorbereiding()

en het probleem is, als ik dit uitvoer krijg ik deze fout

Traceback (most recent call last):
File "C:/Users/Arne/Desktop/volume balk2.py", line 20, in <module>
voorbereiding()
File "C:/Users/Arne/Desktop/volume balk2.py", line 7, in voorbereiding
berekening()
File "C:/Users/Arne/Desktop/volume balk2.py", line 12, in berekening
print str(b) + str(eenheid)
NameError: global name 'eenheid' is not defined


gisteren had ik dit niet, het programma werkt toen prima, ik start het vandaag terug op en geeft deze foutcode hoe kan dit?, wie kan mij helpen?

ps. ik heb al gezocht op google en kon geen antwoord vinden ,, ben ook nog een beginneling

mvg,

[Matis]

Het toverwoord waarnaar jij opzoek bent is scope

[Verwijderd]

Het probleem is dat je twee aparte functies hebt, die elkaars variabelen niet kunnen zien. Zoek eens op 'python scope' als je meer info wilt hebben daarover. Je kunt het oplossen met bijv. het 'global'-keyword, of door de ene functie binnen de andere te definiëren. Beter is misschien om in dit geval de code niet in functies te dumpen (heeft geen echte zin hier) en met loops te werken.

edit: Matis is me voor...

[ Voor 5% gewijzigd door Verwijderd op 14-05-2010 19:43 ]

[0xDEADBEEF]

Scorpia schreef op vrijdag 14 mei 2010 @ 19:34:
gisteren had ik dit niet

Oh dan heb je toch wat gewijzigd/iets anders gedaan.

Python:

eenheid = 0
def voorbereiding():
        while 1:
            stoppen = raw_input("wil je stoppen?(y/n) ")
            if stoppen == 'y':break
            eenheid = raw_input("geef je eenheid in ")
            
            berekening()
    
    
def berekening():
    b = input('geef de breedte ')
    print str(b) + str(eenheid)
    l = input('geef de lengte ')
    print str(l) + str(eenheid)
    h = input('geef de hoogte ')
    print str(h) + str(eenheid)
    uitkomst = b*l*h
    print "\nHet volume van de balk is " + str(uitkomst)

voorbereiding()

>>> voorbereiding()
wil je stoppen?(y/n) n
geef je eenheid in 2
geef de breedte 4
40
geef de lengte 6
60
geef de hoogte 8
80

Het volume van de balk is 192
wil je stoppen?(y/n) y
>>>

[arneschoonvliet]

twearkers ,

bedankt voor de snelle reacties ,, het probleem is nu opgelost

dit door middel van
global te gebruiken .

hartelijk bedankt en tot de volgende vraag XD

[Verwijderd]

Ik zat te sidderen bij het woord Global. En deadbeef heeft waarschijnlijk de oorzaak al opgespoord Zijn oplossing lijkt mij beter, maar misschien overschat ik mijn kennis (op python-gebied).

[Creepy]

Wil je voor een eventuele vraag de volgende keer wel iets meer moeite steken in het zelf oplossen van je probleem of als je dat wel hebt gedaan dat aangeven in je topicstart? Nu is het niet meer dan "dit is mijn code, en ik krijg een foutmelding" en dat is echt te weinig. Dat je beginner bent is niet erg maar met een beetje logisch nadenken en wat zoeken moet je toch wel *iets* hebben geprobeerd om dit op te lossen.

[NMe]

Even achter Creepy's slotje aan omdat er hier nog steeds iets goed fout gaat:

Verwijderd schreef op vrijdag 14 mei 2010 @ 20:10:
Ik zat te sidderen bij het woord Global. En deadbeef heeft waarschijnlijk de oorzaak al opgespoord Zijn oplossing lijkt mij beter, maar misschien overschat ik mijn kennis (op python-gebied).

Nee, want 0xDEADBEEF gebruikt ook de globale scope door de variabele in die scope te initialiseren. Scorpia: zoek liever op hoe je parameters doorgeeft in Python, dat is de enige correcte oplossing hier. Overal maar globals voor gebruiken is iets dat je maar beter niet kan doen, dan zie je over 100 functies door de bomen het bos niet meer...