クラスをカプセル化したい!
解決済
回答 2
投稿
- 評価
- クリップ 0
- VIEW 422
わからないこと
エラーがなぜ出るのか、どう直せばエラー無くなるのかがわかりません。エラーメッセージ
に書いてある、ラインを確認しても何が間違いなのかがわからない状態です。初歩的なミスの可能性もありますが、よろしくお願いします。
補足
以前に、テキストでのミスがあり辞書データとして読み書きできなかった問題がありましたが解決しており、今回はテキストでのミスはないと思われます。> https://teratail.com/questions/208019
コード
このプログラムは、質問をしてテキストから答えを持ってくることで、回答するプログラムです。参考書から引用しているため、大幅なコードの間違いは無いと思われます。
class Dictionary:
def __init__(self):
""" 辞書オブジェクトの作成
"""
self.__load_history()
def __load_history(self):
"""ファイルを読み込み、世界史の辞書オブジェクトを作成
"""
with open('data/world_history.txt', 'r', encoding='utf_8'
) as file:
lines = file.readlines()
new_lines = []
for line in lines:
line = line.rstrip('\n')
if line != '':
new_lines.append(line)
separate = []
for line in new_lines:
sp = line.split('\t')
separate.append(sp)
self.__history = dict(separate)
def save(self):
"""self.historyの内容を加工してファイルに書き込む
"""
write_lines = []
for key, val in self.history.items():
write_lines.append(key + '\t' + val + '\n')
with open('data/world_history.txt', 'w', encoding='utf_8') as f:
f.writelines(write_lines)
def get_history(self):
return self.__history
def set_history(self, history):
self.__history = history
history = property(get_history, set_history)
if __name__ == '__main__':
dictionary = Dictionary()
print(dictionary.history)
dictionary.save()
world_history.txt
古代ギリシアの植民市 ビサンティウム、ネアポリス、マッサリア
古代ギリシアの三大悲劇詩人 アイスキュロス、ソフォクレス、エウリピデス
シェイクスピアの四大悲劇 オセロ、マクベス、リア王、ハムレット
ローマ帝王の五賢帝 ネルヴァ、トラヤヌス、ハドリアヌス、アントニヌス・ピウス、マルクス・アウレリウス・アントニヌス
カースト制度の身分 バラモン、クシャトリヤ、ヴァイシャ、シュードラ
ロシアの歴代大統領 エリッツン、プーチン、メドヴェージェフ
世界四大文明 エジプト文明、メソポタミア文明、インダス文明、黄河文明
三国志の三国 魏、呉、蜀
ルネサンス期の三大発明 火薬、羅針盤、活版印刷
三国時代の三国 高句麗、百済、新羅
エラーメッセージ
Traceback (most recent call last):
File "C:/Users/beret/PycharmProjects/SampleProject/histly_dictionaly.py", line 48, in <module>
dictionary = Dictionary()
File "C:/Users/beret/PycharmProjects/SampleProject/histly_dictionaly.py", line 6, in __init__
self.__load_history()
File "C:/Users/beret/PycharmProjects/SampleProject/histly_dictionaly.py", line 25, in __load_history
self.__history = dict(separate)
ValueError: dictionary update sequence element #0 has length 1; 2 is required
-
気になる質問をクリップする
クリップした質問は、後からいつでもマイページで確認できます。
またクリップした質問に回答があった際、通知やメールを受け取ることができます。
クリップを取り消します
-
良い質問の評価を上げる
以下のような質問は評価を上げましょう
- 質問内容が明確
- 自分も答えを知りたい
- 質問者以外のユーザにも役立つ
評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。
質問の評価を上げたことを取り消します
-
評価を下げられる数の上限に達しました
評価を下げることができません
- 1日5回まで評価を下げられます
- 1日に1ユーザに対して2回まで評価を下げられます
質問の評価を下げる
teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。
- プログラミングに関係のない質問
- やってほしいことだけを記載した丸投げの質問
- 問題・課題が含まれていない質問
- 意図的に内容が抹消された質問
- 過去に投稿した質問と同じ内容の質問
- 広告と受け取られるような投稿
評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。
質問の評価を下げたことを取り消します
この機能は開放されていません
評価を下げる条件を満たしてません
質問の評価を下げる機能の利用条件
この機能を利用するためには、以下の事項を行う必要があります。
- 質問回答など一定の行動
-
メールアドレスの認証
メールアドレスの認証
-
質問評価に関するヘルプページの閲覧
質問評価に関するヘルプページの閲覧
checkベストアンサー
+2
world_history.txtがタブ区切りになってないせいでsplitできてないとか?
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
0
google翻訳
ValueError: 辞書更新シーケンス要素#0の長さは1です。 2が必要です
ということです。2必要なのに1しかないといってますぜ
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
15分調べてもわからないことは、teratailで質問しよう!
- ただいまの回答率 88.19%
- 質問をまとめることで、思考を整理して素早く解決
- テンプレート機能で、簡単に質問をまとめられる
2019/09/01 22:12
2019/09/01 22:14 編集
しかし、これをタブに変えるとうまく動きました。
エディタの設定によってはタブをスペースに変換するものがありますから、その設定を切って保存してください。
2019/09/01 22:18
2019/09/01 22:21
world_history.txtをそのまま
→現象再現する。
world_history.txtのスペースをタブに変換(SAKURAエディタ使用)
→正常動作する。
でした。
2019/09/01 22:21
2019/09/01 22:22