Ruby On Rails-User-Modell für mehrere Typen

FWIW, das sieht fein zu mich. Ich könnte schauen, declarative_authorization zur Verwaltung Ihrer Rollen, die am Ende vielleicht etwas umständlich, vor allem vom UI-Standpunkt. Verwaltung der Benutzer mit mehreren Rollen scheint eher geeignet als STI in dieser situation, weil, wie Sie sagen, ein Benutzer kann sowohl manager und Mieter, etc.

Einen Ansatz zu halten, den code für manager, Mieter und Eigentümer getrennt, während es für die Verwendung von allen drei Rollen in einer einzigen Instanz sein könnte, um dynamisch Module darstellen die Rollen basierend auf dem, was Rollen, die ein Benutzer hat. Sie hätten immer noch eine Basis-Klasse der Benutzer, aber statt mit STI, die würde, beschränken Sie zu einem einzigen Vererbung Struktur, Sie könnten mix-in-Module auf der Grundlage der Rollen, die jeder Benutzer hat, die vielleicht einfach pivot auf welche Eigenschaften belong_to einem bestimmten Benutzer während der Initialisierung.

Für diese könnte ich erstellen Sie eine Benutzer-Fabrik, die Sie besichtigen können die Benutzer für die verschiedenen Rollen, die Sie spielt und verlängern die singleton-Klasse, dass der Benutzer mit dem entsprechenden Modul: erweitern Sie zusammen mit dem Mieter-Modul für Anwender, die Mieter haben Eigenschaften, die Erweiterung der Manager-Modul für Benutzer, die verwalteten Eigenschaften, etc.

Aus der Sicht der declarative_authorization, Sie könnte erklären Sie die role_symbols ähnlich abhängig, ob Verbände wie managed_properties besiedelt wurden, zum Beispiel:

def role_symbols  
  @roles ||= {:manager => :managed_properties, 
    :tenant => :rented_property, 
    :owner => :owned_properties}.map do |k,v|
      k if !send(v).blank?
    end.compact
end

Oder etwas ähnliches. Sie würde wahrscheinlich wollen, legen Sie die Rollen UND schließen Sie die entsprechenden module an der gleichen Zeit. Ruby und Rails bietet viele Möglichkeiten, über metaprogramming-Techniken dynamisch dekorieren Sie mit der Funktionalität der einzelnen Modelle. Mein Ansatz, der möglicherweise oder möglicherweise nicht für Ihre Anwendung geeignet, aber es gibt unzählige andere Möglichkeiten, wie Sie nähern konnten das problem und halten Sie Ihren code sauber und TROCKEN.

Insgesamt scheint es mir deine Daten Modell solide ist, und Ihr Instinkt nicht zu verwenden STI zu verwalten mehrere Rollen, ist richtig. Es sieht nicht zu mir, wie Sie schon overthought es -- ich denke, du bist auf dem richtigen Weg. Es ist eigentlich ziemlich Rails-y für einen ersten Durchlauf.

EDIT: weißt Du, je mehr ich darüber nachdenke, bin ich mir nicht sicher, was der Vorteil von halten die manager - /Mieter - /Eigentümer-Funktionalität in separate Module wirklich ist. In meinem früheren Inkarnation als Java - /C# - Typen, die ich hätte alles über die SRP/IOC und totale Trennung von Bedenken. Aber in Ruby und Rails ist es bei weitem nicht so große einen deal, da es dynamisch typisiert und die Kupplung ist nicht annähernd so groß, oder zumindest die gleiche Art von Sorge, dass es in der statisch typisierten Umgebungen. Sie könnten völlig in Ordnung, nur alles von der einzelnen Rolle die Funktionalität in den single-User-Modell und keine Sorge mit den Modulen, zumindest noch nicht.

Ich bin auf dem Zaun hier, und würde es begrüßen, input von anderen. Für mich einer der Vorteile von Ruby-Klassen, im Gegensatz zu Java-Klassen/Pakete oder .NET-Klassen/Baugruppen, ist, dass man Sie immer umgestalten, wie Sie es brauchen und nicht annähernd so besorgt über die Klassen -, package -, namespace -, dll-oder jar gekoppelt ist, um ein anderes, etc. Ich sage nicht, dass SRP nicht wichtig ist, in Ruby überhaupt nicht. Aber ich bin nicht annähernd so paranoid wie ich verwendet zu werden.

EDIT: Paul Russell ist ein ausgezeichneter Punkt. Ich denke, Sie sollten ernsthaft in Erwägung ziehen, dass mehrere Mieter/Verwalter/Vermieter pro Grundstück. In Rails das könnte ausgedrückt werden durch eine relationale Tabelle und eine has_many :through association, plus STI zu beschreiben, die verschiedenen Arten von Beziehungen. Ich denke auch, dass es notwendig sein wird, zum invertieren der Beziehung zwischen dem Nutzer (als Mieter) und der Eigenschaft. Eine Eigenschaft kann mehr als ein Mieter, aber ein Mieter kann nicht Leben mehr als eine Eigenschaft. (oder vielleicht kann Sie? Scheint nicht rechts, aber...)

Vielleicht so etwas wie (das ist sehr quick and dirty, so verzeihen Sie etwaige fehlende details bitte):

class PropertyRole < ActiveRecord::Base
  belongs_to :user
  belongs_to :property
end

class Managership < PropertyRole
  # Manager functionality here
end

class Ownership < PropertyRole
  # Owner functionality here
end

class User < ActiveRecord::Base
  belongs_to :residence, :class_name => 'Property', :foreign_key => 'residence_id'
  has_many :managerships
  has_many :ownerships

  has_many :owned_properties, :through => :ownerships, :classname => 'Property'
  has_many :managed_properties, :through => :managerships, :classname => 'Property'
end

class Property < ActiveRecord::Base
  has_many :tenants, :class_name => 'User', :foreign_key => 'residence_id'
  has_many :managerships
  has_many :ownerships
  has_many :owners, :class_name => 'User', :through => :ownerships
  has_many :managers, :class_name => 'User', :through => :managerships
end

Wie gesagt, das ist quick and dirty, ein high-level-first-pass. Beachten Sie, dass wir jetzt schon erstellt Managership und Eigentum-Rollen, die enthalten kann, die Manager und Eigentümer-spezifische Funktionen, die Beseitigung der ehemaligen dilemma, ob zu silo, daß die Funktionalität in separate Module.

** Beachten Sie auch, dass habe ich invertiert der Mieter/Objekt-Rolle -- ich denke, dies war eine notwendige änderung, die an Ihre domain. Offensichtlich ein Wohnsitz kann mehr als ein Mieter. Es scheint mir (im moment), dass Sie zu halten kann der Mieter-spezifische Funktionen auf die User-Modell.