Почему onclick не работает?
getElementsByTagName возвращает массив элементов.
пилите цикл, это вам не jquery
getElementsByTagName('p') возвращает множество объектов,
чтобы повесить onclick надо пройтись по полученному массиву.