Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parsing StringBuilder.ToString() #9

Open
RonnieHegelund opened this issue Nov 20, 2014 · 5 comments
Open

Parsing StringBuilder.ToString() #9

RonnieHegelund opened this issue Nov 20, 2014 · 5 comments

Comments

@RonnieHegelund
Copy link
Contributor

It should be possible to parse a stringbuilder for sql string

@dothenrik
Copy link

How would you actually find the SQL string to check for validity? Strings constructed by StringBuilders are typically the result of multiple operations. Very simple examples where the StringBuilders does not manipulate the initial SQL string could be checked, but thats not really a realistic use case.

@RonnieHegelund
Copy link
Contributor Author

I would look for values in the constructor and on the append method, yes it will only work in simple cases.

@dothenrik
Copy link

Yes, it can be done if you limit yourself to very simple scenarios. But is it really worth the effort? Have you seen many (or any at all) developers using StringBuilder for simple SQL statement construction?

@ErikEJ
Copy link
Contributor

ErikEJ commented Nov 20, 2014

Scope, boys, scope...

@RonnieHegelund
Copy link
Contributor Author

well now you can use simple stringbuilder functionality, though I agree this maybe not worth the effort!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants