Python: Memeriksa apakah 'Kamus' kosong sepertinya tidak berfungsi

Saya mencoba memeriksa apakah kamus kosong tetapi tidak berperilaku baik. Itu hanya melompati dan menampilkan ONLINE tanpa apa pun kecuali menampilkan pesan. Ada ide mengapa?

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

Kamus kosong mengevaluasi keFalse dalam Python:

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

Dengan demikian, isEmptyfungsi Anda tidak perlu. Yang perlu Anda lakukan adalah:

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

Berikut adalah tiga cara Anda dapat memeriksa apakah dikte kosong. Saya lebih suka menggunakan cara pertama saja. Dua cara lainnya terlalu bertele-tele.

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

jika panjangnya nol berarti dict kosong

Cara-cara sederhana untuk memeriksa dict kosong adalah di bawah ini:

        a= {}

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

Meskipun metode 1 lebih ketat seperti ketika a = Tidak ada, metode 1 akan memberikan hasil yang benar tetapi metode 2 akan memberikan hasil yang salah.

Kamus dapat secara otomatis dilemparkan ke boolean yang mengevaluasi Falseuntuk kamus kosong dan kamus Truetidak kosong.

if myDictionary: non_empty_clause()
else: empty_clause()

Jika ini terlihat terlalu idiomatis, Anda juga dapat menguji len(myDictionary)nol, atau set(myDictionary.keys())untuk set kosong, atau hanya menguji kesetaraan dengan {}.

Fungsi isEmpty tidak hanya tidak perlu tetapi juga implementasi Anda memiliki beberapa masalah yang dapat saya temukan prima-facie.

1. The return Falsepernyataan lebih menjorok satu tingkat terlalu dalam. Itu harus di luar for loop dan pada tingkat yang sama dengan forpernyataan. Akibatnya, kode Anda hanya akan memproses satu, kunci yang dipilih secara sewenang-wenang, jika ada kunci. Jika kunci tidak ada, fungsi akan kembali None, yang akan dilemparkan ke boolean False. Aduh! Semua kamus kosong akan diklasifikasikan sebagai false-nagatives.
2. Jika kamus tidak kosong, maka kode hanya akan memproses satu kunci dan mengembalikan nilainya ke boolean. Anda bahkan tidak dapat berasumsi bahwa kunci yang sama dievaluasi setiap kali Anda menyebutnya. Jadi akan ada positif palsu.
3. Katakanlah Anda mengoreksi lekukan return Falsepernyataan dan membawanya di luar forlingkaran. Maka yang Anda dapatkan adalah boolean ATAU semua kunci, atau Falsejika kamus kosong. Tetap saja Anda akan memiliki positif palsu dan negatif palsu. Lakukan koreksi dan uji terhadap kamus berikut untuk bukti.

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

Anda juga dapat menggunakan get (). Awalnya saya percaya hanya memeriksa apakah kunci ada.

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

Yang saya suka dengan get adalah itu tidak memicu pengecualian, sehingga membuatnya mudah untuk melintasi struktur besar.

Mengapa tidak menggunakan tes kesetaraan?

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

gunakan 'apa saja'

dict = {}

if any(dict) :

     # true
     # dictionary is not empty 

else :

     # false 
     # dictionary is empty