python ランキング　書き込み

以前作っていただいたコードを書き直してランク付けした上で他のtxtファイルに移す作業のコードを教えていただきたいです。

https://teratail.com/questions/96246
前回質問し、教えていただいた内容です。コードも教えていただきました。

python
1with open('file1.txt') as f:
2    value_list = [line.rstrip() for line in f]
3
4with open('file2.txt') as f:
5    key_list = [line.rstrip() for line in f]
6data_dict = {}
7for raw_value in value_list:
8    key, value = raw_value.split(';')
9    if key in data_dict:
10        data_dict[key].append(value)
11    else:
12        data_dict[key] = [value]
13
14for key, values in sorted(data_dict.items(), key=lambda x: -len(x[1])):
15    for value in values:
16        print(key + ';' + value)
17
18    if len(values) == 0:
19        pass
20    else:
21        print(len(values))

以上です。表記の仕方がわかりにくかったらおっしゃってください。
今、ファイル１にはソートされた下記のようなデータが１００万近くあります。
1234567;7890984
1234567;8764589
1234567;9275785
1345678;9868465
2345678;8979894
2345678;9078990
またファイル2にはソートされた下記のようなデータが30万近くあります。
1234567
1345678
2347889

結果としては、
ファイル１の；より左の数値とファイル２が合致した時のみを考え、合致した場合はファイル１にその数字を含む行がいくつあるのかを数え上位からランク付けします。
例としては、以下のようです。
1234567;7890984
1234567;8764589
1234567;9275785
3
1345678;9868465
1

このように解決、編集していただきました。そのコードに手を加えさせていただき、新しくコードを書きました。（このコードの結果を別のファイルへ保存する方法しようと思い書きました。）
新しく、ファイルを作りそこに上記の結果を保存したいです。
下記のようなコードを書きましたが、最後のf3.write(str1)のところで通りません。（f3.writeは2回しようしてはいけないなどの決まりがありますか？）初心者なので、ヘンテコな編集になっているかもしれません。ご指摘よろしくお願いします。

python
1with open('ファイル１.txt') as f:
2    value_list = [line.rstrip() for line in f]
3
4with open('ファイル２.txt') as f:
5    key_list = [line.rstrip() for line in f]
6
7f3 = open('ファイル３.txt', 'w')
8
9data_dict = {}
10for raw_value in value_list:
11    key, value = raw_value.split(';')
12    if key in data_dict:
13        data_dict[key].append(value)
14    else:
15        data_dict[key] = [value]
16
17for key, values in sorted(data_dict.items(), key=lambda x: -len(x[1])):
18    for value in values:
19        print(key + ';' + value)
20        f3.write(key + ';' + value + '\n')
21
22    if len(values) == 0:
23        pass
24    else:
25        print(len(values))
26        str1 = len(values)
27        f3.write(str1)
28f.close()
29

行動規範の内容に同意します

回答1件

Python
1# 実際にはファイルから読み込む
2value_list = ['1234567;7890984', '1234567;8764589', '1234567;9275785', '1345678;9868465', '2345678;8979894', '2345678;9078990']
3key_list = ['1234567', '1345678', '2347889']
4
5data_dict = {}
6for raw_value in value_list:
7    key, value = raw_value.split(';')
8    if key in data_dict:
9        data_dict[key].append(value)
10    else:
11        data_dict[key] = [value]
12
13for key, values in sorted(data_dict.items(), key=lambda x: -len(x[1])):
14    # for value in values:
15    #     print(key + ';' + value)
16    
17    if len(values) == 0:
18        pass
19    else:
20        print(key + ';' + values[0])
21        print(len(values))
22
23"""実行結果(修正前)
241234567;7890984
251234567;8764589
261234567;9275785
273
282345678;8979894
292345678;9078990
302
311345678;9868465
321
33"""
34
35"""実行結果(修正後)
361234567;7890984
373
382345678;8979894
392
401345678;9868465
411
42"""

前回書いたのは書き捨てのプログラムだったため、変更が必要でした。
どんどん仕様が増えていくと、コーディングの見通しが立ちません。
出来るだけ序盤に『何が出来る必要があるのか』を明確にするようにしてください。

下記のようなデータが１００万近くあります。

このスクリプトがエラーなく動けばいいですが、ちょっと自信がないです。
大きなデータに対応させるためには、工夫が必要かもしれません。

その要素の最初の一つとそれについての要素数を書き出したいです。

修正しました。

また、別に著作権云々大騒ぎする気はありませんが...
質問で他人が書いたプログラムを引用する場合、それを明示した方がよいです。
前回質問へのリンクを貼り、更に引用符を付けるとよいでしょう。

**リンクの貼り方**
[リンク先の呼称](URL)

**引用符の付け方**
> 引用したい文
> 引用したい文

リンクを貼ると、このようになります。

引用符をつけると、このようになります。
Python
1コードも引用できます。

追記を受けて

最後のf3.write(str1)のところで通りません。
TypeError: expected a character buffer object

エラーが再現出来ていないのですが、おそらく数値を渡しているのが原因です。
文字列にキャストしてあげてください。

Python
1str1 = len(values)
2f3.write(str(str1))

ついでに、最後closeするものを間違えています。

Python
1f.close()

明示的に閉じる必要があるのはfではなくf3では。

さらに、コメントを受けて

1回目は (中略) TypeError:cannot concatenate 'str' and 'int' objects

エラーメッセージこそ異なりますが、こういうことです。

Python
1>>> 'hoge' + 1
2Traceback (most recent call last):
3  File "<stdin>", line 1, in <module>
4TypeError: must be str, not int
5>>> 'hoge' + str(1)
6'hoge1'

あとは、質問者様ご自身がどのような工夫をしたのか見えるといいですね。
今の状態では、ただ投げ出しているのか、壁にぶつかっているのかわからないです。
人の書いたコードはよく読んで内容を把握し、そしてアレンジする技術を持ちましょう。

投稿2017/10/13 16:21

編集2017/10/16 07:35

LouiS0616

総合スコア35660

質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう！

追記を受けて

さらに、コメントを受けて

関連した質問