UseShouldProcessForStateChangingFunctions should prove that a function changes system state.

Right now, the UseShouldProcessForStateChangingFunctions rule only looks at the name of the function (the verb) in order to determine that the function changes system state.

I think the rule should look at the body of the function and see if the function uses other functions that changes system state before it issues a warning.

If not, then I think that functions with the "New" verb should not be considered as functions that change system state. A lot of functions from the community with the "New" verb do not change system state, but are actually just different forms of "New-Object". See, for example, the ShowUI module.

Commencez par la règle UseShouldProcessForStateChangingFunctions et examinez comment elle utilise actuellement les noms de fonctions pour émettre des avertissements. Étudiez comment la règle pourrait inspecter les corps et les appels de fonctions, notamment s’il faudrait exclure les fonctions New-verb lorsqu’elles ne modifient pas l’état du système. Le travail est terminé lorsque la règle distingue les fonctions qui modifient l’état des fonctions de construction d’objets sans s’appuyer uniquement sur le verbe.