Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Can't get past illogical line pep8 error

I've been trying to fix this for a while now and i just can't get it to pass pep8. Here is my code:

1.

if (len(regex) > 2 and regex[0] == '(' and regex[-1] == ')' and 
    sum(regex.count(char) for char in splitter) == 1 and 
    regex.count('(') == 1 and regex.count(')') == 1):

    print('hi')

2.

if (len(regex) > 2 and regex[0] == '(' and regex[-1] == ')' and 
    sum(regex.count(char) for char in splitter) == 1 and 
    regex.count('(') == 1 and regex.count(')') == 1):

    print('hi')

3.

if (len(regex) > 2 and regex[0] == '(' and regex[-1] == ')' 
    and regex.count('(') > 1):

    print('hi')

I get the following PEP8 error on each of the 3 if statements:

E125 continuation line does not distinguish itself from next logical line

Any idea on what's wrong with it? The lines are indented with the first bracket so i really don't have a clue.

like image 519
user3050527 Avatar asked Mar 20 '14 01:03

user3050527


2 Answers

1.

if (len(regex) > 2 and regex[0] == '(' and regex[-1] == ')' and
        sum(regex.count(char) for char in splitter) == 1 and
        regex.count('(') == 1 and regex.count(')') == 1):

    print('hi')

2.

if (len(regex) > 2 and regex[0] == '(' and regex[-1] == ')' and
        sum(regex.count(char) for char in splitter) == 1 and
        regex.count('(') == 1 and regex.count(')') == 1):

    print('hi')

3.

if (len(regex) > 2 and regex[0] == '(' and regex[-1] == ')'
        and regex.count('(') > 1):

    print('hi')
like image 102
user3412839 Avatar answered Oct 25 '22 12:10

user3412839


I'm using PyCharm (which is pretty good for pointing out PEP8 errors) for my editing, and it says this version is ok:

if (len(regex) > 2 and regex[0] == '(' and regex[-1] == ')' and
        sum(regex.count(char) for char in splitter) == 1 and
        regex.count('(') == 1 and regex.count(')') == 1):

    print('hi')
like image 25
Steinar Lima Avatar answered Oct 25 '22 12:10

Steinar Lima