python 可変長引数について ※jupyter lab使用

Question

jupyter lab で勉強しながら遊んでたら、プログラムが思ってたのと違う挙動をし始めました

下の画像がその時書いたプログラムと実行結果です。

def init(data):
    data['first'] = {}
    data['middle'] = {}
    data['last'] = {}
    
def lookup(data, label, name):
    return data[label].get(name)

def store(data, *full_names):
    for full_name in full_names:
        names = full_name.split()
    if len(names) == 2: 
        names.insert(1, '')
    labels = 'first', 'middle', 'last'
    for label, name in zip(labels, names):
        people = lookup(data, label, name)
        if people:
            people.append(full_name)
        else:
            data[label][name] = [full_name]

            

d = {}
init(d)
store(d, 'Han Solo', 'Aha-n Solo')
store(d, 'Luke Skywalker', 'Anakin Skywalker', 'Anahaimu Skywalker', 'Aha~n Skywalker')
print(d)
print(lookup(d, 'last', 'Skywalker'))

実行結果

{'first': {'Aha-n': ['Aha-n Solo'], 'Aha~n': ['Aha~n Skywalker']}, 'middle': {'': ['Aha-n Solo', 'Aha~n Skywalker']}, 'last': {'Solo': ['Aha-n Solo'], 'Skywalker': ['Aha~n Skywalker']}}
['Aha~n Skywalker']

修正後のコード

def init(data):
    data['first'] = {}
    data['middle'] = {}
    data['last'] = {}
    
def lookup(data, label, name):
    return data[label].get(name)

def store(data, *full_names):
    for full_name in full_names:
        names = full_name.split()
    　　if len(names) == 2: 
        names.insert(1, '')
    labels = 'first', 'middle', 'last'
    for label, name in zip(labels, names):
        people = lookup(data, label, name)
        if people:
            people.append(full_name)
        else:
            data[label][name] = [full_name]

store関数を用いて複数のデータを入れられるようにしたつもりです。

store(d, 'Han Solo') のように一つのデータであれば正しく挙動してくれるのですが

store(d, 'Han Solo', 'Ahan Solo')のように複数のデータを入れようとすると'Ahan Solo'しか格納してくれません

なぜ'Han Solo', 'Aha~n Solo' 両方とも格納してくれないのか理解できません。

英雄の名前で遊んだのが原因でしょうか？

Comments

[y_waiwai]

コードはテキストで提示しましょう

[morigami]

す、すいませ

[morigami]

これでどうでしょうか？

[y_waiwai]

お手数ですが、、

ちとコードが読めないので、質問を編集し、<code>ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください
これするとインデントもきっちり出てくれます

[morigami]

醜態をさらしてしまい申し訳ありません。
これでどうでしょうか！！

[y_waiwai]

バッチシです

[t_obara]

もうクラスで良いんじゃって思った。
ちなみに、デバッガは使わないのですか？storeの引数がどのような内容になっていて、それを各ステップで処理していくとどのようになっていくのかデバッグしてみればすぐわかるような気がしますが。

Answer 1

英雄の名前で遊んだのが原因でしょうか？

そう思います。だから、以下のような間違ったコードを書いてしまうのです。

    for full_name in full_names:
        names = full_name.split()
    if len(names) == 2: names.insert(1, '')

pythonはインデントに意味があるということを忘れたのは、ハンソロの祟りでしょう。

Comments

[morigami]

あっ、すっかり忘れていました。次から次から英雄の名前で遊ぶのは止めます。
ありがとうございます！！

[morigami]

直して「これで完璧や！！」と思って走らせたら同じ結果になりました。
codeの方はすでに修正済のものに変更しているので、他に別の問題があるのでしょうか？
実はハンソロの祟りではない可能性があるのでは？
※新しく実行結果の方も入れてみました

[ppaul]

今の質問にあるのが、修正版のソースですか？
だとしたら、私の言った意味が理解できなかったのかな？
それとも、修正したソースをまだ載せていませんか。皆さんにわかるように、古いソースとは別に修正版ソースを載せてください。

[morigami]

これでどうでしょうかああああああああああ！！！

[morigami]

ふざけてすいません。直してみました

[ppaul]

やっぱ、わかっていなかったですね。
for full_name in full_names: で回すのは、どこからどこまでですか？

[morigami]

names.insert(1, '')
のとこまでですか？

[ppaul]

祟りは強力ですね。
もしも、storeの引数の名前が一つだけしか受け取らない仕様だったら、その名前について関数storeは最後まで処理をするのでしょう。
でも名前を複数書けるようにしたからfor full_name in full_names:で回すことにした、
では、for full_name in full_names: で回すのは、どこからどこまでですか？　と考えてみてください。

[morigami]

分かりました！！一番最後の
if people:
people.append(full_name)
else:
data[label][name] = [full_name]
までです！！

[ppaul]

おめでとう。呪いが解けましたね。

[morigami]

暗黒面に堕ちなくてよかったです。ありがとうございます

[morigami]

できました！！ありがとうございます