Comment remplacer la “nouvelle” méthode pour rails modèle

Dans mon application rails, j'ai un modèle avec une date_debut et end_date. Si l'utilisateur sélectionne le 1er janvier 2010 la date_debut et 5 janvier 2010 la end_date, je veux qu'il y ait 5 cas de mon modèle créé (un pour chaque jour sélectionné). Donc ça va ressembler à quelque chose comme

Jan 1, 2010
Jan 2, 2010
Jan 3, 2010
Jan 4, 2010
Jan 5, 2010

Je connais une façon de gérer cela est de faire une boucle dans le contrôleur. Quelque chose comme...

# ...inside controller
start_date.upto(end_date) { my_model.new(params[:my_model]) }

Cependant, je veux garder mon contrôleur maigre, plus je veux garder la logique du modèle en dehors d'elle. Je suppose que j'ai besoin de remplacer la "nouvelle" méthode dans le modèle. Quelle est la meilleure façon de le faire?

Pourquoi voulez-vous?
J'ai un formulaire que l'utilisateur remplit pour créer un modèle. Mais cette forme est juste un squelette pour remplir les détails de mon modèle. La forme a des détails comme le "début" et "fin" des points. Pour créer un modèle, il doit les points start et end rempli. Je peux le faire dans le contrôleur, mais je suppose que ce type de logique devrait aller dans le modèle.

OriginalL'auteur Lan | 2010-12-07

  1. 17

    @Brad dit, vous ne voulez certainement pas à remplacer les initialiser. Bien que vous pourriez remplacer after_initialize, qui n'a pas vraiment ressembler à ce que vous voulez ici. Au lieu de cela, vous voudrez probablement ajouter une méthode à la classe comme @Pâtes suggère. Afin de l'ajouter à votre modèle:

    def self.build_for_range(start_date, end_date, attributes={})
  start_date.upto(end_date).map { new(attributes) }
end

    Puis l'ajouter à votre contrôleur:

    models = MyModel.build_for_range(start_date, end_date, params[:my_model])
if models.all?(:valid?)
  models.each(&:save)
  # redirect the user somewhere ...
end

    OriginalL'auteur Trotter

  2. 10

    Ne pas remplacer initialize Il pourrait peut-elle casser beaucoup de choses dans vos modèles. SI nous savions pourquoi vous avez besoin de nous pourrait permettre de mieux ( ne pas bien comprendre votre explication de la forme d'un squelette, vous voulez les attributs de forme pour créer d'autres attributs?? voir ci-dessous). J'utilise souvent un crochet comme Marcel suggéré. Mais si vous voulez tout le temps, pas juste avant de créer ou de sauvegarder un objet, utilisez la after_initialize crochet.

    def after_initialize
  # Gets called right after Model.new
  # Do some stuff here
end

    Aussi, si vous êtes simplement à la recherche pour certaines valeurs par défaut vous pouvez fournir par défaut accesseurs, quelque chose comme: (où some_attribute correspond à la colonne nom de votre modèle d'attribut)

    def some_attribute
  attributes[:some_attribute] || "Some Default Value"
end

    ou un écrivain

    def some_attribute=(something)
  attributes[:some_attribute] = something.with_some_changes
end

    Si je comprends bien votre commentaire correctement, il semble que vous exposez une forme qui permettrait de rendre votre modèle incomplet, avec les autres attributs basés sur des parties de ce formulaire? Dans ce cas, vous pouvez utiliser l'une des méthodes ci-dessus after_initialize ou some_attribute= à créer d'autres attributs de votre modèle.

    Découvrez la ActiveRecord::Base source (github.com/rails/rails/blob/v3.0.3/activerecord/lib/...) Regardez la méthode initialize et voir ce qu'il se passe, c'est l'extraction de colonnes à partir de la db, l'affectation de variables d'instance, etc, si vous remplacer mais ne pas définir ces attributs, il n'y a pas de garantie active_record va se comporter correctement.
    Notez que dans les Rails 3.2.x, les touches de attributes sont des chaînes de caractères, par opposition à des symboles. De sorte que vous devez attributes["some_attribute"] ci-dessus.
    Il n'y a aucune raison de ne pas remplacer les initialiser. Contraire, si vous regardez dans les rails de code source, vous l'aurez compris, initialiser était destinée à être substituée. Tout ce que vous devez faire est d'appeler super dans votre substituée à l'initialiser, mais c'est une règle qui devrait être suivie par tous les développeurs ruby et rails de développeurs silencieusement attend que vous pour le faire.

    OriginalL'auteur brad

  3. 2

    Je suppose que vous voulez définir des valeurs par défaut pour votre modèle attribut ?

    Il y a une autre solution que d'impérieuses ; vous pouvez définir des rappels :

    class Model

before_create :default_values
def default_values
  ...
end

    OriginalL'auteur Marcel Falliere

  4. 2

    pourquoi ne pas simplement créer une méthode dans votre modèle comme celui-ci

     def self.create_dates(params) 
   [...] 
  end

    contenant cette logique (essentiellement votre boucle?)

    Je le veux dans le modèle parce que c'est là la logique de la création elle-même doit être, non? Mon modèle a juste besoin d'un début et une date de fin et ensuite, il sera en mesure de créer "un exemple" de lui-même. (l'instance se trouve être plusieurs lignes dans la table de base de données)
    donc, je ferais ce que j'ai dit dans ma réponse. un create_dates méthode de classe avec votre boucle de l'appel de nouveau à l'intérieur.
    Désolé, j'ai besoin de mot ma question mieux. Mais, fondamentalement, je veux juste savoir comment remplacer Monmodèle.nouvelles(les attributs)
    Je ne ferais pas ça si j'étais vous, mais peut-être que je me trompe. Intéressé par la réponse de toute façon!
    +1, Vous ne devriez pas avoir à remplacer new, indépendamment de la façon dont vous êtes désireux de votre modèle de comportement. Si c'est absolument nécessaire pour ce faire, dans le modèle, je voudrais utiliser une combinaison de la suggestion ci-dessus avec un before_create de rappel. Le rappel ont besoin d'un failsafes toutefois, afin de prévenir la spanwed instances de frai de leur propre.

    OriginalL'auteur Pasta

  5. 2

    Vous pouvez utiliser:

    def initialize(attributes = nil)
  # do your stuff...
end

    Bien que j'ai lu quelque part qu'il n'était pas recommandable...

    ActiveRecord fait beaucoup de trucs de fou et cette méthode n'est pas garanti pour être exécuté comme vous le souhaitez.

    OriginalL'auteur jordinl

  6. 2

    Cela pue de la méthode de factorisation avec patrons...de le rechercher.

    Si vous êtes réticents pour une raison quelconque pour aller avec create_date par @Pâtes, puis éventuellement de créer un simple ruby objet (pas ActiveRecord soutenu), nommé YourModelFactory/Template/que ce soit avec deux instance de vars, vous pouvez utiliser votre standard params[:foo] pour attribuer ces - définir et appeler une méthode de classe qui renvoie vos objets réels.

    Votre contrôleur logique ressemble maintenant à ceci:

    mmf  = MyModelFactory.new(params[:foo])
objs = mmf.create_real_deal_models

    Bonne chance.

    Je suis d'accord avec cela, maintenant qu'il est expliqué qu'il veut réellement créer plusieurs modèles, je pense que l'Usine est en ordre.
    Je trouve les usines sur les modèles sont mieux fait que les méthodes de la classe que de séparer les classes. Donc, def.make_me_things sur MakeMeThings.de nouvelles.
    Je suis tout à fait +1 sur def.make_me_things - juste essayer de lui faire sentir pour modelish pour l'affiche originale.

    OriginalL'auteur Cory

  7. 2

    Strictement, bien que tardive, la bonne façon de remplacer nouveau dans un modèle est

    def initialize(args)
    #
    # do whatever, args are passed to super
    #
    super
end

    OriginalL'auteur s1mpl3