[Implemented] Unlimited Separator$ length @ StringField()

[AND51]

Hello!

My favourite command StringField() can only work correctly, if Separator$ has only 1 character.

Can this be extended? It'd be nice, if Separator$ can have any length!

Concerning this topic, you might have a look at the german forum-thread: http://www.purebasic.fr/german/viewtopi ... D51#127649

[remi_meier]

Or a debugger check to help people avoid this problem. One could also
change the type of the separator to CHARACTER.

But I agree, it would be VERY useful to allow any length of string as
Separator$.

[Fluid Byte]

Agreed!

I want this really bad!

[Konne]

But it would me much slower.

[Dummy]

But it would me much slower.

only if it's badly coded or you use seperator "ae" on a verly long string filled with 50% "a"s

[AND51]

But it would me much slower.

That's not the matter.

If the PB Team has no other idea, the PB team will use one big If/Else, checking if Separator$'s length is =1 or >1. So, the perfomance doesn not decrease.



@ PB Team: What do you think about my idea? As you can see, there are many other ppl who aagree with me.

[Trond]

Agreed!

I want this really bad!

I want it good, if you don't mind?

[mskuma]

I agree this would be useful.. here's hoping for it to appear in the future.. also agree 2 versions of the function should be provided (or internally detected) if the >1 char version is known to be slower than the 1 char version.

[Kiffi]

Agreed too!

Greetings ... Kiffi

[akj]

So we're all agreed then. It's a must-have feature for PureBasic 13.0

[AND51]

Not PB 13.0 but next PB version or PB update.

[JCV]

yup its a nice addition.

[Konne]

Didn't we talk about this and Fred said that it is a stupid idea?.
I don't know.

[PB]

I'd just do it like this, with a non-standard text character such as Chr(1) as a placeholder.
So, use StringField for 1 character delimiters, and StringFieldBig for >1 character delimiters.

Code: Select all

Procedure.s StringFieldBig(string$,index,delimiter$)
  string$=ReplaceString(string$,delimiter$,Chr(1))
  ProcedureReturn StringField(string$,index,Chr(1))
EndProcedure

For k=1 To 6
  Debug StringFieldBig("Hello---I---am---a---split---string", k, "---")
Next

[AND51]

Smaller, faster, better. Morever, you forgot to protect the variable, that's bad (think of EnableExplicit).

Code: Select all

Macro StringFieldBig(string,index,delimiter) 
   StringField(ReplaceString(string,delimiter,Chr(1)) ,index,Chr(1)) 
EndMacro

For k=1 To 6 
  Debug StringFieldBig("Hello---I---am---a---split---string", k, "---") 
Next 

I know, how to code a better Stringfield; but this iis not the point. I would like too see an improved StringField in the next Version natively!