Python: Controllare se un 'Dizionario' è vuoto non sembra funzionare

Sto cercando di verificare se un dizionario è vuoto ma non si comporta correttamente. Lo salta e visualizza ONLINE senza nulla tranne che per visualizzare il messaggio. Qualche idea sul perché?

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

I dizionari vuoti valutanoFalse in Python:

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

Pertanto, la tua isEmptyfunzione non è necessaria. Tutto quello che devi fare è:

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

Ecco tre modi per verificare se dict è vuoto. Preferisco usare il primo modo solo però. Gli altri due modi sono troppo prolissi.

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

se la lunghezza è zero significa che dict è vuoto

Di seguito sono riportati modi semplici per controllare un dict vuoto:

        a= {}

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

Sebbene il metodo 1 sia più rigoroso di quando a = Nessuno, il metodo 1 fornirà il risultato corretto ma il metodo 2 darà un risultato errato.

Un dizionario può essere automaticamente convertito in booleano, che valuta Falseper dizionario vuoto e dizionario Truenon vuoto.

if myDictionary: non_empty_clause()
else: empty_clause()

Se questo sembra troppo idiomatico, puoi anche testare lo len(myDictionary)zero, o set(myDictionary.keys())per un set vuoto, o semplicemente testare l'uguaglianza con {}.

La funzione isEmpty non è solo superflua, ma anche la tua implementazione ha molteplici problemi che posso individuare prima facie.

1. La return Falsefrase è rientrata di un livello troppo profondo. Dovrebbe essere al di fuori del ciclo for e allo stesso livello fordell'istruzione. Di conseguenza, il codice elaborerà solo una chiave, selezionata arbitrariamente, se esiste una chiave. Se non esiste una chiave, la funzione tornerà None, che verrà lanciata su false booleane. Ahia! Tutti i dizionari vuoti saranno classificati come falsi nagativi.
2. Se il dizionario non è vuoto, il codice elaborerà solo una chiave e restituirà il suo valore cast a booleano. Non puoi nemmeno supporre che lo stesso tasto venga valutato ogni volta che lo chiami. Quindi ci saranno falsi positivi.
3. Diciamo che correggi il rientro dell'istruzione return Falsee portalo fuori dal forciclo. Quindi quello che ottieni è il booleano OR di tutte le chiavi, o Falsese il dizionario è vuoto. Avrai comunque falsi positivi e falsi negativi. Esegui la correzione e verifica il seguente dizionario per una prova.

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

Puoi anche usare get (). Inizialmente ho creduto di controllare solo se esistesse la chiave.

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

Quello che mi piace di get è che non fa scattare un'eccezione, quindi rende facile attraversare grandi strutture.

Perché non usare il test di uguaglianza?

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

usa 'qualsiasi'

dict = {}

if any(dict) :

     # true
     # dictionary is not empty 

else :

     # false 
     # dictionary is empty