Рефакторинг JavaScript кода
- div2.innerHTML = 'Рост:<span class="height"></span>';
div4.innerHTML = 'Год рождения:<span class="birth_year"></span>';
div5.innerHTML = 'В скольки фильмах появлялся:<span class="films_count"></span>'; - это вполне себе можно вынести в отдельную функцию. Она будет принимать как аргумент select.value, и в зависимости от значения будет подставлять нужную разметку.
- Есть несколько почти одинаковых строк, например, 'Рост:<span class="height"></span>'. Меняется у них только первое слово и класс. А это значит, что получается вполне себе объект. А точнее, объект "данные для разметки", в котором хранится 3 других объекта - "люди", "корабли" и "планеты". Получится вроде people = { "рост": "height", "вес": "mass", …}.
- Дальше по аналогии) Может, там что-то можно с помощью map перебрать, или циклом пройтись просто, или ещё что-то использовать, чтобы код короче стал.