- Mark as New
- Bookmark
- Subscribe
- Mute
- Subscribe to RSS Feed
- Permalink
- Report
I like the first part of Snippet 1, and the last half of Snippet 3 about the best. The Pick method can just be a 1 liner, because it's not overly long. The check for Nothing can also just be a 1 liner right after that, instead of a 3 line block. But I would prefer to split that really long last part of Snippet 1 into at least 2-3 parts as is done in Snippet 3. Comments are OK if the code is complicated to wrap your mind around or hard to tell what's going on if you were to read again a year later, or someone else may be inheriting your code after you have moved on. I like a good marriage between somewhat condensed, and yet still easy enough for some random mid-level, part time programmer to be able to follow what's going on later.
Wesley Crihfield
(Not an Autodesk Employee)