正規表現：「"abc//def//ghi//jk"」を「"abcdefghijk"」にしたい

Question

先日以下の質問をした者です。
https://teratail.com/questions/105223

また躓きました。
「"abc//def//ghi//jk"」を「"abcdefghijk"」にしたいのですが、
「"abc//def//ghi//jk"」を「"abcdef//ghi//jk"」まではできるのですが、すべての//を削除できません。
text = Regex.Replace(text, "(".+?)//(.+?")", "$1$2");
だと1つ目の//のみ削除となってしまいます。
複数の//を削除する方法が分かりましたらご教授ください。

Answer 1

タグでは「正規表現」とだけありますので、言語を問わない解決法をお望みと解釈します。
正規表現で2回マッチさせれば、期待通りに動作します。

1. 「"」で括られた文字列にマッチ
2. 1. でマッチした文字列から「/」を全削除

'use strict';
'"abc//def//ghi//jk"'.replace(/"[^"]+"/g, function (string) {
  return string.replace(//+/g, '');
});  // "abcdefghijk"

Re: uematsu-satoshi さん

Comments

[Satoshi71]

javascriptだと関数が埋め込めますね。こちらもありがとうございました。

[think49]

専門外ですが、C#にも同じ機能があるように読めます。
https://msdn.microsoft.com/ja-jp/library/ht1sxswy.aspx
ただ、伝えかったのはそういうことではなくて、アルゴリズムの方です。
アルゴリズムは言語を問わずに利用できるものであり、より良いアルゴリズムを選択できる自由があります。
replaceにコールバック機能がなくとも、「マッチした文字列」と「マッチしなかった文字列」に分解して、同じ動作が実現できれば良いのです。
純粋な正規表現だけの質問、とは「特定の言語機能に依存しない質問」だと思います。

ベストアンサーに選ばれたコードは "abc"--//--"def" に誤爆してしまう("の外側も置換する)のではないか、と不安です。
文字列全体を何度も置換する関係で効率も良くないですね。

[Satoshi71]

ありがとうございます。
今まさに最後にご指摘いただいた問題に直面しました。
> "abc"--//--"def" に誤爆
ただ、ご教授いただいたループで回して何とかならないかなって思っています。
textをIndexOFで分解して、"を1個おきに処理するみたいな。（あるいはtextを"で配列化して要素を１個おきに処理するとか）
速度上に問題が出そうですが、ちょっとやってみます。

Answer 2

置換はプログラミング言語によって方法が異なりますので、言語を明示する必要があります。
先の質問を見ると、C#ですかね。
text = text.Replace("//","");

#追記
なるほど、読み取れてませんでした。
テストしてませんがこんな感じで。

while(true){
  work = Regex.Replace(text, "(\".+?)//(.+?\")", "$1$2");
  if(work == text) {
    break;
  }
  text = work;
}

Comments

[Satoshi71]

すみません、おっしゃる通りC#です。
また、１つ言い忘れましたが、"と"の間に挟まれている//をすべて消したいというのが趣旨です。
text = text.Replace("//","");
だと、たしかにすべて消すことができますが、text内すべての//が消えてしまいます。
"と"の間に挟まれている//をすべて消したい場合はどのようにすればよろしいでしょうか。
先に示した
text = Regex.Replace(text, "(\".+?)//(.+?\")", "$1$2");
だと、"と"の間に挟まれている//を検知してくれますが、あくまで最初の1つだけで、複数の//を消してくれません。

よろしくお願いいたします。

[Satoshi71]

ありがとうございました！できました。