javascriptのスマートな記述を知りたい

Question

下記は意図した動きにはなりますが、やや冗長なコードな気がします。
もう少し短くなる書き方やアイデアがあれば、参考にしたいです。

現在のソースコード

jQuery(window).on('load', function(){
$("li a:contains('aaa')").parent("li").addClass("is-active");
$("li a:contains('bbb')").parent("li").addClass("is-active");
$("li a:contains('ccc')").parent("li").addClass("is-active");
});

Comments

[m.ts10806]

コードだけで要件までくみ取るのは他者には困難です。
「何を目的としたアプリケーションなのか」
「制限事項はあるのか」
「HTMLは」
など、赤の他人が手を出しやすいように配慮願います。

[Zuishin]

https://www.jquerystudy.info/reference/selectors/multiple2.html

or 条件を使えばいいんじゃないでしょうか。

[terataimn]

参考サイト、ありがとうございます。参考にさせていただきます。

Answer 1

jQuery(window).on('load', function(){
$("li a:contains('aaa'),li a:contains('bbb'),li a:contains('ccc')")
  .parent("li").addClass("is-active");
});

とか、

jQuery(window).on('load', function(){
$("li:has(a:contains('aaa'),a:contains('bbb'),a:contains('ccc'))")
  .addClass("is-active");
});

Comments

[terataimn]

ありがとうございます。参考にさせていただきます。

Answer 2

行数は増えそうですがcontainsの中身だけの差異ですしまずは関数化して
引数だけ変えるようにしてみるとかですかね。
呼び出し元は少しシンプルになるかなと思います。

Comments

[terataimn]

ありがとうございます。参考にさせていただきます。