Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Global variable should not using in class method #11232

Closed
icecraft opened this issue Jan 10, 2024 · 3 comments · Fixed by #11276
Closed

Global variable should not using in class method #11232

icecraft opened this issue Jan 10, 2024 · 3 comments · Fixed by #11276
Labels
bug

Comments

@icecraft
Copy link

Repository commit

9caf478

Python version (python --version)

python3.11

Dependencies version (pip freeze)

#nop

Expected behavior

Hi, guys. I come across one global variable that used in class method. I think using instance attribute would be better !

    def show_data(self):
        show_list = []
        for i in range(1, N + 1):  ==> should using self.N instead of N !
            show_list += [self.query(i, i)]
        print(show_list)

source

Actual behavior

    def show_data(self):
        show_list = []
        for i in range(1, self.N + 1):  
            show_list += [self.query(i, i)]
        print(show_list)
@icecraft icecraft added the bug label Jan 10, 2024
Merged
@GantedaAravind

This comment was marked as off-topic.

Sign in to view
GantedaAravind added a commit to GantedaAravind/Python that referenced this issue Feb 12, 2024
@GantedaAravind
fixes TheAlgorithms#11232 issue replaces global variable by instance …
52e274d 
…variable
@GantedaAravind GantedaAravind mentioned this issue Feb 12, 2024
Closed
@zakariatahiri

This comment was marked as off-topic.

Sign in to view
@smruthi-sumanth

This comment was marked as off-topic.

Sign in to view
MannCode added a commit to MannCode/Python that referenced this issue Feb 25, 2024
@MannCode
MannCode:TheAlgorithms#11232. Refactor show_data method to use instan…
55c1faa 
…ce attribute self.N instead of global variable N for better encapsulation and readability.
@MannCode MannCode mentioned this issue Feb 25, 2024
Closed
8 tasks
@JyotikaJayani-08 JyotikaJayani-08 mentioned this issue May 1, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
4 participants
@icecraft @zakariatahiri @smruthi-sumanth @GantedaAravind