Return preset from Member::find_guild if possible

If a value is present for `Member::guild_id` when calling
`Member::find_guild`, return that value instead of searching the cache.

This can potentially save a large amount of calculations and time.

1 files changed, 9 insertions, 0 deletions

diff --git a/src/model/guild/member.rs b/src/model/guild/member.rs
index 630610f..5da7d7c 100644
--- a/src/model/guild/member.rs
+++ b/src/model/guild/member.rs
@@ -163,15 +163,24 @@ impl Member {
 
     /// Finds the Id of the [`Guild`] that the member is in.
     ///
+    /// Returns the value of [`Member::guild_id`] if present. Otherwise searches
+    /// the [`Cache`] for the Id.
+    ///
     /// # Errors
     ///
     /// Returns a [`ClientError::GuildNotFound`] if the guild could not be
     /// found.
     ///
+    /// [`Cache`]: ../ext/cache/struct.Cache.html
     /// [`ClientError::GuildNotFound`]: ../client/enum.ClientError.html#variant.GuildNotFound
     /// [`Guild`]: struct.Guild.html
+    /// [`Member::guild_id`]: struct.Member.html#structfield.guild_id
     #[cfg(feature="cache")]
     pub fn find_guild(&self) -> Result<GuildId> {
+        if let Some(guild_id) = self.guild_id {
+            return Ok(guild_id);
+        }
+
         for guild in CACHE.read().unwrap().guilds.values() {
             let guild = guild.read().unwrap();