kawazrepos/Kommonz — comment on commit 00c7f1357f31b929ffb678db2467e66f8ee73cf4
File:  line 0
By giginet:

コード見ました！全体的にスゴく理想的にかけていてよかったです。

今後のために数カ所修正したので、以降よろしくお願いします。

models.py
------------------------
- 大まかには特に問題なし
- verbose_nameはこんなに改まった書き方をしなくても単語一つで十分。それと頭文字はupperの方が良いかも
- permalinkデコレータの使い方についてはこんな感じです。URLのハードコーディングはしない

urls.py
--------------------------------------
- ユーザーのinboxは/messages/で見れた方が良いですね
- Kawazのように、/messages/で送信、受信の最新10件を一覧表示しておいて、別にinbox, outboxを作ってもいいかも

views.py
--------------------------------------
- サブクラスからスーパークラスのメソッドを呼び出すときはsuper(class, instance)を使ってください。何故かというと、Pythonは多重継承が可能なため、直接スーパークラスのメソッドを呼び出すと、よろしくないことが起きる可能性があるため。
- pathの生成は、文字列を直接連結するのではなく、os.path.joinを使いましょう。よく使うのでos.pathのドキュメントを読んでおくといいです
- 文字列を生成するときは直接繋ぐのではなく、変数展開を使って書きましょう。そちらの方が見通しが良いです。
- そもそも、create_template_messageの使い方、目的が謎。まだどこからも呼び出されていないようなので何に使うんだろう･･････？

全般的に
-----------------------
- import順がたまにぐちゃぐちゃなので、意図がないのであればなるべく統一させて書いてください
- 現行の仕様だと、自分自身にメッセージが送れるんだけどこれはいいのかな？仕様としてはアリ。弾いてもOK