リスト内の要素の出現回数を数えるプログラムの作成

Question

前提・実現したいこと

Pythonの質問です。
リスト内の要素の出現回数をそれぞれ数える関数city_countを作成して、main関数内にて結果を表示するプログラムを作成したいのですが、TypeErrorが発生してうまくいきません。
初歩的な質問で本当に申し訳ないですが、どなたか教えていただきたいです。

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

TypeError: city_count() missing 2 required positional arguments: 'osa' and 'ngo'

該当のソースコード

def city_count(tyo, osa, ngo):
    tyo = 0
    osa = 0
    ngo = 0
    count = 0

    for place in city:
        if place[count] == "東京":
            tyo += 1
        elif place[count] == "大阪":
            osa += 1
        else:
            ngo += 1
    count += 1
    return tyo, osa, ngo

city = ["東京", "大阪", "東京", "東京", "大阪", "名古屋"]

tokyo, osaka, nagoya = city_count(city)
print("東京:{}, 大阪:{}, 名古屋:{}".format(tyo, osa, ngo))

Answer 1

pandasを使うと、便利なSeries.value_counts()メソッドを使うことができますよ。

>>> import pandas as pd
>>> city = ["東京", "大阪", "東京", "東京", "大阪", "名古屋"]
>>> lst = pd.Series(city)
>>> lst.value_counts()
東京     3
大阪     2
名古屋    1
dtype: int64
>>>

Comments

[退会済みユーザー]

そんな便利なメソッドがあるんですね・・・。コードが圧倒的に短くなって凄いです！
まだまだ勉強不足でした、ありがとうございました！

Answer 2

いくつか修正いれました。

def city_count(city): # 引数のところに返り値が入っていたのでエラーになっていました
    tyo = 0
    osa = 0
    ngo = 0
    # count = 0 不要

    for place in city:
        if place == "東京": # 元のコードだと、"東京"などのplaceに対しplace[0]と"東"を指定していました。
            tyo += 1
        elif place == "大阪":
            osa += 1
        else:
            ngo += 1
    # count += 1 不要
    return tyo, osa, ngo

city = ["東京", "大阪", "東京", "東京", "大阪", "名古屋"]

tokyo, osaka, nagoya = city_count(city)
print("東京:{}, 大阪:{}, 名古屋:{}".format(tokyo, osaka, nagoya)) # 変数名修正

Comments

[退会済みユーザー]

エラーの出る理由がわかり、とても参考になりました。
まだまだ勉強します。ありがとうございました！

Answer 3

city = ["東京", "大阪", "東京", "東京", "大阪", "名古屋"]

tyo, osa, ngo = city.count("東京"), city.count("大阪"), city.count("名古屋")
print("東京:{}, 大阪:{}, 名古屋:{}".format(tyo, osa, ngo))
# 東京:3, 大阪:2, 名古屋:1
```[リスト].count(要素)で何個あるかが分かります。
要素が多い時は、いっぺんに取得しましょう。以下別解↓
```Python
import collections

city = ["東京", "大阪", "東京", "東京", "大阪", "名古屋"]
dct = collections.Counter(city)
for k, v in dct.items():
    print("{}:{}".format(k, v))
# 東京:3
# 大阪:2
# 名古屋:1

Comments

[退会済みユーザー]

for文を使わなくてもそのように短くかけるのですね・・・！
とても勉強になりました。覚えておきます！ありがとうございました！