Just playing around with tool making. I have a couple steps left to do to incorporate a couple of switch parameters I’ve included but, generally, with what’s there so far, thoughts? What could be done smarter or cleaner? Would love any feedback you can share!
Looks good and there doesn’t appear to be any errors. The only thing I would suggest and this is just a preference of mine, is to use white space to make the code more readable. Other than that, if you want to maybe shorten a few lines that have parameters that are being passed, look into splatting. Also, good job documenting and using regions to break up the code sections!
Thanks Bill! I understand the spacing preference. I found, when I really started using regions, it was easier for me to keep it tight, start off reading with everything minimized, and then just walk through one small portion at a time. As for the splatting, good call. My screen at home is a little wider so I hadn’t noticed. I went back and looked after your comment and I can see some overruns there. I’ll check out cleaning that up. Thanks for taking the time!
[quote quote=138949]Thanks Bill! I understand the spacing preference. I found, when I really started using regions, it was easier for me to keep it tight, start off reading with everything minimized, and then just walk through one small portion at a time. As for the splatting, good call. My screen at home is a little wider so I hadn’t noticed. I went back and looked after your comment and I can see some overruns there. I’ll check out cleaning that up. Thanks for taking the time!
[/quote]
If you are using vscode, there’s a module you need to get and use to help you with splatting the easy way. I just found it today and it’s awesome:
Good deal. For what I’m working on right now, still hanging around in PS ISE. Will be converting over in the next couple of months. Bookmarking for the future. Thank you!
You may want to define the [string]$logPath variable outside the Try/Catch block, or even outside the ForEach block; this need to be declared only once.
The $OutGridView variable is defined but no used (yet?). One possible other option would be to output all results to a file (CSV or text log file) in case you run that script against a very large number of computers.
It’s more a personal preference; but some messages are very verbose (“Thank you. This script will now halt.”) (with a typo on line 69 by the way).
Depending who is running that script and how often, you may want to reduce this. Pointing to the built-in help should be sufficient.
I completely agree on the log file and changed it just last night. Sloppy. Not sure what my original thinking was that made that make sense. I’ve adjusted a little, bringing it both up front and optional to change (parameter with default but can enter something else).
Regarding OutGridView, hadn’t built in… was one of the things I meant by it not yet being fully written out yet. Do have that there now.
And regarding the deep commenting and verbose options, it’s on purpose for the audience intended. Agreed that in an environment of folks generally friendly/comfortable with PowerShell, pretty fluffy, but meant to cater to a less familiar group and let them know more of what’s going on both in code and/or progress.
Will post updated soon. Reviewing a few more things first.