Python: проверка «словаря» пуста не работает

Я пытаюсь проверить, является ли словарь пустым, но он не ведет себя должным образом. Он просто пропускает его и отображает онлайн без чего-либо, кроме отображения сообщения. Есть идеи почему?

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

Вот три способа проверить, является ли dict пустым. Я предпочитаю использовать только первый способ. Два других способа слишком многословны.

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

если длина равна нулю, значит, что dict пуст

Простые способы проверить пустой dict ниже:

        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

Что мне нравится в get, так это то, что он не вызывает исключение, поэтому он позволяет легко проходить через большие структуры.

Почему бы не использовать тест на равенство?

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