特定の条件で、非表示にするJSの書き方

Question

(.a1)というクラスを持つ要素のhref属性内に拡張子(.png)が含まれていたら、対象のクラス名(a2,a3)のコーナーを非表示にしたいです
以下で書いてみましたが、うまくいかずご教示いただきたいですmm

-非表示したいコーナーについて追記-
a1とは別の要素を対象として、a2,a3の両方のクラスを持つ要素の全体を非表示にしたいです

$(".a1").each(function(){
  if ($(this).attr('href').match('png')) {
    $(this).removeClass("a2 a3");
  }
});

Comments

[yambejp]

> (png)が含まれていたら

が曖昧です。textContent内？属性も含む？
大文字小文字の違いは？拡張子じゃなくても全部ヒットする？
・・・など

[hii0824]

追加しましたmmm

[itagagaki]

「クラス名の中に」ではないですよね。
「a1というクラスを持つ要素のhref属性内に」ですか？

あとできればタグに JQuery を追加してください。

[itagagaki]

まだ「対象のクラス名(a2,a3)のコーナーを非表示」の意味が不明ですね。
ここで処理している「a1を持つ要素」とは関係なく、a2またはa3のどちらかまたは両方のクラスを持つ別の要素を対象として何か操作したいということですか？
で、コーナーを非表示とは、上記の条件に合致する要素の全体を非表示にしたいということですか？

[hii0824]

修正しました。
よろしくお願いしますmm

Answer 1

評価の仕方はこんな感じです

<script>
$(function(){
  var html=$(".a1").map(function(){
    return $(this).attr('href');
  }).get().join("\n");
  console.log(html);
  $('.a2,.a3').toggle(!/.png$/mi.test(html));
});
</script>


<a class="a1" href="hoge.jpg">1</a>
<a class="a1" href="fuga.png">2</a>
<a class="a1" href="piyo.gif">3</a>
<div class="a2">a2-1</div>
<div class="a2">a2-2</div>
<div class="a3">a3</div>
<div class="a4">a4</div>

Answer 2

複数の誤りがあると思います。
まずgauさんが指摘されているpngは'png'ではないのか？ということ。
そしてreplaceが意味不明（おそらくそこでもエラーが出ているはず）。
やりたいのはa2とa3というクラスの追加ですか？
それならこうなります。

$(".a1").each(function(){
  if ($(this).html().match('png')) {
    $(this).addClass("a2 a3");
  }
});

---

追記

「a1というクラスを持つ要素のhref属性内にpngという文字列が含まれていたらa2とa3というクラスを削除する」のならこうです。

$(".a1").each(function(){
  if ($(this).attr('href').match('png')) {
    $(this).removeClass("a2 a3");
  }
});

質問の編集を受けてさらに追記

「a1というクラスを持つ要素のhref属性内にpngという文字列が含まれていたら
a2というクラスとa3というクラスを両方持っている要素を非表示にする」

$(".a1").each(function(){
  if ($(this).attr('href').match('png')) {
    $(".a2"+".a3").hide();
  }
});

Comments

[hii0824]

ありがとうございますmm
a2 a3を非表示にしたく[replace]を使いました＞＜

[hii0824]

ありがとうございますm
うまくいかずで、、
説明不足で申し訳ないのですが、対象のクラス名のコーナーを削除したい、が正しいです

Answer 3

if ($(this).html().match(png)) {
                         ↑

ここでエラーでてませんか？
この書き方だとpngという変数を表します
'png'ではないでしょうか