While trying to input my API key python is giving me a line too long code
E501: line too long
What I have is
notifications_client = NotificationsAPIClient(aaaaaaa_aaaaaaaa-11aa1a1a-aa11-111a-aaaa-11111aaa1a1a-aa11a1a1-0aa1-11a1-1111-1aa111a0a111)
For obvious reasons I have changed the API key to have only a’s 1’s and 0’s but how can I break up this line of code so I no longer get this error?
asked Nov 5, 2018 at 16:21
3
E501 is a linter error, not a Python interpreter error. Your code, in theory, should work just fine. If you want to prevent this error, simply break the value up (assuming it’s a string … you don’t make that clear):
my_key = ('aaaaaaa_aaaaaaaa-11aa1a1a-aa11-111a-aaaa-'
'11111aaa1a1a-aa11a1a1-0aa1-11a1-1111-'
'1aa111a0a111')
notifications_client = NotificationsAPIClient(my_key)
answered Nov 5, 2018 at 16:26
Jonah BishopJonah Bishop
12.1k5 gold badges47 silver badges72 bronze badges
E501 is not a python error, rather than a PEP8 error. Meaning your line is longer than 80 chars (in your case it’s 137 chars long).
Your editor or runtime are verifying that your code is correct by PEP8 rules and that’s why you are getting this «error». Your Python code has actually no errors at all.
If you want your code to be PEP8 compliant I suggest:
- Extract the API key to a local variable.
- If it’s still too long you can break up the string into multiple lines
Here is an example:
API_KEY = 'aaaaaaa_aaaaaaaa-11aa1a1a-aa11-111a'
'-aaaa-11111aaa1a1a-aa11a1a1-0aa1-'
'11a1-1111-1aa111a0a111'
notifications_client = NotificationsAPIClient(API_KEY)
answered Nov 5, 2018 at 16:31
YuvalYuval
1,2792 gold badges12 silver badges33 bronze badges
Use to break your line. Like;
notifications_client = NotificationsAPIClient(aaaaaaa_aaaaaaaa-11aa1a1a-
aa11-111a-aaaa-11111aaa1a1a-
aa11a1a1-0aa1-11a1-1111-1aa111a0a111)
answered Nov 5, 2018 at 16:27
ChaosPredictorChaosPredictor
3,5301 gold badge32 silver badges44 bronze badges
Option which doesn’t involved breaking the string literal:
notifications_client = NotificationsAPIClient(
"kkkkkkkkkkkkkeeeeeeeeeeeeeeeeeeeeeeeeeeyyyyyyyyyyyyyyyyyyyyy"
)
So long as your key is <73 (minus scope indentation) characters long. If not, you’ll have to split it.
answered Nov 5, 2018 at 16:31
While trying to input my API key python is giving me a line too long code
E501: line too long
What I have is
notifications_client = NotificationsAPIClient(aaaaaaa_aaaaaaaa-11aa1a1a-aa11-111a-aaaa-11111aaa1a1a-aa11a1a1-0aa1-11a1-1111-1aa111a0a111)
For obvious reasons I have changed the API key to have only a’s 1’s and 0’s but how can I break up this line of code so I no longer get this error?
asked Nov 5, 2018 at 16:21
3
E501 is a linter error, not a Python interpreter error. Your code, in theory, should work just fine. If you want to prevent this error, simply break the value up (assuming it’s a string … you don’t make that clear):
my_key = ('aaaaaaa_aaaaaaaa-11aa1a1a-aa11-111a-aaaa-'
'11111aaa1a1a-aa11a1a1-0aa1-11a1-1111-'
'1aa111a0a111')
notifications_client = NotificationsAPIClient(my_key)
answered Nov 5, 2018 at 16:26
Jonah BishopJonah Bishop
12.1k5 gold badges47 silver badges72 bronze badges
E501 is not a python error, rather than a PEP8 error. Meaning your line is longer than 80 chars (in your case it’s 137 chars long).
Your editor or runtime are verifying that your code is correct by PEP8 rules and that’s why you are getting this «error». Your Python code has actually no errors at all.
If you want your code to be PEP8 compliant I suggest:
- Extract the API key to a local variable.
- If it’s still too long you can break up the string into multiple lines
Here is an example:
API_KEY = 'aaaaaaa_aaaaaaaa-11aa1a1a-aa11-111a'
'-aaaa-11111aaa1a1a-aa11a1a1-0aa1-'
'11a1-1111-1aa111a0a111'
notifications_client = NotificationsAPIClient(API_KEY)
answered Nov 5, 2018 at 16:31
YuvalYuval
1,2792 gold badges12 silver badges33 bronze badges
Use to break your line. Like;
notifications_client = NotificationsAPIClient(aaaaaaa_aaaaaaaa-11aa1a1a-
aa11-111a-aaaa-11111aaa1a1a-
aa11a1a1-0aa1-11a1-1111-1aa111a0a111)
answered Nov 5, 2018 at 16:27
ChaosPredictorChaosPredictor
3,5301 gold badge32 silver badges44 bronze badges
Option which doesn’t involved breaking the string literal:
notifications_client = NotificationsAPIClient(
"kkkkkkkkkkkkkeeeeeeeeeeeeeeeeeeeeeeeeeeyyyyyyyyyyyyyyyyyyyyy"
)
So long as your key is <73 (minus scope indentation) characters long. If not, you’ll have to split it.
answered Nov 5, 2018 at 16:31
It looks like tokenize doesn’t generate a ‘n’ token for lines with the final newline escaped, so maybe_check_physical doesn’t run check_physical:
(1, 'open', (1, 0), (1, 4), "open('long_long_long_long_long_long_long_long_long_long', mode='w+', buffering=1024) \n")
(51, '(', (1, 4), (1, 5), "open('long_long_long_long_long_long_long_long_long_long', mode='w+', buffering=1024) \n")
(3, "'long_long_long_long_long_long_long_long_long_long'", (1, 5), (1, 56), "open('long_long_long_long_long_long_long_long_long_long', mode='w+', buffering=1024) \n")
(51, ',', (1, 56), (1, 57), "open('long_long_long_long_long_long_long_long_long_long', mode='w+', buffering=1024) \n")
(1, 'mode', (1, 58), (1, 62), "open('long_long_long_long_long_long_long_long_long_long', mode='w+', buffering=1024) \n")
(51, '=', (1, 62), (1, 63), "open('long_long_long_long_long_long_long_long_long_long', mode='w+', buffering=1024) \n")
(3, "'w+'", (1, 63), (1, 67), "open('long_long_long_long_long_long_long_long_long_long', mode='w+', buffering=1024) \n")
(51, ',', (1, 67), (1, 68), "open('long_long_long_long_long_long_long_long_long_long', mode='w+', buffering=1024) \n")
(1, 'buffering', (1, 69), (1, 78), "open('long_long_long_long_long_long_long_long_long_long', mode='w+', buffering=1024) \n")
(51, '=', (1, 78), (1, 79), "open('long_long_long_long_long_long_long_long_long_long', mode='w+', buffering=1024) \n")
(2, '1024', (1, 79), (1, 83), "open('long_long_long_long_long_long_long_long_long_long', mode='w+', buffering=1024) \n")
(51, ')', (1, 83), (1, 84), "open('long_long_long_long_long_long_long_long_long_long', mode='w+', buffering=1024) \n")
(1, 'or', (2, 4), (2, 6), " or ''n")
(3, "''", (2, 7), (2, 9), " or ''n")
(4, 'n', (2, 9), (2, 10), " or ''n")
Motivation
I am working on fixing some of the issues raised by flake8 (a Python Linter) in a Python-based backend repository and thought it would be nice to discuss some of the common issues and the solutions that I gathered from the web (well, mostly StackOverflow). The use of an auto formatter such as black will help resolve some of these common issues automatically. flake8rules is an excellent resource of a complete list of issues as well.
line too long (92 > 79 characters)flake8(E501)
Line too long issues mainly happen for the following cases:
- string
- if-statement
- method chaining
- parameter list
…
I was going to explain with examples how to use Python’s implied line continuation inside parentheses, brackets and braces but decided not to. Nowadays I chose to leave it to my auto formatter to do the job.
For those who insist to write code without any helpful plugins or IDE support, I would like to share that practice does make perfect. I used Vim for a period of writing Java code without autocomplete or format-on-save. I ran style checks and manually fixed issues raised such as having a space between operators. After a month or two, these things became second nature and I was pretty happy with the ability to write well-formatted code without any help. I suppose that was an interesting experience so go ahead and try it yourself.
do not use bare ‘except’flake8(E722)
Example:
def create(item):
try:
item("creating")
except:
pass
Enter fullscreen mode
Exit fullscreen mode
Fix:
def create(item):
try:
item("creating")
except Exception:
pass
Enter fullscreen mode
Exit fullscreen mode
Explanation:
Bare except will catch exceptions you almost certainly don’t want to catch, including KeyboardInterrupt (the user hitting Ctrl+C) and Python-raised errors like SystemExit
A better fix:
- Think about what exact exception to catch and specify that instead of just catching any exception.
- Think about whether this exception handling is necessary and are you unintentionally using it for control flow?
- When catching an exception, use either logging or other proper resolutions to handle the anticipated error.
‘from some_package_name_here import *’ used; unable to detect undefined names flake8(F403)
I thought this is an interesting topic for discussion. Earlier in my coding journey, I was amused by the many lines of import statements found in some scripts. Sometimes the number of import statements outweigh the number of practical code within the same file.
Nowadays I know better than to fear abstractions (I still fear BAD abstractions and rightfully so). However, with the help of powerful IDEs, importing and tracing the variable/function to their imported package is easier than before. The problem with ‘from xxx import *’ is that it is unclear what has been imported. Following this, IDEs could not decide whether some of the undefined variables come from the package that you imported or they are errors.
Example
from package_a import *
from package_b import *
# suppose both packages included a function named pretty_print
# it is unclear which method is invoked below
pretty_print("abc")
Enter fullscreen mode
Exit fullscreen mode
Fix
from package_a import pretty_print
from package_b import other_function_required
pretty_print("abc")
Enter fullscreen mode
Exit fullscreen mode
Conclusion
When browsing an unfamiliar code repository, we tend to have less sentimental feelings and that fresh perspective allows us to see the good, the bad, and the evil. Besides learning from the good practices, the hacky and the code standard violations (and things like commented out code) are excellent places to start reviewing concepts of coding styles and coding standards, to find out why code misbehaved.
That’s all for now.
External resources:
- Guide to setup black on VSCode
- Using black with flake8