Skip to content

Complete week8 work#119

Open
faridsalau wants to merge 1 commit intoBlue-movement:masterfrom
faridsalau:week8
Open

Complete week8 work#119
faridsalau wants to merge 1 commit intoBlue-movement:masterfrom
faridsalau:week8

Conversation

@faridsalau
Copy link
Copy Markdown

No description provided.

@jried31
Copy link
Copy Markdown
Collaborator

jried31 commented Jul 27, 2020

From a readability perspective, and also the ability to verify sound logic it is NEVER recommended to use non-descriptive variable names (outside of for loops). I encourage you to Google "Sample Code" over the internet. Look at what you see. You will be hard pressed to find any example code that use non-descriptive single or even double letter variables.

Variable names should be concise and descriptive. Almost like I should be able to read the line (or concise comment above the line) and follow exactly what your code is doing. Look at this website:
https://realpython.com/python-pep8/ and search in the page for "How To Choose Names" --- You'll find an example of code that looks similar to what you've submitted and how they do not recommend it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants