Use try-except instead of checking for membership.

Basically, a lot of the checks revolving "if x in y" could be written
more efficiently by doing the task anyway and ignoring the exception
a la the EAFP guideline.

1 files changed, 8 insertions, 2 deletions

diff --git a/discord/client.py b/discord/client.py
index 12bc6766..9d1ae27c 100644
--- a/discord/client.py
+++ b/discord/client.py
@@ -311,8 +311,11 @@ class ConnectionState(object):
         if server is not None:
             user_id = data['user']['id']
             member = utils.find(lambda m: m.id == user_id, server.members)
-            if member in server.members:
+            try:
                 server.members.remove(member)
+            except ValueError:
+                return
+            else:
                 self.dispatch('member_remove', member)
 
     def handle_guild_member_update(self, data):
@@ -364,8 +367,11 @@ class ConnectionState(object):
             self.dispatch('server_unavailable', server)
             return
 
-        if server in self.servers:
+        try:
             self.servers.remove(server)
+        except ValueError:
+            return
+        else:
             self.dispatch('server_remove', server)
 
     def handle_guild_role_create(self, data):