Python： 'Dictionary'が空かどうかの確認が機能しないようです

辞書が空かどうかを確認しようとしていますが、正しく動作しません。メッセージを表示するだけで、それをスキップして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())) 

空の辞書はPythonで評価さFalseれます：

>>> 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が空かどうかを確認する方法は3つあります。ただし、最初の方法のみを使用することをお勧めします。他の2つの方法は非常に冗長です。

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が空であることを意味します

空の辞書をチェックする簡単な方法は以下の通りです：

        a= {}

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

a = Noneの場合のようにメソッド1stはより厳密ですが、メソッド1は正しい結果を提供しますが、メソッド2は誤った結果を提供します。

辞書はブール値に自動的にキャストできFalse、空の辞書と空でない辞書を評価しTrueます。

if myDictionary: non_empty_clause()
else: empty_clause()

これがあまりにも慣用的に見える場合はlen(myDictionary)、ゼロまたはset(myDictionary.keys())空のセットをテストするか、単にと等しいかどうかをテストすることもできます{}。

isEmpty関数は不必要であるだけでなく、実装にも複数の問題があり、一見してわかります。

1. return Falseステートメントが深すぎる1つのレベルにインデントされます。これは、forループの外側で、forステートメントと同じレベルにある必要があります。その結果、キーが存在する場合、コードは任意に選択された1つのキーのみを処理します。キーが存在しない場合、関数はを返しNone、ブール値のFalseにキャストされます。痛い！空の辞書はすべて偽陰性として分類されます。
2. 辞書が空でない場合、コードは1つのキーのみを処理し、その値キャストをブール値に返します。同じキーが呼び出されるたびに評価されるとは限りません。したがって、誤検知があります。
3. return Falseステートメントのインデントを修正して、forループの外に移動したとします。次に、すべてのキーのブールOR、または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