整数を各桁の数に分ける

Question

引数に入ってきた整数を各桁に分け、各桁の位をかけ、最後には文字列として表示・
例えば、12=“10+2”,345="300+40+5”,70704="70000+700+4"のように表示したいです（0から始まるけたのかずは飛ばします。）。最後の出力は文字列として出力したいです。

自分が書いたコードはlen（）でlistの要素の数をしらべ、その(len(list)-1)**10のかずを一番大きい位の数にかけ、そのあとlistのindexをインクリメント、(len(list)-1)をデクリメントさせかけ合わせていくというのをループします。

がしかし、12のときは2となり42のときは２となり,なぜかいちのくらいしかでません。
なお、0からはじまるものはremoveとstartwith(0)をつかえばけせるかなとはおもっています

ご指摘お願いします。ご教授ください

def expanded_form(num):
    list=[int(a) for a in str(num)]
    print(list)
    keta=len(list)
    print(keta)
    index=0
    while keta>=1:
        list_2=[]
        numb=list[index]*(10**(keta-1))
        keta-=1
        index+=1
        numb=str(numb)
        list_2.append(numb)
    return("+".join(list_2))

Answer 1

次のコードの実行結果は分かりますか。

for i in range(3):
    lst = []
    lst.append(i)

print(lst)

出力は [2] です。
もし [0, 1, 2] だと思ったのなら、ループの挙動への理解が足りないです。

---

ループはfor文やwhile文で実現できますが、その実は**『ブロック内の繰り返し』**です。
上記のコードは次のように書いているのと同じように動作します。

i = 0
lst = []
lst.append(i)

i = 1
lst = []
lst.append(i)

i = 2
lst = []
lst.append(i)

print(lst)

このコードならば、実行結果が [2] になることに気付けるのでは無いでしょうか。

---

さて、質問のコードを振り返ってみます。

while keta>=1:
    list_2=[]
    ...
    list_2.append(numb)

return("+".join(list_2))

なぜ上手く動作しないのか、今なら分かりますね。

Comments

[aiueo04]

なるほど！！！list_2をなんかいも上書きしていたのですね！非常にありがたい限りです！！

Answer 2

ちゃんと見てないですけど、多分 while の中の list_2 の初期化が原因では？
ループのたびに上書きしています。

追記
デバッガ動かして、変数の推移を見るのでもいいですけど、以下のようなサイトを利用するのもおすすめです。

http://pythontutor.com

Comments

[aiueo04]

このサイトで段階を一つ一つ確認できたので今回のミスに気づけました！これからフル活用していきます。

Answer 3

4 つの方法を示します。

• 質問文のコードを修正したもの
• さらに変数を練らす等してみたもの
• str(num) でなく、 num の 商/余りの計算して処理するようにしたもの
• while をつかわないでかいてみたもの

pp.py

def expanded_form(num):
    digits = [int(a) for a in str(num)]
    keta = len(digits)
    index = 0
    list_2 = []
    while keta >= 1:
        numb = digits[index] * (10 ** (keta - 1))
        keta -= 1
        index += 1
        if numb > 0:
          list_2.append(str(numb))

    return(" + ".join(list_2))

def expanded_form_x(num):
    digits = [int(a) for a in str(num)]
    d_len = len(digits)
    terms = []
    for i in range(len(digits)):
        n = digits[i] * (10 ** (d_len - i - 1))
        if n > 0:
          terms.append(str(n))

    return(" + ".join(terms))

def expanded_form_y(num):
  terms = []
  zeros = ""
  while num > 0:
    num, d = num // 10, num % 10
    if d > 0:
      terms.insert(0, str(d) + zeros)
    zeros += "0"

  return " + ".join(terms)

def expanded_form_z(num):
  digits = [int(x) for x in str(num)]
  muls = reversed([10 ** i for i in range(len(digits))])
  return " + ".join([str(n * mul) for n, mul in zip(digits, muls) if n > 0])

for n in [0, 12, 345, 70809]:
  print(expanded_form(n))
  print(expanded_form_x(n))
  print(expanded_form_y(n))
  print(expanded_form_z(n))
  print()

実行例：

Answer 4

直接の回答ではないですが、こんな書き方ができるという例を示しておきます。
ひどい書き方なので真似しなくていいですが、いつかこのコードが読める日が来ることを期待して・・・

def expanded_form(num):
    return '+'.join([d + '0'*i for i, d in enumerate(str(num)[::-1]) if d != '0' or num == 0][::-1])

print(expanded_form(0))
print(expanded_form(12))
print(expanded_form(345))
print(expanded_form(70704))
print(expanded_form(90900))