Sollte ich die Klasse, Methode oder Instanzmethode, und warum?

In meiner Rails-app, bei der Erstellung eines business ich habe ein Formular das folgende Feld:

   <%= check_box_tag(:default_company) %> 
   <%= label_tag(:default_company, "Set Company as Default") %>

Im wesentlichen, wenn ich ein Geschäft, wenn Sie dieses Kontrollkästchen aktivieren, ich muss es ausführen so etwas wie den folgenden code:

def set_default_company(company, user)
  exists = DefaultCompany.find(user.id)
  if exists
    exists.update_attributes(company: company)
  else  
    DefaultCompany.create(company: company, user: user)
  end
end

Beim lernen, möchte ich in der Regel tun, dass die Sachen in meinem controller, aber ich versuche zu Folgen, best practices und verwenden Sie einen fat model, skinny controller, also bin ich, die Logik wie diese:

def create
  @company = Company.new(params[:company])

  if @company.save
    if params[:default_company]
      Company.set_default_company(@company.id, current_user.id,)
    end
    flash[:notice] = "Company was successfully created."
    redirect_to @company
  else
    redirect_to new_company_path
  end
end

Hier bin ich immer verwirrt auf, ob die Methode einer Klasse oder eine Instanz-Methode, um den Anruf set_default_company. Sie beide scheinen, wie Sie funktionieren würde, und ich kann nicht einen Vorteil sehen, um den ein oder anderen.

Zusätzlich zu geben mir keine Informationen darüber, welche Methode zu verwenden, wenn jemand mir zeigen könnte eine kurze Implementierung zu schreiben als Methode einer Klasse vs. Instanz-Methode kann es geben, mich besser zu verstehen, warum.

Hier ist, wie würde ich es schreiben:

def self.set_default_company(company, user)
  # Logic here
end

def set_default_company(company, user)
  # Logic here
end

Schreiben Sie auf diese Weise sehe ich keinen nutzen für.

InformationsquelleAutor ruevaughn | 2012-07-25

15

Als Ihr name vermuten lässt, Instanz-Methoden auf ein Modell sollte verwendet werden, für die Logik/Operationen, die sich auf eine bestimmte Instanz ein Benutzer (der auf dem die Methode aufgerufen wird.) So könnte man denken, dass die Einstellung der Standard-Firma, die für einen Benutzer als eine Instanz-Methode auf User. Klasse Methoden sind für Dinge, die nicht funktionieren mit einer einzelnen Instanz eines Modells oder für die Fälle, in denen Sie nicht über die Instanz zur Verfügung. beispielsweise könnten Sie eine Klasse Methode, um aufzuräumen Ihre Datenbank wie User.purge_expired_users würde nicht für einen einzelnen Benutzer-Objekt.

z.B.
```
class User
  def set_default_company(company)
    exists = DefaultCompany.find(self.id)
    if exists
      exists.update_attributes(company: company)
    else  
      DefaultCompany.create(company: company, user: self)
    end
  end
end
```
dann Ihre controller-Methode würde wie folgt Aussehen:
```
def create
  @company = Company.new(params[:company])

  if @company.save
    if params[:default_company]
      current_user.set_default_company @company
    end
    flash[:notice] = "Company was successfully created."
    redirect_to @company
  else
    redirect_to new_company_path
  end
end
```
Alternativ könnte man denken, dass die Beziehung aus der anderen Perspektive-und eine Instanz-Methode auf Company z.B. company.set_as_default_for(user).
- Sehr gute Beschreibung für die Klasse vs. Instanz, eigentlich wirklich gelöscht, es sich für mich. Sie sind richtig, ich werde zu bewegen, diese auf meine Benutzer-Modell eher dann der Gesellschaft, der am meisten Sinn macht und liest sich besser. Ich habe eine Letzte Frage. Statt der Speicherung der Unternehmen standardmäßig in einer Tabelle namens Default-Gesellschaft, die ein Beitritt der Tabelle user_id und company_id, sollte ich einfach nur einen "default_company" - Feld im user-Modell? Der Grund, warum ich ging mit einem Beitritt der Tabelle ist, da nicht für alle Benutzer einer Firma, in der Tat die meisten nicht haben eine Firma, ich hätte also leere Felder, eine Menge.
InformationsquelleAutor mikej
4

Ich würde tatsächlich set_default_company eine Instanz-Methode auf User. Ein User hat eine Standard -Company; warum sollte ein Company müssen, was die Nutzer ist es Standard für?
```
class User
    def set_default_company(company)
        exists = DefaultCompany.find(id)
        if exists
            exists.update_attributes(company: company)
        else  
            DefaultCompany.create(company: company, user: self)
         end
     end
 end
```
- Sind Sie richtig. War ich ursprünglich im Begriff dies zu tun, aber für einige Grund meine überlegungen führen mich Weg von diesem.
InformationsquelleAutor icecream
2

Meiner Meinung nach, ich erstelle immer ein class method wenn die Methode in Frage stellt Informationen/Verhalten, das ist ziemlich generisch, unter allen die instanziierten Objekte, die sich von der instance methods, dass ich verwenden, wenn ich glaube, es ist mehr wie eine bestimmte Aktion des instanziierten Objekts in Frage.

Aber das ist meine Sicht.
- Vielen Dank für Ihre Informationen, es ist alles sehr hilfreich
InformationsquelleAutor MurifoX
1

Ein paar Dinge: haben Sie eine separate Tabelle für DefaultCompany? Dies scheint, wie es sollte ein boolean-flag auf der Tabelle Unternehmen.

Nächste, ist es, eine Verbindung zwischen Unternehmen und Nutzer? Wenn dem so ist, scheint es der beste Weg, es zu tun wäre

In der Benutzer-Modell
```
def set_default_company(company)
  self.companies.each do |c|
    c.update_attributes(:default => false)
  end
  company.update_attributes(:default => true)
end
```
Oder in der Firma Modell
```
def set_as_default
  update_attributes(:default_company => true)
end
```
- Ich habe eine separate Tabelle für DefaultCompany, der Gedanke führte mich Weg von der boolean ist, dass ein Benutzer has_many Unternehmen über Rollen, so dass ein Benutzer kann möglicherweise viele oder gar keine Unternehmen. Also das boolean-Feld funktioniert nicht wirklich in diesem Fall, glaube ich nicht
- Betrachten wir den Fall, wo ein Unternehmen 5 Benutzer, wenn ich die Firma boolean-Feld auf true, das würde es für alle Benutzer, richtig?
- Habe vergessen zu erwähnen, es war ein Unternehmen haben viele Benutzer über Rollen und Benutzer haben viele Unternehmen durch Rollen sowie
- wenn jeder Benutzer nur ein Standard-Unternehmen, dann sollte die Vorgabe sein, die einen booleschen auf der Rollen-Tabelle
- Hmmm ja, die Sinn machen, ein Benutzer könnte in der Theorie eine Rolle "Besitzer" und "Manager", scheint es mir so, wie ich müsste, um zu verfolgen die beiden? Oder ich glaube, ich würde nur prüfen, für alle Rollen, denen der Benutzer a gehört zu Firma x und all diese Rollen zu wahren, das scheint besser zu dir dann einen Beitritt zu Tisch?
- was ist der Zweck der DefaultCompany Tabelle? Ist es zu sagen, die Firma ist ein Benutzer, der die primäre Unternehmen, und dann gibt es ein anderes Konzept der primären Rolle? Oder, wenn ein Benutzer eine primäre Rolle, konnten Sie Rückschlüsse auf die primäre Unternehmen, die von der primären Rolle? Wenn es notwendig ist, um einen anderen primären Unternehmen aus dem Unternehmen eine primäre Rolle, dann bräuchte man eine andere Tabelle. Allerdings scheint es hier, dass Sie alles tun können, mit einem Wert auf der Zeile.
- so, primären Unternehmen wäre primary_role.company
- Der Zweck der DefaultCompany Tabelle ist die Angabe einer default-Wert, wenn ein Benutzer mehr als ein Unternehmen. Diese app beinhaltet ein system von geboten, und wenn ein Benutzer mehreren Unternehmen verbunden ist (aka eine Rolle mit zwei Firmen) dann muss ich wissen, wenn Sie ein Gebot abgeben, welche Unternehmen für die Standard-Ansage, anstatt Sie entscheiden zu jeder Zeit. Also ich möchte in der Lage sein zu nennen, so etwas wie user.default_company
- wenn ja, Sie könnte auf jeden Fall tun Sie es mit einem boolean auf Rolle für primäre Rolle, und haben dann eine Methode in der Benutzer-Modell Benutzer.default_company
- Ist es ein Vorteil, es zu tun auf diese Weise dann eher den Tisch? Ich Plane Zwischenspeichern, damit ich nicht suchen müssen jedes mal, aber vielleicht ist es sauberer als gut?
- aus einem data-Komplexität Standpunkt, wird es wahrscheinlich leichter sein, in die Zukunft zu äußern, die gleichen Ideen / haben die gleichen Daten, mit einem weniger Tabelle
InformationsquelleAutor Solomon

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.