'if'문에서 여러 줄 조건을 스타일링합니까? [닫은]

때로는 ifs의 긴 조건 을 여러 줄로 나눕니다. 이를 수행하는 가장 확실한 방법은 다음과 같습니다.

  if (cond1 == 'val1' and cond2 == 'val2' and
      cond3 == 'val3' and cond4 == 'val4'):
      do_something

동작이 조건과 혼합되기 때문에 시각적으로별로 매력적이지 않습니다. 그러나 4 공백의 올바른 파이썬 들여 쓰기를 사용하는 자연스러운 방법입니다.

내가 사용하는 순간 :

  if (    cond1 == 'val1' and cond2 == 'val2' and
          cond3 == 'val3' and cond4 == 'val4'):
      do_something

그러나 이것은 매우 예쁘지 않습니다. :-)

다른 방법을 추천 할 수 있습니까?

두 번째 조건부 라인에는 4 개의 공백을 사용할 필요가 없습니다. 아마도 사용 :

if (cond1 == 'val1' and cond2 == 'val2' and 
       cond3 == 'val3' and cond4 == 'val4'):
    do_something

또한 공백이 생각보다 유연하다는 것을 잊지 마십시오.

if (   
       cond1 == 'val1' and cond2 == 'val2' and 
       cond3 == 'val3' and cond4 == 'val4'
   ):
    do_something
if    (cond1 == 'val1' and cond2 == 'val2' and 
       cond3 == 'val3' and cond4 == 'val4'):
    do_something

그러나 둘 다 상당히 추악합니다.

괄호를 잃을 수도 있습니다 ( 스타일 가이드 는 이것을 권장하지 않습니다)?

if cond1 == 'val1' and cond2 == 'val2' and \
   cond3 == 'val3' and cond4 == 'val4':
    do_something

이것은 적어도 당신에게 차별화를 제공합니다.

또는:

if cond1 == 'val1' and cond2 == 'val2' and \
                       cond3 == 'val3' and \
                       cond4 == 'val4':
    do_something

나는 내가 선호하는 것 같아 :

if cond1 == 'val1' and \
   cond2 == 'val2' and \
   cond3 == 'val3' and \
   cond4 == 'val4':
    do_something

다음은 스타일 가이드 입니다 (2010 년 이후) 괄호 사용을 권장합니다.

나는 단순히 AND 또는 OR 인 퇴화 된 경우에 다음에 의지했습니다.

if all( [cond1 == 'val1', cond2 == 'val2', cond3 == 'val3', cond4 == 'val4'] ):

if any( [cond1 == 'val1', cond2 == 'val2', cond3 == 'val3', cond4 == 'val4'] ):

그것은 몇자를 면도하고 조건에 미묘함이 없음을 분명히합니다.

누군가 가 여기에서 수직 공백의 사용을 옹호해야합니다! :)

if (     cond1 == val1
     and cond2 == val2
     and cond3 == val3
   ):
    do_stuff()

이것은 각 조건을 명확하게 보여줍니다. 또한보다 복잡한 조건을보다 명확하게 표현할 수 있습니다.

if (    cond1 == val1
     or 
        (     cond2_1 == val2_1
          and cond2_2 >= val2_2
          and cond2_3 != bad2_3
        )
   ):
    do_more_stuff()

예, 우리는 명확성을 위해 약간의 수직 부동산을 거래하고 있습니다. 그만한 가치가 있습니다.

나는 매우 큰 if 조건이있을 때이 스타일을 선호합니다.

if (
    expr1
    and (expr2 or expr3)
    and hasattr(thingy1, '__eq__')
    or status=="HappyTimes"
):
    do_stuff()
else:
    do_other_stuff()

내 개인적인 견해는 다음과 같습니다. 장기 조건은 (불확실한) 부울 리턴 함수 / 메소드로 리팩토링을 제안하는 코드 냄새입니다. 예를 들면 다음과 같습니다.

def is_action__required(...):
    return (cond1 == 'val1' and cond2 == 'val2'
            and cond3 == 'val3' and cond4 == 'val4')

이제 여러 줄 조건을보기 좋게 만들 수있는 방법을 찾게되면 아마도 그 내용이 만족스럽고 리팩토링을 건너 뛸 수있을 것입니다.

반면에, 나의 미적 감각을 교란시키는 것은 리팩토링에 대한 인센티브로 작용합니다.

따라서 필자의 결론은 여러 회선 조건이보기 흉한 것으로 보이며이를 피하기위한 동기가된다는 것입니다.

이것은 크게 향상되지 않지만 ...

allCondsAreOK = (cond1 == 'val1' and cond2 == 'val2' and
                 cond3 == 'val3' and cond4 == 'val4')

if allCondsAreOK:
   do_something

and키워드를 두 번째 줄로 옮기고 조건을 포함하는 모든 줄을 4 개가 아닌 2 개의 공백으로 들여 쓰는 것이 좋습니다 .

if (cond1 == 'val1' and cond2 == 'val2'
  and cond3 == 'val3' and cond4 == 'val4'):
    do_something

이것이 바로 코드 에서이 문제를 해결하는 방법입니다. 키워드를 줄의 첫 단어로 사용하면 조건을보다 쉽게 ​​읽을 수 있으며 공간 수를 줄이면 조건과 동작을 더욱 구분할 수 있습니다.

PEP 0008 (Python의 공식 스타일 가이드)을 인용 할 가치가 있습니다 .

if문장 의 조건부 부분이 여러 줄에 걸쳐 쓰여질만큼 충분히 길면 두 문자 키워드 (예 :) if와 단일 공백, 여는 괄호의 조합이 자연스럽게 생성 된다는 점에 주목할 가치가 있습니다. 멀티 라인 조건부의 후속 라인에 대해 들여 쓰기 이것은 if-statement 안에 중첩 된 들여 쓰기 된 코드 세트와 시각적으로 충돌 할 수 있으며 , 자연스럽게 4 개의 공백으로 들여 쓰기됩니다. 이 PEP는 이러한 조건부 라인을-문 내부의 중첩 된 스위트와 시각적으로 구별하는 방법 (또는 여부)에 대한 명시적인 입장을 취하지 않습니다 if. 이 상황에서 허용되는 옵션은 다음과 같습니다.

# No extra indentation.
if (this_is_one_thing and
    that_is_another_thing):
    do_something()

# Add a comment, which will provide some distinction in editors
# supporting syntax highlighting.
if (this_is_one_thing and
    that_is_another_thing):
    # Since both conditions are true, we can frobnicate.
    do_something()

# Add some extra indentation on the conditional continuation line.
if (this_is_one_thing
        and that_is_another_thing):
    do_something()

위의 인용문에서 "제한되지 않음"에 유의하십시오. 스타일 가이드에서 제안 된 접근 방식 외에도이 질문에 대한 다른 답변에서 제안 된 접근 방식 중 일부도 허용됩니다.

여기에 내가하는 일은 "all"과 "any"가 iterable을 허용한다는 것을 기억하십시오. 그래서리스트에 긴 조건을 넣고 "all"이 작동하게합니다.

condition = [cond1 == 'val1', cond2 == 'val2', cond3 == 'val3', cond4 == 'val4']

if all(condition):
   do_something

선호하는 솔루션을 보지 못하는 것에 놀랐습니다.

if (cond1 == 'val1' and cond2 == 'val2'
    and cond3 == 'val3' and cond4 == 'val4'):
    do_something

and키워드 이기 때문에 내 편집기에서 강조 표시되며 아래의 do_something과 충분히 다르게 보입니다.

@krawyoti의 말에 덧붙여서 ... 장기적인 조건은 읽기 어렵고 이해하기 어렵 기 때문에 냄새가납니다. 함수 나 변수를 사용하면 코드가 더 명확 해집니다. 파이썬에서는 세로 공간을 사용하고 괄호를 묶고 각 행의 시작 부분에 논리 연산자를 배치하여 표현식이 "부동"처럼 보이지 않는 것을 선호합니다.

conditions_met = (
    cond1 == 'val1' 
    and cond2 == 'val2' 
    and cond3 == 'val3' 
    and cond4 == 'val4'
    )
if conditions_met:
    do_something

while루프 에서와 같이 조건을 두 번 이상 평가해야하는 경우 로컬 기능을 사용하는 것이 가장 좋습니다.

개인적으로 저는 긴 if 문에 의미를 추가하고 싶습니다. 적절한 예제를 찾으려면 코드를 검색해야하지만 다음은 염두에 두어야 할 첫 번째 예입니다. 많은 변수에 따라 특정 페이지를 표시하려는 기발한 논리가 발생한다고 가정 해 봅시다.

영어 : "로그인 한 사용자가 관리자 교사가 아니라 정규 교사이고 학생이 아닌 경우 ..."

if not user.isAdmin() and user.isTeacher() and not user.isStudent():
    doSomething()

물론 이것은 잘 보일지 모르지만 if 문을 읽는 것이 많은 작업입니다. 논리를 의미있는 레이블에 할당하는 것은 어떻습니까? "라벨"은 실제로 변수 이름입니다.

displayTeacherPanel = not user.isAdmin() and user.isTeacher() and not user.isStudent()
if displayTeacherPanel:
    showTeacherPanel()

이것은 어리석은 것처럼 보일 수 있지만 교사 패널을 표시하거나 사용자가 기본적으로 다른 특정 패널에 액세스 할 수있는 경우에만 다른 항목을 표시하려는 또 다른 조건이있을 수 있습니다.

if displayTeacherPanel or user.canSeeSpecialPanel():
    showSpecialPanel()

변수를 사용하여 논리를 저장하고 레이블을 지정하지 않고 위의 조건을 작성하십시오. 매우 복잡하고 읽기 어려운 논리 문으로 끝날뿐만 아니라 자신을 반복했습니다. 합리적인 예외가 있지만 다음을 기억하십시오 : 스스로 반복하지 마십시오 (건조).

"all"과 "any"는 동일한 유형의 여러 조건에 적합합니다. 그러나 그들은 항상 모든 조건을 평가합니다. 이 예에 표시된대로 :

def c1():
    print " Executed c1"
    return False
def c2():
    print " Executed c2"
    return False


print "simple and (aborts early!)"
if c1() and c2():
    pass

print

print "all (executes all :( )"
if all((c1(),c2())):
    pass

print

(고정 너비 이름은 실제 코드를 대표하지 않으므로 적어도 실제 코드를 나타내지 않기 때문에 식별자를 약간 수정했습니다. 최소한 실제 코드는 아닙니다.)

if (cond1 == "val1" and cond22 == "val2"
and cond333 == "val3" and cond4444 == "val4"):
    do_something

이것은 "and"및 "or"에 대해 잘 작동하지만 (두 번째 줄에있는 것이 중요 함) 다른 긴 조건에서는 훨씬 적습니다. 다행히 전자가 더 일반적인 경우 인 반면 후자는 종종 임시 변수로 쉽게 다시 작성됩니다. (일반적으로 어렵지는 않지만 다시 작성할 때 "및"/ "또는"단락을 유지하는 것은 어렵거나 훨씬 덜 명확하거나 읽을 수 있습니다.

C ++에 대한 블로그 게시물 에서이 질문을 찾았으므로 C ++ 스타일이 동일하다는 것을 포함시킬 것입니다.

if (cond1 == "val1" and cond22 == "val2"
and cond333 == "val3" and cond4444 == "val4") {
    do_something
}

평범하고 단순하며 pep8 검사도 통과합니다.

if (
    cond1 and
    cond2
):
    print("Hello World!")

최근에 나는 And와 Or 비교를 거의 섞지 않기 때문에 alland와 any함수를 선호 하고 있습니다.

if all([
    cond1,
    cond2,
]):
    print("Hello World!")

하나의 iterable을 전달하는 것을 잊지 마십시오! N- 인수를 전달하는 것이 올바르지 않습니다.

참고 : any많은 or비교 all와 마찬가지로 많은 and비교 와 같습니다 .

이것은 다음과 같은 생성기 이해와 잘 결합됩니다.

# Check if every string in a list contains a substring:
my_list = [
    'a substring is like a string', 
    'another substring'
]

if all('substring' in item for item in my_list):
   print("Hello World!")

# or

if all(
    'substring' in item
    for item in my_list
):
    print("Hello World!")

더 알아보기 : 발전기 이해

상태와 몸 사이에 빈 줄을 추가로 삽입하고 나머지는 정식 방식으로 수행하면 어떻게됩니까?

if (cond1 == 'val1' and cond2 == 'val2' and
    cond3 == 'val3' and cond4 == 'val4'):

    do_something

추신 : 나는 항상 공백이 아닌 탭을 사용합니다. 미세 조정할 수 없습니다 ...

내가 보통하는 일은 :

if (cond1 == 'val1' and cond2 == 'val2' and
    cond3 == 'val3' and cond4 == 'val4'
   ):
    do_something

이런 식으로 닫는 중괄호와 콜론은 시각적으로 우리의 상태의 끝을 표시합니다.

if 문에 대해 다중 조건을 제공하는 모든 응답자는 제시된 문제만큼이나 추악합니다. 동일한 작업을 수행하여이 문제를 해결할 수 없습니다.

PEP 0008 답변조차도 반발력이 있습니다.

훨씬 더 읽기 쉬운 접근법이 있습니다.

condition = random.randint(0, 100) # to demonstrate
anti_conditions = [42, 67, 12]
if condition not in anti_conditions:
    pass

내 말을 먹을 래? 다중 조건이 필요하다고 확신하면 말 그대로 이것을 인쇄하여 즐거움을 위해 먹을 것입니다.

@zkanda의 솔루션은 약간의 왜곡으로 좋을 것이라고 생각합니다. 자체 목록에 조건과 값이있는 경우 목록 이해를 사용하여 비교를 수행 할 수 있으므로 조건 / 값 쌍을 추가하는 데 좀 더 일반적 일 수 있습니다.

conditions = [1, 2, 3, 4]
values = [1, 2, 3, 4]
if all([c==v for c, v in zip(conditions, values)]):
    # do something

이 문장을 하드 코딩하고 싶다면 가독성을 위해 다음과 같이 작성하십시오.

if (condition1==value1) and (condition2==value2) and \
   (condition3==value3) and (condition4==value4):

그리고 iand연산자를 사용하여 다른 해결책을 제시하십시오 .

proceed = True
for c, v in zip(conditions, values):
    proceed &= c==v

if proceed:
    # do something

완전성을 위해 몇 가지 다른 임의의 아이디어. 그들이 당신을 위해 일한다면, 그들을 사용하십시오. 그렇지 않으면 다른 것을 시도하는 것이 좋습니다.

사전을 사용하여이 작업을 수행 할 수도 있습니다.

>>> x = {'cond1' : 'val1', 'cond2' : 'val2'}
>>> y = {'cond1' : 'val1', 'cond2' : 'val2'}
>>> x == y
True

이 옵션은 더 복잡하지만 유용 할 수도 있습니다.

class Klass(object):
    def __init__(self, some_vars):
        #initialize conditions here
    def __nonzero__(self):
        return (self.cond1 == 'val1' and self.cond2 == 'val2' and
                self.cond3 == 'val3' and self.cond4 == 'val4')

foo = Klass()
if foo:
    print "foo is true!"
else:
    print "foo is false!"

그것이 당신에게 효과적이라면 Dunno이지만, 고려해야 할 또 다른 옵션입니다. 한 가지 더 방법이 있습니다.

class Klass(object):
    def __init__(self):
        #initialize conditions here
    def __eq__(self):
        return (self.cond1 == 'val1' and self.cond2 == 'val2' and
               self.cond3 == 'val3' and self.cond4 == 'val4')

x = Klass(some_values)
y = Klass(some_other_values)
if x == y:
    print 'x == y'
else:
    print 'x!=y'

마지막 두 가지는 테스트하지 않았지만 원하는 개념이라면 개념이 충분합니다.

(그리고 레코드의 경우, 이것이 한 번만 발생하면 처음에 제시 한 방법을 사용하는 것이 좋습니다. 많은 곳에서 비교를 수행하는 경우 이러한 방법으로 가독성을 향상시킬 수 있습니다 그들이 해 키다는 사실에 대해 너무 나쁘게 느끼지 않습니다.)

나는 이것도 할 수있는 적절한 방법을 찾기 위해 고심하고 있었으므로 아이디어 (주로 맛의 문제이기 때문에은 총알이 아님)를 생각해 냈습니다.

if bool(condition1 and
        condition2 and
        ...
        conditionN):
    foo()
    bar()

나는이 솔루션에서 내가 본 다른 솔루션과 비교하여 몇 가지 장점을 발견합니다. 즉, 정확히 4 개의 들여 쓰기 공간 (bool)을 얻으므로 모든 조건을 세로로 정렬 할 수 있으며 if 문의 본문을 들여 쓸 수 있습니다 분명한 방법. 이것은 또한 부울 연산자의 단락 평가의 이점을 유지하지만, 기본적으로 아무것도 수행하지 않는 함수 호출의 오버 헤드를 추가합니다. 인수를 반환하는 모든 함수가 부울 대신 여기에서 사용될 수 있다고 주장 할 수는 있지만, 내가 말했듯이 그것은 단지 아이디어 일 뿐이며 궁극적으로 맛의 문제입니다.

이 문제를 작성하고 "문제"에 대해 생각 하면서 함수 호출의 오버 헤드를 제거하는 또 다른 아이디어를 생각해 냈습니다 . 여분의 괄호 쌍을 사용하여 복잡한 조건에 들어갔다고 표시해 보시겠습니까? if 문의 본문과 관련하여 하위 조건을 멋지게 2 칸 들여 쓰기 위해 2 개를 더 말하십시오. 예:

if (((foo and
      bar and
      frob and
      ninja_bear))):
    do_stuff()

당신이 그것을 볼 때, "이봐 요, 복잡한 일이 일어나고 있습니다!" . 예, 괄호가 가독성에 도움이되지 않는다는 것을 알고 있습니다. 그러나 이러한 조건은 거의 나타나지 않아야하며, 표시 될 때는 어쨌든 중지해야합니다 ( 복잡 하기 때문에 ).

어쨌든, 여기에 보지 않은 제안이 두 개 더 있습니다. 희망이 누군가에게 도움이되기를 바랍니다 :)

두 줄로 나눌 수 있습니다

total = cond1 == 'val' and cond2 == 'val2' and cond3 == 'val3' and cond4 == val4
if total:
    do_something()

또는 한 번에 하나의 조건을 추가하십시오. 그렇게하면 적어도 그것은 혼란을에서 분리합니다 if.

이 스레드가 오래되었다는 것을 알고 있지만 Python 2.7 코드가 있으며 PyCharm (4.5)은 여전히이 경우에 대해 불평합니다.

if foo is not None:
    if (cond1 == 'val1' and cond2 == 'val2' and
        cond3 == 'val3' and cond4 == 'val4'):
            # some comment about do_something
            do_something

PEP8 경고 "다음 논리 행과 같은 들여 쓰기가있는 시각적 들여 쓰기 행"이라고해도 실제 코드는 완전히 정상입니까? "과도하게 들여 쓰기하지 않습니까?"

... 파이썬이 총알을 물고 중괄호와 함께 간 적이 있기를 바랍니다. 실수로 들여 쓰기로 인해 몇 년 동안 실수로 몇 개의 버그가 발생했는지 궁금합니다 ...

조건을 목록으로 묶은 다음 smth를 수행하십시오. 처럼:

if False not in Conditions:
    do_something

조건이 길면 코드 본문이 짧은 경우가 많습니다. 이 경우 본문을 두 번 들여 쓰기 만하면됩니다.

if (cond1 == 'val1' and cond2 == 'val2' and
    cond3 == 'val3' and cond4 == 'val4'):
        do_something

  if cond1 == 'val1' and \
     cond2 == 'val2' and \
     cond3 == 'val3' and \
     cond4 == 'val4':
      do_something

또는 이것이 더 명확한 경우 :

  if cond1 == 'val1'\
     and cond2 == 'val2'\
     and cond3 == 'val3'\
     and cond4 == 'val4':
      do_something

이 경우 들여 쓰기가 4의 배수 여야하는 이유는 없습니다. 예를 들어 "열기 분리 문자와 정렬"을 참조하십시오.

http://google-styleguide.googlecode.com/svn/trunk/pyguide.html?showone=Indentation#Indentation

다른 접근 방식이 있습니다.

cond_list = ['cond1 == "val1"','cond2=="val2"','cond3=="val3"','cond4=="val4"']
if all([eval(i) for i in cond_list]):
 do something

또한 목록에 다른 조건을 추가하여 if 문을 변경하지 않고도 다른 조건을 쉽게 추가 할 수 있습니다.

cond_list.append('cond5=="val5"')

나는 보통 다음을 사용합니다.

if ((cond1 == 'val1' and cond2 == 'val2' and
     cond3 == 'val3' and cond4 == 'val4')):
    do_something()

if & else 조건이 내부에서 여러 명령문을 실행 해야하는 경우 아래와 같이 작성할 수 있습니다. 우리가 그 안에 하나의 진술로 다른 예를 가지고 있다면 매번.

고맙습니다.

#!/usr/bin/python
import sys
numberOfArgument =len(sys.argv)
weblogic_username =''
weblogic_password = ''
weblogic_admin_server_host =''
weblogic_admin_server_port =''


if numberOfArgument == 5:
        weblogic_username = sys.argv[1]
        weblogic_password = sys.argv[2]
        weblogic_admin_server_host =sys.argv[3]
        weblogic_admin_server_port=sys.argv[4]
elif numberOfArgument <5:
        print " weblogic UserName, weblogic Password and weblogic host details are Mandatory like, defalutUser, passwordForDefaultUser, t3s://server.domainname:7001 ."
        weblogic_username = raw_input("Enter Weblogic user Name")
        weblogic_password = raw_input('Enter Weblogic user Password')
        weblogic_admin_server_host = raw_input('Enter Weblogic admin host ')
        weblogic_admin_server_port = raw_input('Enter Weblogic admin port')
#enfelif
#endIf

내 멍청한 놈을 용서하지만, 나는 여기에있는 사람만큼 #Python에 대해 잘 모르지만 3D BIM 모델링에서 내 자신의 객체를 스크립팅 할 때 비슷한 것을 발견했기 때문에 알고리즘을 파이썬의 것.

여기서 찾은 문제는 양면입니다.

1. 대본을 해독하려고 시도하는 사람에게는 내 가치가 외국인 것 같습니다.
2. 값을 변경하거나 (가장 가능성이 높거나) 새로운 조건을 추가해야하는 경우 코드 유지 관리 비용이 많이 듭니다.

이 모든 문제를 무시하고 스크립트는 다음과 같아야합니다

param_Val01 = Value 01   #give a meaningful name for param_Val(i) preferable an integer
param_Val02 = Value 02
param_Val03 = Value 03
param_Val04 = Value 04   # and ... etc

conditions = 0           # this is a value placeholder

########
Add script that if true will make:

conditions = conditions + param_Val01   #value of placeholder is updated
########

### repeat as needed


if conditions = param_Val01 + param_Val02 + param_Val03 + param_Val04:
    do something

이 방법의 장점 :

1. 스크립트를 읽을 수 있습니다.

2. 스크립트를 쉽게 관리 할 수 ​​있습니다.

3. 조건은 원하는 조건을 나타내는 값의 합계에 대한 1 비교 연산입니다.
4. 다단계 조건 불필요

그것이 당신 모두를 돕기를 바랍니다