Можно ли написать код лаконичнее?

Question

[GreenMan1899]

$('a').click(function(e) {
  var attr = $(this).attr('href');
  if (!attr) {
      e.preventDefault();
  }
});

Comments

[emp1re]

$("a").click((e) => if (! $(this).attr('href')) e.preventDefault());

Лаконичней это короче?

[RidgeA]

можно еще так:

$("a").click((e) => !$(this).attr('href') && e.preventDefault();

Только надо ли?
С тем кодом, который в вопросе, ИМХО и так все неплохо

[GreenMan1899]

emp1re, и да и не совсем.
Мб есть какие-то способы не использовать if в этом коде.
В других похожих случаях видел такое.

[GreenMan1899]

emp1re, Но ваш ответ я тоже принимаю

[GreenMan1899]

RidgeA, да типа такого

[RidgeA]

GreenMan1899, не надо так делать.
Это был пример плохого кода.

[Сергей Соколов]

emp1re, RidgeA, в стрелочной функции же контекст остаётся внешний. У вас this будет window.

[emp1re]

Сергей Соколов, вы правы. Каюсь jquery не мое.

[kulaeff]

Костыль детектед.

Answer 1

[Сергей Соколов]

Переписать код короче – можно:

$("a").click( function(e){ $(this).attr('href') || e.preventDefault() });

Если поясните, чего добиваетесь таким кодом, можно предложить решение лучше.

Answer 2

[gleendo]

// Лаконичнее этот вакуумный кусок кода написать на pure JS: (Но это не лучшая практика, делать так все таки не стоит)

[...document.querySelectorAll("a")].forEach(link => link.addEventListener("click", event => {
  event.preventDefault();
}, false));

// К тому же...Либо вы используете ссылки с href, либо используете другие компоненты, например button. 
// Ссылка она на то и ссылка, что должна куда то ссылаться.

Comments

[Negwereth]

Поупражняемся

const blockLink = event => event.preventDefault();

[...document.getElementsByTagName("a")]
  .filter(item => !item.hasAttribute("href"))
  .forEach(link => link.addEventListener("click", blockLink, false));

[gleendo]

Negwereth,

// Тогда уж лучше так...но смысла в данном коде особого нет
Array.prototype.forEach.call(document.querySelectorAll("a"), link => !link.href ? event.preventDefault() : undefined);

[Negwereth]

iamevg_, ты постоянно упускаешь, что при аттрибуте [href] блочить не надо.

[gleendo]

Negwereth, Отрицание пропустил...