Удаляйте сразу все роли, зачем по одной
if after.channel:
await member.remove_roles(etaj1, etaj2, etaj3, ...)
Ну и я бы не делал сразу 20 запросов на получение всех ролей, если выполнится условие
before.channel, то в нем не используется ни одна из 10 переменных, на создание которых ушло время.
Ну и неплохо было бы в конструкции if использовать elif, чтоб не проверять по сто раз условия, которые точно не выполнятся.
