書き換えがうまくいかない

質問

次はスクリプトの一部です。
var index = 0;
var name;
function newName() {
index++;
name = "_" + index.toString(16);
}
var str = "10回";
str = str.replace(/^(.+?)回(以下を繰り返す)?$/, function(times) {
newName();
return "for ("+name+"=0; "+name+"<"+times+"; "+name+"++) {"
});
document.write(str);
"for (_1=0; _1<10; _1++) {"と表示させたいのですが、
"for (_1=0; _1<10回; _1++) {"と表示されます。
*?を使っているのですが、replaceメソッドの
第２引数を文字列リテラルにするとうまくいくき、
上のように関数を使うとうまく書き換えられません。

よろしくお願いします。

回答No.1

t_wada · ベストアンサー

以下のコードでは、replaceの第二引数に、functionの実行結果ではなく、実体を渡してしまっています。

str = str.replace(/^(.+?)回(以下を繰り返す)?$/, function(times) {
newName();
return "for ("+name+"=0; "+name+"<"+times+"; "+name+"++) {"
});

また、上記コードは以下のように置き換える事も出来ます。
var v = function(times) {
newName();
return "for ("+name+"=0; "+name+"<"+times+"; "+name+"++) {"
}

str = str.replace(/^(.+?)回(以下を繰り返す)?$/, v);

こうなると判りやすいのですが、本来replaceに渡すべき値は、functionの実行結果なので、以下のように修正すると、正常に動作します。

var v = function(times) {
newName();
return "for ("+name+"=0; "+name+"<"+times+"; "+name+"++) {"
}

str = str.replace(/^(.+?)回(以下を繰り返す)?$/, v(10));

ちなみに、functionの実体を文字列に変換すると、functionの内容がそのまま渡されてしまいます。
alert(v);でどんな値がreplaceに渡ってしまっているか、内容を確認してみると良いかもしれません。

お礼

なるほど！うまくいきました。
ありがとうございます。