search

felromo / animetube

smtube inspired anime viewer written in python
Other
8 stars 1 forks source link

Variable names #8

Open RomeuG opened 9 years ago

RomeuG commented 9 years ago

Variable names should change! Problems arise because of the naming system you're using. An example:

In file main.py, line 45: parser = parser.parser()

It's confusing, and there are naming conflicts. parser is a class, parser is also a variable.

I recommend giving SnakeCase type of names to Classes, and using more descriptive names for variables.

felromo commented 9 years ago

Haven't been working on this project in some time, if you submit a patch, I'll be glad to review and merge it