Add an analyzer to recommend using named parameters when passing literals as arguments
#43 988 ouverte le 2 mars 2020
Métriques du dépôt
- Stars
- (20 414 étoiles)
- Métriques de merge PR
- (Merge moyen 6j 17h) (256 PRs mergées en 30 j)
Description
We employ a guideline to always use named parameters when the meaning of an argument is not obvious from the argument itself.
For example, the intent is not very clear here, from just looking at the code:
var employees = await FetchEmployeesAsync(true);
If you're in an IDE, you can of course hover over the method call for a few seconds to get a tooltip, or go to the definition of the method, but it adds friction and delay when reading code. And oftentimes, code is read outside of an IDE (for example, when reviewing a PR or perusing code on GitHub).
But it's much clearer if you do:
var employees = await FetchEmployeesAsync(includeContractors: true);
Or:
var includeContractors = true;
var employees = await FetchEmployeesAsync(includeContractors);
For most cases, this could be formalized as "always use named parameters when passing literal values as arguments in a method call".
It would be nice to have an analyzer for this. Let me know if you think this is a good idea or not. I wouldn't mind taking a stab at implementing it myself, if it doesn't already exist and nobody else is already working on it.