Несмотря на то, что некоторые люди могут сказать, что это неплохая практика (с учетом того, что это единственный прослушиватель событий, который вам нужен для этого элемента), и это самый простой (и наиболее распространенный) кросс-браузерный способ добавить прослушиватель событий, но я бы сделал два изменения.
Прежде всего, если Javascript не включен, ссылка будет бесполезной (хотя ссылка #
заставит браузер прокрутить страницу вверх, что, вероятно, нежелательно). Точно так же при включенном Javascript нажатие на ссылку все равно заставит браузер перейти по ссылке и прокрутить вверх.
Вместо этого я бы использовал что-то вроде этого:
<a href="[url to JS-less way of doing the same thing]" onclick="doSomething(); return false">Click here</a>
С другой стороны, если это действительно только JavaScript, вы можете сделать ссылку скрытой по умолчанию с помощью CSS и использовать Javascript, чтобы сделать ее видимой (чтобы пользователи с отключенным JS не увидели бесполезную ссылку).