Bu kullanıcı yöntemi parametresi iyi adlandırılmış mı?

Question

Bir Kullanıcının bir Rails uygulamasında bir gönderi veya yorum düzenleyip düzenleyemeyeceğini kontrol etme yöntemim var. Bir kullanıcı kuruluşlar her iki türdeki sahip olabilirler, çünkü bu yöntem bir parametre post_or_comment olarak bunlardan birini almak yapmaya karar: Bu muğlak bir parametre olarak bu gibi herhangi bir nesneyi almak için iyi bir uygulama mı

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

ve yapar Parametre için seçtiğim isim anlamlı mı?

CanCan gibi sofistike bir kullanıcı rolü işleyicisi ile ilgilenmiyorum, öğrenmeyi ve basit tutmayı tercih ediyorum.

Answer 1

Şema içinde bir gönderinin bir yorum türü olduğu veya tersi olduğu anlaşılırsa, o zaman her iki tür için de eşit olarak uygulanacağı anlamına gelen bir veya başka bir biçim olarak ifade etmek için kafa karıştırıcı bir şey değildir. .

Genel olarak, iyi bir nedeniniz olmadığı sürece işleri aşırı kısıtlamaktan kaçınmak en iyisidir.

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 

bu kodu için en kötü durum başarısızlık durumu öyle mi: bunu sahipsiz bir şeyi geçiyoruz çünkü Orada muhtemelen çoğu zaman işleri bir birine çok özel bir yöntem dönüm yolları, ve değilse bu kadar "hayır" diyor. Artık bir user mülküne sahip olabilecek veya olmayabilecek keyfi şeylerden geçebilir ve beklendiği gibi çalışacaktır. Diğer özel durumlar için, case ana sayfasına başka bir when ekleyebilirsiniz.