Подписка на события через атрибуты HTML-элементов плоха двумя вещами:
1) Происходит смешивание разметки и поведения. Это значит, что для того, чтобы разобраться, что происходит, надо попеременно заглядывать в *.html и в *.js, больше шансов что-то нечаянно сломать или допустить глупую ошибку типа опечатки.
2) Через атрибут больше одного обработчика не повесишь.
Однако, при всем вышесказанном, это может быть удобно, если кода крайне мало. Если у вас вся страничка состоит из калькулятора с тремя полями и двумя кнопками, нет смысла подключать туда фреймворк и фанатично соблюдать все лучшие практики.
в файлах вк заметил
Разработчики VK, несмотря на невероятную раскрученность сервиса, обыкновенные говнякеры, по крайней мере, в части фронтенда.