obj.onclick = shuffle();가 shuffle 함수를 호출하는것이니깐
클릭시 shuffle함수가 호출되어야하므로
obj.onclick = shuffle; 이 아니라 obj.onclick = shuffle();가 되어야하는거 아닌가요?