forループのpythonで複数の項目をチェックする

Question

私は、角括弧のバランスが取れていないことをユーザーに伝えるコードを書いています。

私のコードがどこに間違っているのか正確に伝えることができます。

ブラケットの最初の状況になると、間違ったものや他の正しいものを探し続けることはありません（私は思っています）。

私は（今のすなわち何派手なショートカット）シンプルですが、長い、それを維持したいん

あなたはreturnとすぐに何かがOKであるとして、あなたは見つけることができません場合はここに私のコード

def isbalanced(text): 
    openingbracket=[] 
    for i, next in enumerate (text): 
     if next=='(' or next=='{' or next=='[': 
      openingbracket.append(next) 
     if next==')' or next=='}' or next==']': 
      if len(openingbracket)==0: 
       print("ops only opening brackets") 
       return False 
      else: 
       a=openingbracket.pop() 
       if a =='(' and next==')': 
        print("its matched parenthesis") 
        return True 
       if a =='{' and next=='}': 
        print("its matched curly brackets") 
        return True 
       if a =='[' and next==']': 
        print("its matched square") 
        return True 
       else: 
        print("wrong closing brackets") 
        return False 
    if len(openingbracket): 
     print ("no closing brackets") 
     return False 
    else: 
     print("no brackets") 
     return True 
isbalanced("Hello()(]") 

Answer 1

を行い、インデックスを使用していないc

if next=='(' or next=='{' or next=='[':がif c in "({[":

for i, next in enumerate (text):によって置き換えることができます使用します残りの文字列をチェックする前に、 Trueを返すことができることが分かっている唯一の時間は、文字列全体を処理した後です。したがって、return Trueはforループが終了した後でなければなりません。

def isbalanced(text): 
    openingbracket=[] 
    for i, next in enumerate (text): 
     if next=='(' or next=='{' or next=='[': 
      openingbracket.append(next) 
     if next==')' or next=='}' or next==']': 
      if len(openingbracket)==0: 
       print("ops only opening brackets") 
       return False 
      else: 
       a=openingbracket.pop() 
       if a =='(' and next==')': 
        print("its matched parenthesis") 
        return True      # REMOVE THIS LINE 
       if a =='{' and next=='}': 
        print("its matched curly brackets") 
        return True      # REMOVE THIS LINE 
       if a =='[' and next==']': 
        print("its matched square") 
        return True      # REMOVE THIS LINE 
       else: 
        print("wrong closing brackets") 
        return False 
    if len(openingbracket): 
     print ("no closing brackets") 
     return False 
    else: 
     print("no brackets") 
     return True 
isbalanced("Hello()(]") 

Answer 2

です文字列内の更なるエラーが...それはまさにあなたが3ヶ所でやっている：

if a =='(' and next==')': 
    print("its matched parenthesis") 
    return True # let it continue to check the string. 

は、あなたの方法の終わりにreturn Trueを追加します：ループが経由する場合は、文字列は別に

（実際に、それはあなたの現在のコードですでにOKです）OKです：ビルトインだとのiterableから値を取得するために

• は、あなたの変数にnextを使用しないでください。私はあなたがそれらが引き起こすので、全体のメソッドがTrueを返すために、単にあなたのreturn True文を削除だけfor c in text: