refactor(tempvc): use list comprehension and asyncio vs loops

Author: amadaluzia

tux/cogs/services/temp_vc.py

@@ -1,3 +1,5 @@
+import asyncio
+
 import discord
 from discord.ext import commands
 
@@ -34,15 +36,16 @@ async def on_voice_state_update(
         # Ensure CONFIGants are set correctly
         temp_channel_id = int(CONFIG.TEMPVC_CHANNEL_ID or "0")
         temp_category_id = int(CONFIG.TEMPVC_CATEGORY_ID or "0")
-        if temp_channel_id == 0 or temp_category_id == 0:
+
+        if 0 in {temp_category_id, temp_channel_id}:
             return
 
         # When user joins the temporary voice channel
         if after.channel and after.channel.id == temp_channel_id:
             await self._handle_user_join(member, after.channel)
 
         # When user leaves any voice channel
-        elif before.channel:
+        if before.channel:
             await self._handle_user_leave(before.channel, after.channel, temp_channel_id, temp_category_id)
 
     async def _handle_user_join(
@@ -61,11 +64,15 @@ async def _handle_user_join(
             The channel that the member joined.
         """
 
-        for voice_channel in channel.guild.voice_channels:
-            # Check if the channel is a temporary channel and if it is the user's channel
-            if voice_channel.name == self.base_vc_name + member.name:
-                await member.move_to(voice_channel)
-                return
+        # for voice_channel in [v for v in channel.guild.voice_channels if v.name == self.base_vc_name + member.name]:
+        #     await member.move_to(voice_channel)
+        #     return
+        tasks = [
+            member.move_to(voice_channel)
+            for voice_channel in channel.guild.voice_channels
+            if voice_channel.name == self.base_vc_name + member.name
+        ]
+        asyncio.gather(*tasks)
 
         # Create a new channel for the user if it doesn't exist
         new_channel = await channel.clone(name=self.base_vc_name + member.name)
@@ -107,19 +114,16 @@ async def _handle_user_leave(
             return
 
         # Delete the channel if it is empty
-        if len(before_channel.members) == 0:
+        if before_channel.members == []:
             await before_channel.delete()
 
         # Search and delete all empty temporary channels
-        for channel in category.voice_channels:
-            if (
-                not channel.name.startswith(self.base_vc_name)
-                or len(channel.members) != 0
-                or channel.id == temp_channel_id
-            ):
-                continue
-
-            await channel.delete()
+        tasks = [
+            channel.delete()
+            for channel in category.voice_channels
+            if channel.name.startswith(self.base_vc_name) and channel.members == [] and channel.id == temp_channel_id
+        ]
+        await asyncio.gather(*tasks)
 
 
 async def setup(bot: Tux) -> None: