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

While, should be do while. #5

Open
jwallTopps opened this issue Apr 13, 2022 · 1 comment
Open

While, should be do while. #5

jwallTopps opened this issue Apr 13, 2022 · 1 comment

Comments

@jwallTopps
Copy link

Hey, thank you very much for this example. It's super helpful for a project I'm working on. To help out a little bit, I wanted to point out a tiny issue in your code.

Unfortunately this bit here will skip the first property, as you are immediately calling sp.NextVisible(true) right after getting the first property. You might want to change this to do{ ... }while(sp.NextVisible(true)); so it will process the first iterator before advancing.

@Ommand
Copy link

Ommand commented Aug 24, 2022

In fact, even if the code skips the first property, it does not affect the final result, since the first property are always seems to be "Base" (of type Generic), which is of no interest.

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

2 participants