[Implemented] ReplaceString: Add opt. para. [,NbOccurences]

[IceSoft]

String$ = ReplaceString(String$, StringToFind$, StringToReplace$ [, Mode [, StartPosition]])

I cannot say: Replace only the first occurence of a search string.

[Fred]

Where is the bug ?

[IceSoft]

ReplaceString iterating trough the whole string...I miss the occurence parameter for stopping the iteration e.g. after the first replace

Code: Select all

s1.s = "abcdeabcdeabcdeabcde"
s2.s = "ab"
Debug s1
String$ = ReplaceString(s1, s2, "AB" , #PB_String_InPlace, 1)
Debug s1 ; Result: ABcdeABcdeABcdeABcde, but I want: ABcdeabcdeabcdeabcde

ReplaceString have also have to have the NbOccurences parameter as the RemoveString has:

Code: Select all

String$ = RemoveString(String$, StringToRemove$ [, Mode [, StartPosition [, NbOccurences]]])

[Arctic Fox]

Isn't this a feature request then?

[IceSoft]

Isn't this a feature request then?

Yes for all who not using ReplaceString at the moment.
If you use it...than it is a bug.

[Rings]

moved....

[IceSoft]

moved....

Please add this optional parameter. You know: it is more a wrong design as a feature wish.

[freak]

Just because you want it doesn't make it wrong not to have it

[Seymour Clufley]

+1

[IceSoft]

Just because you want it doesn't make it wrong not to have it

In a real programmer life I can find more examples on which you want ONLY replace one occurence.

So my wish is:
Add a ReplaceString which is not searching through a string...a replace on a given position is enough.

For the actual implemented ReplaceString... please add the forgotten optional parameter:[, NbOccurences].
For example see RemoveString(...):
String$ = RemoveString(String$, StringToRemove$ [, Mode [, StartPosition [, NbOccurences]]])