Leon-on12,
document.getElementById() это нет смысла менять, остальное если работает, то лучше не трогать
иногда
getElements - удобно тем что всегда содержит актуальные данные , querySelectorAll делает снимок на момент поиска.
из-за актуальности getElements если что-то меняют то идут с конца коллекции(это так на всякий)i=elem.length - 1