デコレータ エラーにつきまして

Question

Python 3　初心者です。
すみません、下記エラーが出る理由をどなたか教えて頂けないでしょうか…

def document_it(func):

... def new_function(*args, **kwargs):
... print('Running function:', func.name )
... print('Positional arguments:', args)
... print('Keyward arguments:', kwargs)
... result = func(*args, **kwargs)
... print('Result:', result)
... return result
... return new_function
...

def add_ints(a, b):

... return a + b
...

add_ints(3, 5)

8

cooler_add_ints = document_it(add_ints)
cooler_add_ints(3, 5)

Traceback (most recent call last):
File "<stdin>", line 1, in <module>
TypeError: 'NoneType' object is not callable

お手数をお掛け致しますが、よろしくお願いいたします。

Comments

[KSwordOfHaste]

インデントが再現できるようにと「...」を用いて記載されてますが、本サイトではコード用のマークダウン（コード行群の前後に2行だけ```だけの行を挿入）で元のコードを正確に表記できます。「https://teratail.com/help#about-markdown」の「コードを入力」を参考にして質問を編集していただければと思います。

[umyu]

＞KSwordOfHasteさんへ　File "<stdin>", line 1, in <module>となっていることから、Pythonの対話モードの内容をコピペした感じがします。

[KSwordOfHaste]

はい、そうではありますがコードであることは変わりないと思いました。指摘しても対応してくださる方が少ない印象なのですが気づいたときは啓蒙の意味でコメントしておくのもいいかなと思いまして。

[KokiTashiro]

すみません、コピペしてました…　「https://teratail.com/help#about-markdown」参考にさせて頂きます。　ありがとうございます。

Answer 1

TypeError: 'NoneType' object is not callable

なので、関数:document_itの戻り値がNoneです。

     return result
     return new_function

のインデントが一段深くてReturnの既定値Noneが返ってるのではないでしょうか。
return new_functionのインデントを一段浅くしてみてくださいな。

あと、ソースコードは.pyファイルに書いて保存すると、python ファイル名.pyで実行できるかと。

Comments

[KokiTashiro]

ありがとうございます！　解決致しました。

Answer 2

本件の質問からは外れますが、kwargsで可変長引数を受けるには引数のところだけ**kwargsと言うようにアスタリスクを2つ頭に付ける必要がありますよ。

markdownのboldに取られてしまっていただけのようでしたorz

Comments

[mkgrei]

markdownのせいで太字に…

[KSwordOfHaste]

自分は元のコードを想像するのが面倒なときteratail apiを使って原文を覗きみするようなSPAこさえましたが、我ながら「なにか余計なことしてる」と感じます。
いつも指摘できるわけでもないですが、気が向いたときにでもコメント欄で「マークダウン」啓蒙をするのは悪くないかと思います。後続する質問の際にマークダウンを使ってくださる方もいらっしゃるのでまったくの無駄でもないかなと思ったりします。かなりの確率でスルーされちまうのが寂しい点ではありますが。

[wakame]

なるほどこういうパターンもあるのですね

[KSwordOfHaste]

to: 質問者さん
混乱されるかも知れないので蛇足コメントしておきます。teratailの質問と回答（コメントは除く）はマークダウン記法が前提となってます。質問文の本文では本来コード部分を
```python
def new_function(*args, **kwargs):
　　...
```
のように```で囲んで書かないといけなくて、そうしておけばumyuさん回答内にあるようにちゃんとコードとして見やすく自動フォーマットされます。しかし```で囲まないとコードの中にある**などは「太字にする」という別のマークダウンとして解釈されてしまい、本来のコードがどうなってるか分かりにくくなります。
自分が質問コメントで書いておいたurlにマークダウンの書き方があります。こうした混乱を避けるためにそれを参照して正しくマークダウンを使ってほしいということです。おわかりでしょうか？