繰り返し処理をしてるつもりなのに結果が１つしか返ってきません

Question

前提・実現したいこと

Python3において単語の長さが偶数ならA、奇数ならBを返すプログラムを作りたいです。

names =["car","book","chair"]を使って
B、A、Bと返ってくるようにしたいです。

発生している問題・エラーメッセージ

関数を定義し、繰り返し処理を行ったつもりですが結果が"car"に対応していると思われるBの１つしか返ってこずどうしたらいいか困っております。

該当のソースコード

def judge(names):
    for name in names:
        if len(name) %2==0:
            return "A"
        else:
            return "B"

names =["car","book","chair"]

result=greetings(names)
print(result)

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[hentaiman]

forの中でprintデバッグしてみましたか？

[miyabi_pudding]

Pythonはインデントがないと動かないので、質問にコードブロック使わないとダメですよ。

[ybteru_0218]

ありがとうございます。
コードの貼り付け方が分かっていなかったのでコードの挿入からコードを挿入しました。

Answer 1

まず judge が返してくれるオブジェクトの型を考えましょう。return "A" ないし return "B" といっていることから string 型を返してくれるんですよね？でもほしいのは ['B', 'A', 'A'] という string の入った list です。そもそも期待するものに対して返すものが違うわけです。

また、繰り返しの中身にも問題があります。

def judge(names):
    for name in names:
        if len(name) %2==0:
            return "A"
        else:
            return "B"

return というのは即座に関数から抜け出て値を返す構文です。繰り返しの中でそれを使っても同じで、最初の繰り返しで return してしまいます。なのですぐ return せず結果を変数に溜めてそれを後で return する、というのがよいでしょう。ヒントはあげたので後は自分で考えてみてください。わからなければどこがどうわからないのか自分なりに考えてまた聞きに来てください。

---

わかる人向けの良くないやーつ

def judge(names):
    for name in names:
        if len(name) %2==0:
            yield "A"
        else:
            yield "B"

Comments

[ybteru_0218]

すごく詳しくありがとうございます。
リストにして返すこと、なんとかできました。
yieldはよくないやつとのことですが使ったことがないので勉強します。
ありがとうございました！