💥🩹Pass params by vals instead of by refs #50
Closed
alexis.drai
wants to merge 0 commits from pass-by-val-unless-assign-new-value
into main
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'pass-by-val-unless-assign-new-value'
Deleting a branch is permanent. It CANNOT be undone. Continue?
if we pass parameters by reference, it must be because we want to
change them in the method and to keep those changes.
This seems like a different situation, so we should be able
to pass the objects by value.
Je vous invite à regarder mon code et à le valider SVP.
Mon code modifie encore une fois l'interface IManager 😅 Je suis désolé si ça provoque des changements sur vos branches, mais j'espère que ça sera facile à gérer, et je pense que ça va faciliter le développement pendant le reste du projet.
Comme je disais dans le message de commit, si on passe à une méthode des paramètres par référence (mot-clé
ref
), typiquement il faut une bonne raison : en général, c'est qu'on veut pouvoir changer l'objet passé par référence à l'intérieur de cette méthode, et que les changements persistent en dehors de cette méthode.https://learn.microsoft.com/en-us/dotnet/csharp/programming-guide/classes-and-structs/passing-reference-type-parameters#passing-reference-types-by-reference
Ca marche avec les
ref
dans les params, mais je crois que ce n'est pas nécessaire. On n'a pas besoin de modifier ces objets, donc on peut simplifier un peu le code.WIP: 🩹 Pass params by vals instead of by refsto WIP: 💥🩹Pass params by vals instead of by refs 3 years agoWIP: 💥🩹Pass params by vals instead of by refsto 💥🩹Pass params by vals instead of by refs 3 years ago