Python謎のエラー

Question

みんなのPython第3版のアプリケーションを作るというチャプターに乗っていたサンプルコードなのですが、エラーが出てしまいます。
今日ずっと調べていたのですがわからず、進捗が溶けていきます。
何度も確認しているので打ち間違えはないと思います。
教えていただけると助かります。

todoitem.py

#A class to save ToDo's Item.
class ToDoItem(object):

	#Initialization Method
	def __init__(self, title, description, duedate, addeddate = None):

		if not addeddate:
			addeddate = datetime.now()

		self.title = title
		self.description = description
		self.duedate = duedate
		self.addeddate = addeddate
		self.finished = False
		self.finisheddate = None


	#Finish ToDo
	def finish(self, date = None):
	
		self.finished = True

		if not date:
			date = datetime.now()
		
		self.finisheddate = date

		#Debug
		def __repr__(self):

			return "<ToDoItem {}, {}>".format(self.title, self.duedate.strftime('%Y/%m/%d %H:%M'))

todocontainer.py

#import module
from datetime import datetime, timedelta

#A Class To Save List of ToDo
class ToDoContainer(object):
	
	#Initialization Method	
	def __init__(self):

		self.todos = []

	#return Items of ToDo
	def __len__(self):
	
		return len(self.todos)

	#ToDo is sorted as DeadLine
	def sort(self):

		self.todos.sort(key = lambda x : x.duedate)	#!!!


		def __add__(self, item):

			self.todos.append(item)
			self.sort()

		def __iadd__(self, item):

			self.__add__(self,item)
			return self

	#IndexAccess emulation
	def __getitem__(self, idx):

		return self.todos[idx]
	
		#eassignment of Index emulation
		def __setitem__(self, idx, item):
		
			self.todos[idx] = item
			self.sort()

		#del points Index emulation
		def __delitem__(self, idx):

			del self.todos[idx]

	#return notEnd list of ToDo
	def get_remaining_todos(self):
	
		return [t for t in self.todos if not t.finished]

user@ubuntu~$ python3

from datetime import datetime
from todoitem import ToDoItem
from todocontainer import ToDoContainer

todos = ToDoContainer()
for d in range(19, 0, -1):

. . . td = ToDoItem('title {}'.format(d), datetime(2012, 10, d))
. . . todos += td
. . .
Traceback (most recent call last):
File "<stdin>", line 3, in <module>
TypeError: unsupported operand type(s) for +=: 'ToDoContainer' and 'ToDoItem'

コメントの英語は自分が分かるように書いているので汚いのは気にしないでください。

最後の行の

todos += td

でエラーが出ていると思っているのですが
これは特殊メソッド__add__()で呼び出しているんですよね？
だったらなぜ+=が通らないのかが分からないです。
Pythonのバージョンなどが原因とは考えられないです。(たぶん
ちなみにPython3.4です。

お時間があれば回答よろしくお願いします。

Answer 1

他の方も指摘されていますが、クラスのメソッド定義のインデントが揃っていないせいだと思います。
少なくとも__add__や__iadd__はクラスのメソッド定義になるべきもので、関数内定義の関数になっていてはおかしいですよね。

Comments

[tsunpoko]

ありがとうございました。
インデントを揃えてみたらできました。

Answer 2

インデントがずれているってことば、そもそも文法的におかしな記述になっているって事なので、試しにデバッガ（こちら）を使ってみては？

♪飾りじゃないのよぉ～○○はぁ～っ！

Comments

[tsunpoko]

ありがとうございます。

Answer 3

これのインデントよくわからないんですが、投稿したソースコードであってますか？

Comments

[TaMaMhyu]

その書籍の記載が間違っているような気がします。
おそらくaddとiaddの関数部分はインデントが1段多くなってしまっている気がします。
setitemとdelitemも同様かもしれません。

[TaMaMhyu]

公式の正誤表には出てなさそうですが、正誤表にない誤植を見つけたという人もいるようなので、そのうち更新されるのかもしれません。

Answer 4

Pythonはよくわかりませんが、
+=はaddではなくiaddが呼び出されるのではないでしょうか？

原因かどうかはわかりませんが、質問にあるコードはインデントがずれているようですが、問題ないのでしょうか？
addとiaddがsortの中で定義されていることになっていないでしょうか？
もし、sortの中で定義されていることになっているとすると、ToDoContainerには+と+=が定義されていないことになるかと思われます。

Comments

[tsunpoko]

addにiが抜けていました、すいません。
一応+と+=の場合も試したけどダメできませんでした..

インデントをずらして試してみますね。ありがとうございます。
自己回答のところに画像をはったので
インデントがあっているか確かめてもらえまでしょうか。

[tsunpoko]

ありがとうございます。