Update player_actions.cairo

onchain/src/systems/player_actions.cairo

@@ -36,16 +36,13 @@ mod PlayerActions {
             let caller: ContractAddress = get_caller_address();
             let zero_address: ContractAddress = contract_address_const::<0x0>();
 
-            // assert(caller != zero_address, 'Cannot create from zero address');
             assert(caller != zero_address, Errors::ADDRESS_ZERO);
 
-
             let new_player: Player = PlayerTrait::new(username, caller);
 
             // Ensure player username is unique
             let mut existing_player = get!(world, username, (Player));
 
-            // assert(existing_player.owner == 0.try_into().unwrap(), 'username already taken');
             assert(existing_player.owner == 0.try_into().unwrap(), Errors::USERNAME_TAKEN);
 
             set!(world, (new_player));
@@ -57,7 +54,6 @@ mod PlayerActions {
             let mut player: Player = get!(world, username, (Player));
 
             // Validate username
-            // assert(player.owner != 0.try_into().unwrap(), 'player with username not found');
             assert(player.owner != 0.try_into().unwrap(), Errors::USERNAME_NOT_FOUND);
 
             player.owner
@@ -86,17 +82,15 @@ mod PlayerActions {
             let mut player: Player = get!(world, old_username, (Player));
 
             // Only allow the player to update their own username
-            // assert(player.owner == caller, 'only user can udpate username');
             assert(player.owner == caller, Errors::ONLY_OWNER_USERNAME);
 
             // Check if the new username is already taken
             let mut existing_player = get!(world, new_username, (Player));
-            // assert(existing_player.owner == 0.try_into().unwrap(), 'username already exist');
             assert(existing_player.owner == 0.try_into().unwrap(), Errors::USERNAME_EXIST);
 
             // Update the player's username
             player.username = new_username;
             set!(world, (player));
         }
     }
-}
+}