filter()が機能しません

Question

ある関数に入ってきた文字列の中で大文字だけを取り出して表示させたいんですけどfilter()の部分で大文字だけを拾ってくれないのはなぜですか？
list_2では文字列全部がはいってしまいます。

def abbrevName(name):
    list=[a for a in name]
    list_2=[b for b in filter(lambda n: n.isupper, list)]
    print(list_2)
    return "{}.{}".format(list_2[0],list_2[1])
     
abbrevName("Sam Smith")

Answer 1

引数を一つ取る関数を指定すればいいので、この場合ならこう書くこともできますよ。

def abbrevName(name):
    uppers = [c for c in filter(str.isupper, name)]
    print(uppers)
    return f"{uppers[0]}.{uppers[1]}"

print(abbrevName("Sam Smith"))

Comments

[aiueo04]

なるほど！！そのような方法でもかけるのですね。勉強になります

Answer 2

str.isupper() は Attribute でなく、関数なので、() をつけて関数として呼び出さないといけません。

- n.isupper
+ n.isupper()

修正後のコード

def abbrevName(name):
    list=[a for a in name]
    list_2=[b for b in filter(lambda n: n.isupper(), list)]
    print(list_2)
    return "{}.{}".format(list_2[0],list_2[1])

abbrevName("Sam Smith")

Comments

[aiueo04]

list_2=[b for b in filter(isupper(), list)]
こんな感じでしょうか？

[tiitoi]

それで正しく動くはずです

[aiueo04]

NameError: name 'isupper' is not definedがでてしまうってことは僕の表記の仕方がまちがってるってことですよね

[aiueo04]

あれ、とするとなんだろうｗ

[tiitoi]

すいません。isupper() ではなく、n.isupper() でした。
修正後のコードを追記しました。

[aiueo04]

書いてもらったこーどは正しく起動しましたが、なぜかぼくのは動きません

[aiueo04]

あ、動きました！！ありがとうございました！