ネストされたリスト内の文字列をチェックする

Question

グリッドの3つのスペースがすべて 'x'または 'o'としてマークされているかどうかをチェックするtic-tac-toeプログラムに関数を書き込もうとしています。 lst_1はラインのすべての可能な組み合わせのリストです：

def conc(): 
    global lst 
    lst_1 = [[lst[0],lst[1],lst[2]],[lst[3],lst[4],lst[5]],[lst[6],lst[7],lst[8]],[lst[0],lst[3],lst[6]],[lst[1],lst[4],lst[7]], [lst[2],lst[5],lst[8]],[lst[0],lst[4],lst[8]],[lst[2],lst[4],lst[6]], [lst[0],lst[1],lst[2]],[lst[3],lst[4],lst[5]],[lst[0],lst[3],lst[6]],  [lst[1],lst[4],lst[7]],[lst[2],lst[5],lst[8]],[lst[0],lst[4],lst[8]], [lst[2],lst[4],lst[6]]] 
    triplet = [s for s in lst_1] 
    if all('o' == item for item in triplet): 
     print('Player 2 wins!') 
     reply() 
    elif all('x' == item for item in triplet): 
     print('Player 1 wins!') 
     reply() 
    else: 
     pass 

私は誰かが行の3を得ることができたときに、このゲームを止めるだろうと思ったが、それは行くを続けています。何か案は？

Answer 1

あなたの質問に簡単に回答するには、lst_1を繰り返していません。私はあなたがtripletのために使用している声明であると思うと思うが、実際にはlst_1からtripletまでしかコピーされない。

実際にジェネレータをlst_1から外してtripletに割り当てようとしていますが、間違いなくリストの理解を誤って使用していて、ジェネレータの動作を完全に理解していません。発電機の詳細についてはhereをご覧ください。

ジェネレータは特定の状況では間違いありませんが、実際にはここでコードを単純化するとは思われません。とにかくループが必要になり、発電機がいつ終了するかを知るためにはStopIterationの例外をチェックする必要があります。

私の提案はループを使用することです。for

def conc(): 
    global lst 
    lst_1 = [[lst[0],lst[1],lst[2]],[lst[3],lst[4],lst[5]],[lst[6],lst[7],lst[8]],[lst[0],lst[3],lst[6]],[lst[1],lst[4],lst[7]], [lst[2],lst[5],lst[8]],[lst[0],lst[4],lst[8]],[lst[2],lst[4],lst[6]], [lst[0],lst[1],lst[2]],[lst[3],lst[4],lst[5]],[lst[0],lst[3],lst[6]],  [lst[1],lst[4],lst[7]],[lst[2],lst[5],lst[8]],[lst[0],lst[4],lst[8]], [lst[2],lst[4],lst[6]]] 
    for triplet in lst_1: 
     if all('o' == item for item in triplet): 
      print('Player 2 wins!') 
      reply() 
      break 
     elif all('x' == item for item in triplet): 
      print('Player 1 wins!') 
      reply() 
      break 

いくつかの他のシラミ： はあなたif-elif-elseループの最後にelse: passを追加すると、完全に無関係のです。そこにいる必要はありません。

また、グローバル変数を避けてください。 Pythonのスコープについて少しお読みください。シンプルなプログラムではうまくいくかもしれませんが、リモートで複雑なものに遭遇したときにそれらに頼ると、あなたを傷つけるでしょう。