Review for inital release

[christof-ochs]

No description provided.

[BeckerStS]

@christof-ochs

THX for the contribution :)

Review-comments:

• pls. add some tests to the repository
• pls. adjust the namespace to fit the Namespace of the community like "Simatic.Ax.StringBuilder"
  (this might be debateable @sjuergen what you think ?, community pkg's that extent the "system" ?)
• pls. describe more in details why using this is more convenient in comparison to the native system.string usage for the user (point out the benefits)
• pls. mention the available methods on the Readme.md
• optional: would be great if you could add the other string operations as methods as well

Greetings
Stefan

[BeckerStS]

committed a fix to the example program in the Readme.md

[christof-ochs]

@BeckerStS thank you for the feedback.
I adjusted the following topics in the last commits:

• added tests
• describe benefit of using fluent interface in Readme
• add table of methods to the readme

I would appreciate if you could clarify the namespace topic. I would prefer the current extension of the System.Strings NS

[sjuergen]

@BeckerStS @christof-ochs
Regarding the namespace: System.Strings is already used in the system libraries on @ax. That could be lead to a misunderstanding from my point of view. Another reason: What happens when the System.Strings library will be extended by a string builder class?

[christof-ochs]

@sjuergen
The way you phrased that it sounds like you assumed the namespace is identical by accident. Just to make clear my intend: I selected it by choice. The reasoning is, that the functionality exclusively works on the content of that library and extends it.

but if you want to have a different ns, i can of course adapt it.

[sjuergen]

I have a preference. But I'm open for other arguments. I understand your point. I would like to discuss it with the team. @theBadT have you any opinion on it?

[christof-ochs]

@BeckerStS I also added the other string operations from the System.Strings library