Python: Перевірка, чи "Словник" порожній, здається, не працює

Я намагаюся перевірити, чи словник порожній, але він не працює належним чином. Він просто пропускає його і відображає ONLINE без нічого, крім відображення повідомлення. Будь-які ідеї чому?

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

Порожні словники оцінюютьсяFalse в Python:

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

Таким чином, ваша isEmptyфункція непотрібна. Все, що вам потрібно зробити, це:

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

Ось трьома способами можна перевірити, чи диктатура порожня. Хоча я вважаю за краще використовувати лише перший спосіб. Два інших способи є надто виразними.

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

якщо довжина дорівнює нулю, означає, що дикт порожній

Нижче прості способи перевірити порожній вигляд

        a= {}

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

Хоча метод 1-го більш суворий, як коли a = None, метод 1 дасть правильний результат, але метод 2 дасть неправильний результат.

Словник може бути автоматично переведений на булевий, який оцінює як Falseдля порожнього словника, так і Trueдля непустого словника.

if myDictionary: non_empty_clause()
else: empty_clause()

Якщо це виглядає занадто ідіоматично, ви також можете перевірити len(myDictionary)на нуль, або set(myDictionary.keys())на порожній набір, або просто перевірити на рівність {}.

Функція isEmpty не тільки непотрібна, але й у вашої реалізації є кілька проблем, які я можу помітити prima-facie.

1. return FalseЗаява відступ на один рівень занадто глибоко. Він повинен знаходитися поза циклом for і на тому ж рівні, що і forтвердження. Як результат, ваш код обробить лише один, довільно вибраний ключ, якщо ключ існує. Якщо ключа не існує, повернеться функція None, яка буде передана в булева помилка False. Ой! Усі порожні словники будуть класифіковані як хибнонаціональні.
2. Якщо словник не порожній, код буде обробляти лише один ключ і повертати його значення, передане булевим. Ви навіть не можете припустити, що той самий ключ оцінюється кожен раз, коли ви його викликаєте. Тож будуть помилкові позитиви.
3. Скажімо, ви виправляєте відступ return Falseзаяви і виводите його за межі forциклу. Тоді ви отримаєте булеве АБО всіх клавіш, або Falseякщо словник порожній. Все-таки у вас будуть помилкові позитиви та хибні негативи. Зробіть виправлення та протестуйте наступний словник для підтвердження.

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

Ви також можете використовувати get (). Спочатку я вважав це лише перевірити, чи існує ключ.

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

Мені подобається отримати те, що це не викликає винятку, тому це дозволяє легко переміщати великі структури.

Чому б не використовувати тест на рівність?

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

використовувати "будь-який"

dict = {}

if any(dict) :

     # true
     # dictionary is not empty 

else :

     # false 
     # dictionary is empty