Ticket #30274

strreplace で、結果を格納する変数を引数に渡したときの動作

Open Date: 2012-12-07 22:44 Last Update: 2013-04-17 22:45

Reporter:
(del#1144)
Owner:
(None)
Type:
Status:
Closed
Component:
Priority:
5 - Medium
Severity:
5 - Medium
Resolution:
Won't Fix
File:
None
Vote
Score: 0
No votes
0.0% (0/0)
0.0% (0/0)

Details

r5069の変更で落ちないように対処していただいた。

仕様がどうあるべきか検討が必要。

現象:strreplace で matchstr や groupmatchstr1-9 を置き換えようとすると落ちる

Ticket History (3/7 Histories)

2012-12-07 22:44 Updated by: (del#1144)
  • New Ticket "strreplace で、結果を格納する変数を引数に渡したときの動作" created
2013-02-16 00:47 Updated by: (del#1144)
  • Details Updated
2013-02-16 01:03 Updated by: (del#1144)
Comment

内部で strmatch と同じ FindRegexStringOne() を利用しているので格納されてしまう。

「引数受け取り時にチェックしてErrSyntaxを返す」は可能でしょうか?

2013-03-01 09:01 Updated by: None
2013-03-01 13:13 Updated by: (del#1144)
2013-04-16 20:43 Updated by: doda
Comment

エラーにする事は可能ですが、エラーコードがErrSyntaxだとちょっと分かり辛いと思います。 かといって他のはもっと合わないので、エラーにするのだったらこれ用のエラーコードの新設ですかね。

あと、matchstr等への代入自体は禁止されていないので、strreplaceでだけエラーにするというのもちょっと抵抗があります。 矛盾が起きるのがstrreplaceのみなのでしょうがないと言えばそうなのですが。

例えば、

matchstr="@@HOGE@@"
strreplace matchstr 0 "[A-Z]+" "FUGA"
というコードを書いた場合は、マッチした文字列としてのmatchstrを使おうとは考えていないと思われる(*1)ので、 現状のように置き換えられた後の文字列が入る形でいいんじゃないかなと思っています。

*1: マッチした文字列としてのmatchstrを使う気があるのならば当然matchstrを二重に使おうとしている事に気づくだろうし、 その上でmatchstrにマッチした文字列が入って欲しいのならばそもそもstrmatchを使えばいい事になりますし。

2013-04-17 22:45 Updated by: (del#1144)
  • Resolution Update from None to Won't Fix
  • Ticket Close date is changed to 2013-04-17 22:45
  • Status Update from Open to Closed
Comment

わかりました。現状のままでいいと思います。

Attachment File List

No attachments

Edit

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » Login