Problem With Scope Of A Variable

[Saif133]

I keep getting an error 

if ((YesAvailable == "True") and (len(str(FullNameEntry.get())) > 3) and (

NameError: name 'YesAvailable' is not defined

When I execute this script, 
What I want here is 
If the SearchingForSpace() function has been called before FinallyReserveParkingBySubmitButton()
It should change the value of the globally declared variable (YesAvailable) to "YES"
Otherwise, 
YesAvailable Should stay as "NO"

ParkingPlaceInfo= "0"
YesAvailable = "NO"

def SearchingForSpace():

   DestinationName = VariableForDestination.get()
   CarType = VariableForCars.get()
   for i in PlacesWithParkingSpaces:
       if (DestinationName == i):
           for j in PlacesWithParkingSpaces[i]:
               if (CarType == j):
                   for k in PlacesWithParkingSpaces[i][j]:
                       if PlacesWithParkingSpaces[i][j][k] == 1:
                           BookedOrNotBooked.config(text="Yes! Place Is Available")
                           global YesAvailable
                           YesAvailable="YES"
                           break
                       else:
                           BookedOrNotBooked.config(text="Pre-Booked.Choose Another Destination\Car Make")

def FinallyReservingParkingBySubmitButton():
   global YesAvailable
   if ((YesAvailable == "YES") and (len(str(FullNameEntry.get())) > 3) and (
   len(str(CarLicensePlateNoEntry.get())) > 3) and (len(str(EmailAddressEntry.get())) > 3) and (
   len(str(PhoneNumberEntry.get())) > 3) and (len(str(DateEntry.get())) > 3) and (len(str(NoOfParkingSpaces.get())) >= 1)):
       result = messagebox.askquestion("Book Now",
                                       "Are You Sure You Want To Book This?(Once Booked, It can not be cancelled)",
                                       icon='warning')

[panoss]

I think you should do it with a getter - setter:

class MyClasss():
   def __init__(self):      
       self.__YesAvailable = ''

   def setYesAvailable (self, new_value):
       self.__YesAvailable = new_value

   def getYesAvailable (self,):
       return self.__YesAvailable

  
   def SearchingForSpace():
       YesAvailable = MyClasss.getYesAvailable(self)
       if ((YesAvailable == "True") and (len(str(FullNameEntry.get())) > 3) and

[snippsat]

I think you should do it with a getter - setter:

No no no
Python Is Not Java

Getters and setters are evil. Evil, evil, I say! Python objects are not Java beans. Do not write getters and setters. This is what the 'property' built-in is for. And do not take that to mean that you should write getters and setters, and then wrap them in 'property'. That means that until you prove that you need anything more than a simple attribute access, don't write getters and setters. They are a waste of CPU time, but more important, they are a waste of programmer time. Not just for the people writing the code and tests, but for the people who have to read and understand them as well.

[panoss]

Ok, you 're telling us what he should 't do, but not what he should.

My second suggestion (haven't tried):

class MyClasss():
   def __init__(self):      
       self.YesAvailable = ''


  def SearchingForSpace():
     if ((self.YesAvailable == "True") and (len(str(FullNameEntry.get())) > 3) and 
      
      

[nilamo]

You're still using global variables, though. And comparing with a string, instead of just setting it as a bool.

If there's shared information, that information should either...
a) be passed around as an argument/return value, or
b) these should be methods on a class, and YesAvailable should just be an instance variable.

Which option you choose is a choice between project scope and personal preference.

[snippsat]

My second suggestion (haven't tried):

Yes and can make it look better and something that run.

class MyClass():
    def __init__(self, available):
        self.available = available

    def space_search(self, name_entry):
        if self.available == True and name_entry.get('hello') > 3:
            return 'This is True'

Use:

>>> available = 1
>>> name_entry = {'hello': 999}
>>> obj = MyClass(available)
>>> print(obj.space_search(name_entry))
This is True

@nilamo advice is okay

@Safi133 your code dos not look good at all.
Look how i do naming in the short example over and look/use at PEP-8.