特定の文字列があれば指定の箇所の文字列を書き換える、ができません....

前提・実現したいこと

ECのカスタマイズでコードを修正しようとしています。
やりたいことは、OPENPRICEという文字列をもっている商品のみ販売価格→オープンプライスと価格を書き換えたいのですが、うまくいきません。どこがおかしいのでしょうか？

発生している問題・エラーメッセージ

該当のソースコード

    <!-- 商品1 -->
    <div class="AE_txt_head">販売価格：1,000円</div>
    <!-- 商品2 -->
    <div class="AE_txt_head">販売価格：1,000円</div>
    <div class="free7">OPENPRICE</div>
    <!-- 商品3 -->
    <div class="AE_txt_head">販売価格：1,000円</div>
</body>

</html>

＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝

【皆さんに教えてもらい解決したコード↓↓】

<!DOCTYPE html> <html lang="ja"> <head> <meta charset="UTF-8"> <script src="https://ajax.googleapis.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script> <title>testpage</title> </head> <body> <style> .free7{display: none;} </style> <script> $(function(){ $("div:contains('OPENPRICE')").prev('.AE_txt_head') .each(function(){ var txt= $(this).html(); $(this).html(txt.replace('販売価格：','オープンプライス：')); }); }); </script>  <div class="AE_txt_head">参考販売価格：1,000円</div>  <div class="AE_txt_head">参考販売価格：1,000円</div> <div class="free7">OPENPRICE</div>  <div class="AE_txt_head">参考販売価格：1,000円</div> </body> </html>

kei344

2019/11/29 04:15

回答が付いた質問の編集は慎重に行ってください。質問文のコードについて回答にて指摘があった場合は「追記」し、元のコードを編集する場合も「直したこと」がわかるようにしてください。後から見た人から見て、編集前にされた回答が「無いコードに対しての指摘」になり、意味がわからなくなります。

行動規範の内容に同意します

回答3件

どこがおかしいのでしょうか？

エラーメッセージのとおりです。$("div:contains('OPENPRICE')")という関数呼び出しの直後に{は続きません。

投稿2019/11/28 06:22

maisumakun

総合スコア145183

izm_nanomate

2019/11/28 08:18 編集

勉強不足で恐縮です。修正したところ、下記の様になりエラーはなくなりました。しかし肝心の文言が差し変わりません。※正確にいうと常にオープンプライスになっている状態です。なぜでしょうか...？ <script> $(function(){ if ($("div:contains('OPENPRICE')")) $('.AE_txt_head').each(function(){ var txt= $(this).html(); $(this).html(txt.replace('販売価格：','オープンプライス：')); }); else {; } }); </script> <div class="AE_txt_head">参考販売価格：</div> <div class="free7">OPENPRICE</div>

行動規範の内容に同意します

) が足りない。

投稿2019/11/28 06:31

kei344

総合スコア69400

izm_nanomate

2019/11/28 08:08

)不足理解できました！ちゃんと閉じてみました。

行動規範の内容に同意します

ベストアンサー

if を使う意味がわかりません。
HTML が提示されていないので正確な回答はできませんが、次のようにすればいいのでは？

javascript
1$(function(){
2	$("div:contains('OPENPRICE') .AE_txt_head").each(function(){
3		var txt= $(this).html();
4		$(this).html(txt.replace('販売価格：','オープンプライス：'));
5	});
6});

追記

.prev() を使えばできます。

javascript
1$(".div:contains('OPENPRICE') .AE_txt_head")

この行をいじればいいだけなので、あとはご自身で書いてください。

投稿2019/11/29 02:16

編集2019/11/29 04:45

2KOH

総合スコア999

izm_nanomate

2019/11/29 02:25

ifは必要ないのでしょうか？...(初心者ですみません) 試しに頂いたコードでやってみましたが、常に販売価格が表示されました。 HTMLの提示、失礼しました下記全容になります↓ =============================== <!DOCTYPE html> <html lang="ja"> <head> <meta charset="UTF-8"> <script src="https://ajax.googleapis.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script> <title>testpage</title> </head> <body> <style> .free7{display: none;} </style> <script> $(function(){ $("div:contains('OPENPRICE') .AE_txt_head") .each(function(){ var txt= $(this).html(); $(this).html(txt.replace('販売価格：','オープンプライス:')); }); }); </script> <div class="AE_txt_head">販売価格：</div> <div class="free7">OPENPRICE</div> </body> </html> ===============================