Why not? If you going to make such a bold claim (and do it anonymously), why not say why you feel this was wrong, instead of just YELLING don’t do it?

There are better ways, and I presented on below. But you criticize but can’t even bother to say why?

In my domain, I have some 600,000 users. This change meant that instead of fetching all of the accounts, only to later throw away the data on enabled or disabled, I instead only collected the information that was needed

(Get-AdUser -filter ‘Enabled -eq $true’).count # This gets the number of enabled accounts
The difference here is filtering on the AD site with a filter instead of using -Filter * and more importantly not comparing a boolean value (Enabled) to a string since casting string to bool nearly always equals $true
Same with the disabled accounts:
(Get-AdUser -filter ‘Enabled -eq $false’).count
Same stuff here. Use a filter on the AD site of things. Compare with actual boolean values. Don’t switch around the comparison ‘-ne’ vs ‘-eq’ in the first example to make the commands understandable.