Kosei Kitahara's Blog

id:Voluntasが、早速コードレビューしてくださった。6つぐらいるけど、今日はとりあえず、Modelのリファクタリングのアドバイスを実装した。diffはここ。

timeline = Timeline.all()
timeline = timeline.filter('author =', user)
timeline = timeline.filter('group =', group)

と今まで書いていたけど、Pythonは長いコードを'\'を使って改行できる。それを利用し、Modelで複数のfilterを利用する場合でも以下のように書く。

timeline = Timeline.all() \
  .filter('author =', user) \
  .filter('group =', group)

他にも色々とアドバイスを頂いているので、コードを見やすくしていくことにする。 Special thanks voluntas.