Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add: Scope Notion - FR #51
base: master
Are you sure you want to change the base?
add: Scope Notion - FR #51
Changes from 10 commits
57e707f
702067a
d8a032e
070c292
ef557bc
2b01feb
9a837a6
70f24c8
8ab8f24
a1dc713
af0f55c
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Petite typo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je ne suis pas d'accord avec ça, scope n'est pas la traduction de « contexte » mais bien de « portée »
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Le contenu d'une fonction est déjà défini par un bloc de code en soi, donc il s'agit déjà d'un scope local déterminé par un bloc.
Enfin, du moins c'est ce qui me semble évident, contredisez moi si vous pensez le contraire 🤷
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
La notion de scope n'est pas spécifique au paradigme orienté objet, du coup j'ai un petit problème avec cette phrase.
Le Java est vraiment un mauvais exemple pour parler de scope global, car il n'y en a pas.
Les classes servent de namespace, du coup on ne peut pas réellement polluer le scope global, vu que même des constantes (
public static final String trucRandom = "LOL";
) seront scopées dans la classe dans laquelle elles sont définies.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Il ne s'agit pas là de variables globales, mais de variables d'instance.
Cet exemple est donc faux.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.