вся функция каждый раз работает с DOM'ом, подумай как переписать это таким образом, что бы операция с домом была только единожды