パイソン 各要素の出現回数を辞書として返す

Question

半日考えましたが分かりません。

numbers=[]
count=0
for c in list:
    if c==c:
     count=count+1
     numbers.append(count)

なぜこれでうまくいかないのでしょうか？
よろしくお願いします。

コード
def count(things):
    counts = {}
    for x in things:
        if x not in counts:
            counts[x] = 0
        counts[x] = counts[x] + 1
    return counts

答えです

Comments

[y_waiwai]

このままではコードが読めないので、質問を編集し、<code>ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください

Answer 1

半日考えましたが分かりません。

まず、半日、Python の入門書を読みましょう。

「引数が 1つで、辞書を返す関数 count を作れ」とあったら、
つぎのようなコードを書くはずです。

def count(a):   # 引数1つの関数 count を定義する
	d = {}      # 空の辞書を用意する
	# リスト a の中身を調べて辞書 d に追加する
	return d    # 辞書を返す

質問のコードには、関数定義も辞書も return もありません。
count=0 だと count は関数ではなく変数です。
Python の基礎を勉強してください。

追記
私が回答を書いた後に質問に追記がありましたね。
穴埋め問題なのに、どうして最初に無茶苦茶なコードを書いたのですか？

Python の基礎

a = [31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31]
a はリストです。
a[0] は 31 です。a[1] は 28 です。a[11] は 31 です。
リスト a に 75 を追加するのは、a.append(75) です。

d = {31: 7, 28: 1, 30: 4}
d は辞書です。
d[31] は 7 です。d[30] は 4 です。
辞書 d に 75:1 を追加するのは、d[75] = 1 です。
d[28] = 5 だと既に存在する 28:1 が 28:5 に変更されます。

Comments

[aki1111]

関数の定義とかが抜けていたことに気付いていなかったのと、基本的な部分を書かなくても問題の部分を書けば伝わるだろうと思ったのです。
今回の課題にてこずってる人が多いのを見かねた先生がヒントをくれたので、後から追加したのみです。

[aki1111]

分かりやすい説明をどうもありがとうございます。授業で習ってもすぐに忘れてしまうので、ちゃんと覚えていかないといけないですね

[kazuma-s]

伝わりません。こんなコードを見せられたら、
このコードを書いた人は Python の基礎が全く理解できていないと判断されます。

Answer 2

def count(things):
counts = {}
for x in things:
if x not in counts:
counts[x] = 0
counts[x] = counts[x] + 1
return counts

Comments

[kazuma-s]

Python はインデントが重要です。
コードを ```Python と ``` で挟んでください。

質問は、「なぜこれでうまくいかないのでしょうか？」でした。
最初のコードがなぜうまくいかないのかを説明しないと解決になりません。

[aki1111]

そうでしたか

Answer 3

counts[]の意味が分かりません。

辞書の勉強しましたか？？
辞書の要素の追加方法、更新方法が分かれば分かる筈です（これが分からないと辞書の使いようがないので、勉強したけど分からないなら勉強してないのと同じです）。

Answer 4

回答ではありませんが、

この問題は引数で与えられるリストにかなり複雑な条件を付けないと正解がありません。

仮にcount_listという関数でそれが実現できたと思ったとしましょう。
その時には以下を試してみてください。

>>> print(count_list([1, True, 1.0]))
{1: 3}

のような結果が返ってくることでしょう。

また、以下も試してみてください。

>>> print(count_list([[1,2], [3,4], [5,6]]))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<stdin>", line 4, in count_list
TypeError: unhashable type: 'list'

のような結果が返ってくるでしょう。

Answer 5

• 以下の 「c == c」はどのような状態を検出しようとしていますか？ C と C は常に同じです。

for c in list:
   if c==c:

• numbersはリストですが、出現回数は辞書として返すのでは？

以下のような感じで作ればいいでしょう。

• 返す辞書は以下のようなものとする。

入力が「apple」であれば、 {"a":1, "e":1, "l":1, "p":2} になる。

• 入力を1文字ずつ取り出し
  辞書になければ追加して、値を1とする。
  辞書にあれば、値に1を足す
• 入力が無くなれば、辞書を返す

Comments

[aki1111]

for文は繰り返しなので、取り出した数とループした他の数と一致するかどうかを調べようとしてました。
直接辞書型にする方法が分からないので、リストから辞書型にもうひと手間加えるつもりでした。

辞書になければ追加する方法がよくわかりません。最初の辞書は空のはずですが、｛｝で定義するのですか？

[TakaiY]

はい。 count_dict = {} のように定義します。

辞書にそのキーがあるかどうかは、in を使います。
if c in count_dict.keys()
とすると、あるかどうかわかります。

ちなみに、リストにはたぶん個数しか入らないので、後で辞書にしようとしてもその個数がどの文字なのかわかりませんね。