ryok-0319/rails_sns

rails_sns's Issues

test

コードレビュー@そのほか

dbのcolumnには基本的にnot null制約をつけましょう！
つぎからでok!
https://github.com/ryok-0319/rails_sns/blob/master/mysns/db/schema.rb

[あったらいいななので参考程度に]
replyのlevelってちょうっと抽象度が高めかな！
なんのレベル??ってなるかなと
日本語だと「公開レベル」だとおもうので accessibility_levelとかviewability_levelとか??

rubocopいれてインデントとか気をつけましょう！

コードレビュー@コントローラ

https://github.com/ryok-0319/rails_sns/blob/master/mysns/app/controllers/relationships_controller.rb#L4
オブジェクト指向っぽくてよい！
https://github.com/ryok-0319/rails_sns/blob/master/mysns/app/controllers/relationships_controller.rb#L3
strong params
https://www.transnet.ne.jp/2016/05/18/rails%E5%88%9D%E5%AD%A6%E8%80%85%E3%82%B9%E3%83%88%E3%83%AD%E3%83%B3%E3%82%B0%E3%83%91%E3%83%A9%E3%83%A1%E3%83%BC%E3%82%BF%E3%83%BCcolnr/
strong paramsをつかおう！

https://github.com/ryok-0319/rails_sns/blob/master/mysns/app/controllers/replies_controller.rb
よい!

https://github.com/ryok-0319/rails_sns/blob/master/mysns/app/controllers/tweets_controller.rb#L10
permitted_tweetってtweetが帰ってくるmethodぽい名前になっていて紛らわしいと思います

    @tweets = []
    # 全てのツイートのうち、条件を満たしているものだけを@tweetsに格納
    permitted_tweet(@tweets, @all_tweets)

を

@tweets = Tweet.permitted_tweets(user)

とすると、modelにロジックを移せてMVCの分離という意味でも良いかと思います

https://github.com/ryok-0319/rails_sns/blob/master/mysns/app/controllers/tweets_controller.rb#L75
ここも
tweet.viewable?(user)
とかにするとmodelに移せてよいかも！
unauthorized? だと、なんの権限(閲覧なのか、編集なのか)わからなそう

https://github.com/ryok-0319/rails_sns/blob/master/mysns/app/controllers/users_controller.rb#L6
railsdmでも言ってたけど、ここらへん、なるべくcontrollerはcrudだけに保てるとベストかと！
どうやればいいかはちょっと考えて見てもらえれば