このユーザー・メソッド・パラメーターの名前は？

Question

私は、ユーザが自分のRailsアプリケーションの投稿やコメントを編集できるかどうかをチェックする方法を持っています。ユーザーがエンティティの両方のタイプを所有することができますので、私は、このメソッドは、パラメータpost_or_commentとしてそれらのいずれかを取ることにしました：これは曖昧パラメータとして、このように任意のオブジェクトを取ることをお勧めし

class User < ActiveRecord::Base 
    def can_edit?(post_or_comment) 
    post_or_comment.user == self || self.admin? 
    end 
end 

、といパラメータのために私が選んだ名前は理にかなっていますか？

CanCanのような洗練されたユーザーロールハンドラに興味はありません。私は学んでいるので、簡単に保つことにします。

Answer 1

あなたのスキーマ内の投稿がコメントの一種であると理解されていれば、両方の種類に等しく適用されるという意味で、どちらかというと、 。

非常に正当な理由がない限り、過度に制限することを避けるのが一般的です。あなたはそれを所有されていないものを渡しているからだとあり、おそらくほとんどの時間の作品1にあなたの非常に具体的な方法を回すの方法だ、そうでない場合：

def can_edit?(thing) 
    # Admin can edit anything. 
    return true if (admin?) 

    case (thing) 
    when User 
    # Users can edit themselves 
    thing === self 
    else 
    if (thing.respond_to?(:user)) 
     # If the owner matches. 
     thing.user === self 
    else 
     # Don't really know, so say no by default. 
     false 
    end 
    end 
end 

このコードのための最悪の場合の障害状態が、それということです"いいえ"と言います。これで、userプロパティがあるかもしれないし、そうでないかもしれない任意のものを渡すことができ、それは期待通りに動作します。その他の特別な場合は、caseに別のwhenを追加することができます。