Python: Sprawdzanie, czy „Słownik” jest pusty, nie działa

Próbuję sprawdzić, czy słownik jest pusty, ale nie działa poprawnie. Po prostu pomija go i wyświetla ONLINE bez niczego oprócz wyświetlania wiadomości. Jakieś pomysły dlaczego?

 def isEmpty(self, dictionary):
   for element in dictionary:
     if element:
       return True
     return False

 def onMessage(self, socket, message):
  if self.isEmpty(self.users) == False:
     socket.send("Nobody is online, please use REGISTER command" \
                 " in order to register into the server")
  else:
     socket.send("ONLINE " + ' ' .join(self.users.keys())) 

Puste słowniki oceniająFalse w Pythonie:

>>> dct = {}
>>> bool(dct)
False
>>> not dct
True
>>>

Twoja isEmptyfunkcja jest więc niepotrzebna. Wszystko, co musisz zrobić, to:

def onMessage(self, socket, message):
    if not self.users:
        socket.send("Nobody is online, please use REGISTER command" \
                    " in order to register into the server")
    else:
        socket.send("ONLINE " + ' ' .join(self.users.keys()))

Oto trzy sposoby sprawdzenia, czy dykt jest pusty. Wolę jednak używać tylko pierwszego sposobu. Pozostałe dwa sposoby są zbyt trudne.

test_dict = {}

if not test_dict:
    print "Dict is Empty"


if not bool(test_dict):
    print "Dict is Empty"


if len(test_dict) == 0:
    print "Dict is Empty"

dict = {}
print(len(dict.keys()))

jeśli długość wynosi zero, oznacza to, że dykt jest pusty

Poniżej znajdują się proste sposoby sprawdzenia pustego nagrania:

        a= {}

    1. if a == {}:
           print ('empty dict')
    2. if not a:
           print ('empty dict')

Chociaż metoda 1 jest bardziej rygorystyczna, niż gdy a = Brak, metoda 1 zapewni poprawny wynik, ale metoda 2 da niepoprawny wynik.

Słownik może być automatycznie rzutowany na boolean, którego wynikiem jest Falsepusty słownik i Trueniepusty słownik.

if myDictionary: non_empty_clause()
else: empty_clause()

Jeśli to wygląda zbyt idiomatycznie, możesz również przetestować len(myDictionary)na zero lub set(myDictionary.keys())na pusty zestaw, lub po prostu przetestować na równość z {}.

Funkcja isEmpty jest nie tylko niepotrzebna, ale także twoja implementacja ma wiele problemów, które mogę rozpoznać na pierwszy rzut oka.

1. return FalseOświadczenie jest wcięty jeden poziom zbyt głęboko. Powinien znajdować się poza pętlą for i na tym samym poziomie, co forinstrukcja. W rezultacie Twój kod przetworzy tylko jeden, dowolnie wybrany klucz, jeśli klucz istnieje. Jeśli klucz nie istnieje, funkcja zwróci wartość None, która zostanie przekazana na wartość logiczną False. Auć! Wszystkie puste słowniki zostaną sklasyfikowane jako fałszywe.
2. Jeśli słownik nie jest pusty, kod przetworzy tylko jeden klucz i zwróci jego wartość rzutowaną na boolean. Nie można nawet założyć, że ten sam klucz jest oceniany przy każdym wywołaniu. Będą więc fałszywe alarmy.
3. Powiedzmy, że poprawiłeś wcięcie return Falseinstrukcji i wyprowadzasz ją poza forpętlę. Otrzymasz więc logiczne OR wszystkich kluczy lub Falsejeśli słownik jest pusty. Nadal będziesz mieć fałszywe pozytywy i fałszywe negatywy. Wykonaj korektę i przetestuj w oparciu o następujący słownik w celu uzyskania dowodów.

myDictionary={0:'zero', '':'Empty string', None:'None value', False:'Boolean False value', ():'Empty tuple'}

Możesz także użyć get (). Początkowo wierzyłem, że sprawdza tylko, czy klucz istnieje.

>>> d = { 'a':1, 'b':2, 'c':{}}
>>> bool(d.get('c'))
False
>>> d['c']['e']=1
>>> bool(d.get('c'))
True

To, co lubię w get, to to, że nie wyzwala wyjątku, więc ułatwia przechodzenie przez duże struktury.

Dlaczego nie skorzystać z testu równości?

def is_empty(my_dict):
    """
    Print true if given dictionary is empty
    """
    if my_dict == {}:
        print("Dict is empty !")

użyj „dowolne”

dict = {}

if any(dict) :

     # true
     # dictionary is not empty 

else :

     # false 
     # dictionary is empty