💥🩹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
Owner

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.

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.
alexis.drai added this to the Model milestone 2 years ago
alexis.drai added the help wanted label 2 years ago
najlae.lambaraa was assigned by alexis.drai 2 years ago
ismail.taha_janan was assigned by alexis.drai 2 years ago
alexis.drai self-assigned this 2 years ago
alexis.drai added 1 commit 2 years ago
continuous-integration/drone/push Build is passing Details
dcb9de97e5
🩹 Pass params by vals instead of by refs
alexis.drai requested review from najlae.lambaraa 2 years ago
alexis.drai requested review from ismail.taha_janan 2 years ago
alexis.drai added this to the dice_app_KANBAN project 2 years ago
Poster
Owner

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.

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.
alexis.drai changed title from WIP: 🩹 Pass params by vals instead of by refs to WIP: 💥🩹Pass params by vals instead of by refs 2 years ago
alexis.drai changed title from WIP: 💥🩹Pass params by vals instead of by refs to 💥🩹Pass params by vals instead of by refs 2 years ago
alexis.drai removed review request for najlae.lambaraa 2 years ago
alexis.drai removed review request for ismail.taha_janan 2 years ago
alexis.drai closed this pull request 2 years ago
alexis.drai removed the help wanted label 2 years ago
najlae.lambaraa was unassigned by alexis.drai 2 years ago
ismail.taha_janan was unassigned by alexis.drai 2 years ago
alexis.drai deleted branch pass-by-val-unless-assign-new-value 2 years ago
All checks were successful
continuous-integration/drone/push Build is passing
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Loading…
There is no content yet.