SoftwareFoundationGroupAtKyotoU / automata

Other
3 stars 10 forks source link

Refactor and add comments on lib/user.rb. #374

Open krtx opened 9 years ago

krtx commented 9 years ago

プログラム中にいくつかコメントがあるので見て頂けるとありがたいです。

krtx commented 9 years ago

app.rb の 47 行目で real_login を隠すべきときに User#login を token に上書きしている処理があって、real_login と login の使い分けには意味がありました。

real_login を隠さなくてよいときに login と real_login の2つのメソッドが提供されていることに意味がなくなるので、User 側で Conf を読み込んで、状況に応じて token と real_login のいずれかを返すメソッドを用意したほうが良いようにも思える。ただ今のままでも別に良いかもしれない。

westpaddy commented 9 years ago

real_loginとtokenを使い分けようとしていた痕跡は残っているんですが、使い分けの方針が周知されていなくて(私も知りません)、あってないような状況に今なってます。