ルーピングオーバー* argsカウンタをリセット

Question

*argsの機能を理解しようとするコードを作成しましたが、counter変数をリセットしないと問題が発生しています。

4番目の項目の後に、ナンバリングは0にリセットされますが、私はそれをargsを越えて継続します。

lst = ['apple', 'banana', 'orange', 'lemon'] 
lst2 = ['apple2', 'banana2', 'orange2', 'lemon2'] 
lst3 = ['apple3', 'banana3', 'orange3', 'lemon3'] 


def generateMenu(*args): 
    counter = 0 

    for i in args: 
     def recurse(l, counter): 
      for i in l: 
       counter += 1 
       if isinstance(i, (list, tuple)): 
        recurse(i, counter) 
       else: 
        print("{}. {}.".format(counter, i)) 
     recurse(i, counter) 


generateMenu(lst, lst2, lst3) 

出力：

1. apple. 
2. banana. 
3. orange. 
4. lemon. 
1. apple2. 
2. banana2. 
3. orange2. 
4. lemon2. 
1. apple3. 
2. banana3. 
3. orange3. 
4. lemon3. 

Answer 1

あなたは、代わり

lst = ['apple', 'banana', 'orange', 'lemon'] 
lst2 = ['apple2', 'banana2', 'orange2', 'lemon2'] 
lst3 = ['apple3', 'banana3', 'orange3', 'lemon3'] 


def generateMenu(*args): 
    counter = 0 

    for i in args: 
     def recurse(l): 
      nonlocal counter 
      for i in l: 
       counter += 1 
       if isinstance(i, (list, tuple)): 
        recurse(i) 
       else: 
        print("{}. {}.".format(counter, i)) 
     recurse(i) 


generateMenu(lst, lst2, lst3) 

（ローカル以外はのpython3で導入されました）counternonlocalを作るPythonは関数の外の変数への書き込みについて少し奇妙なルールがありrecurse

lst = ['apple', 'banana', 'orange', 'lemon'] 
lst2 = ['apple2', 'banana2', 'orange2', 'lemon2'] 
lst3 = ['apple3', 'banana3', 'orange3', 'lemon3'] 

def generateMenu(*args): 
    counter = 0 

    for i in args: 
     def recurse(l, counter): 
      for i in l: 
       counter += 1 
       if isinstance(i, (list, tuple)): 
        counter = recurse(i, counter) 
       else: 
        print("{}. {}.".format(counter, i)) 
      return counter 
     counter = recurse(i, counter) 


generateMenu(lst, lst2, lst3) 

Answer 2

問題は、あなたのrecurse機能は、上位のスコープからのごcounterを認識していないということです。だから、それぞれのfor i in l:ループで増加しているのは、単に "内側" counterの変数です。

同じ変数名を2回（counterおよびi）使用しているため、この例では混乱する可能性があります。別の方法で名前を変更しようとすると、コードで何が起こっているのかをよりよく知ることができます。

Answer 3

からcounterを返す必要があります範囲。このIMOを避ける最も簡単な方法は、再帰関数が各状態でカウンタ値を返すことを確認することです。私は若干リファクタリングして、何が起こっているのかを少しはっきりさせました。

lst = [('apple', 'banana'), 'orange', 'lemon'] 
lst2 = ['apple2', 'banana2', 'orange2', 'lemon2'] 
lst3 = ['apple3', 'banana3', 'orange3', 'lemon3'] 


def generateMenu(*args): 
    outer_counter = 1 

    def recurse(l, inner_counter): 
     for i in l: 
      if isinstance(i, (list, tuple)): 
       inner_counter = recurse(i, inner_counter) 
      else: 
       print("{}. {}.".format(inner_counter, i)) 
       inner_counter += 1 
     return inner_counter 

    for i in args: 
     outer_counter = recurse(i, outer_counter) 


generateMenu(lst, lst2, lst3)