А вообще очень странная функция, которая каждый раз пробегается по DOM'у... Проще уж, если нужно, закэшировать в data-атрибуты