Python: vérifier si un «dictionnaire» est vide ne semble pas fonctionner

J'essaie de vérifier si un dictionnaire est vide mais il ne se comporte pas correctement. Il saute simplement et affiche EN LIGNE sans rien, sauf d'afficher le message. Des idées pourquoi?

 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())) 

Les dictionnaires vides sont évaluésFalse en Python:

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

Ainsi, votre isEmptyfonction n'est pas nécessaire. Il vous suffit de:

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()))

Voici trois façons de vérifier si le dict est vide. Je préfère cependant utiliser la première façon. Les deux autres façons sont beaucoup trop verbeuses.

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()))

si la longueur est nulle signifie que dict est vide

Voici des moyens simples de vérifier un dict vide:

        a= {}

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

Bien que la méthode 1 soit plus stricte que lorsque a = None, la méthode 1 fournira un résultat correct mais la méthode 2 donnera un résultat incorrect.

Un dictionnaire peut être automatiquement converti en booléen qui évalue Falsepour un dictionnaire vide et Truepour un dictionnaire non vide.

if myDictionary: non_empty_clause()
else: empty_clause()

Si cela semble trop idiomatique, vous pouvez également tester len(myDictionary)pour zéro, ou set(myDictionary.keys())pour un ensemble vide, ou simplement tester l'égalité avec {}.

La fonction isEmpty n'est pas seulement inutile, mais votre implémentation a également plusieurs problèmes que je peux repérer à première vue.

1. La return Falsedéclaration est en retrait d'un niveau trop profond. Elle doit être en dehors de la boucle for et au même niveau que l' forinstruction. Par conséquent, votre code ne traitera qu'une seule clé sélectionnée arbitrairement, si une clé existe. Si une clé n'existe pas, la fonction retournera None, qui sera convertie en booléen False. Aie! Tous les dictionnaires vides seront classés comme faux-nagatifs.
2. Si le dictionnaire n'est pas vide, le code ne traitera qu'une seule clé et renverra sa valeur en booléen. Vous ne pouvez même pas supposer que la même clé est évaluée chaque fois que vous l'appelez. Il y aura donc des faux positifs.
3. Disons que vous corrigez l'indentation de la return Falsedéclaration et la forsortez de la boucle. Ensuite, ce que vous obtenez est le booléen OU de toutes les clés, ou Falsesi le dictionnaire est vide. Vous aurez toujours de faux positifs et de faux négatifs. Effectuez la correction et testez le dictionnaire suivant pour obtenir des preuves.

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

Vous pouvez également utiliser get (). Au début, je pensais qu'il ne vérifiait que si la clé existait.

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

Ce que j'aime avec get, c'est qu'il ne déclenche pas d'exception, donc il est facile de parcourir de grandes structures.

Pourquoi ne pas utiliser le test d'égalité?

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

utilisez «tout»

dict = {}

if any(dict) :

     # true
     # dictionary is not empty 

else :

     # false 
     # dictionary is empty