Получить запись с url

Question

Я знаю, что это очень простой вопрос, но я думаю, что мой мозг и google-fu сегодня не работают так хорошо.

Предположим, у меня есть событие с регистраторами, и они могут заплатить за мероприятие, используя один или несколько платежей.

Я пытаюсь создать платеж, связанный с регистрантом (кто связан с событием).
Так что мой платеж должен иметь как registrant_id, так и event_id.

Мой URL выглядит следующим образом: (вложенные маршруты)

http://mysite.com/events/1/registrants/1/payments/new 

Мой контроллер выглядит что-то вроде:

def create 
    @event = Event.find(params[:event_id]) 
    @registrant = Registrant.find(:first, conditions: {id: params[:registrant_id], event_id: params[:event_id]}) 

    @payment = Payment.new params[:payment] 
end 

Я знаю, что есть гораздо лучший способ сделать это, но У меня возникли проблемы с правильной формулировкой этого слова :)

Какой синтаксис я должен использовать для создания .new aut omatically известно о event_id и registrant_id?

Answer 1

на основе обсуждения в комментариях, есть несколько способов, что вопрос можно решить: прямой путь и путь Rails.

Прямой подход к созданию объектов, связанных с ними, заключается в создании объекта с использованием new_object = ClassName.new, как это было предложено в вопросе. Затем возьмите идентификатор созданного объекта и установите его на существующий объект (непосредственно с existing_object.id = new_object.id или с помощью какого-либо другого метода, если требуется дополнительная логика). Или установить идентификатор на новом объекте, определяя пользовательский инициализатору, такие как:

class Payment 
    def initializer id_of_registrant 
    @registrant_id = id_of_registrant 
    end 
... 
end 

Преимущества такого подхода заключается в том, что она позволяет назначать регистратор идентификаторов, которые могут возникнуть из ряда объектов с различными классами, без необходимости иметь ненужное или, возможно, неправильное (для вашего решения) наследование и полиморфизм.

Путь Rails, если вы всегда имеют прямое отношение (1 к 1) между регистраторами и «обязательной» Оплата является использование has_many или belongs_to ассоциации, как описано в руководстве Rails: http://guides.rubyonrails.org/association_basics.html

для примера классов от вопроса:

class Registrant < ActiveRecord::Base 
    has_one :payment 
end 

class Payment < ActiveRecord::Base 
    belongs_to :registrant 
end 

Вы хотите использовать соответствующую миграцию для создания таблиц базы данных и внешние ключи, которые идут с этим. Например:

class CreateRegistrants < ActiveRecord::Migration 
    def change 
    create_table :registrants do |t| 
     t.string :name 
     t.timestamps 
    end 

    create_table :payments do |t| 
     t.integer :registrant_id 
     t.string :account_number 
     t.timestamps 
    end 
    end 
end 

Конечно, если вы регистранты только при необходимости произвести оплату, или сделать несколько платежей, то вам нужно будет смотреть на использовании has_many ассоциации.

С имеют и принадлежит ассоциациям, вы можете делать хорошие вещи, как:

@payment.registrant = @registrant 

если вы инстанцированы объектам вручную или

@payment.new(payment_amount) 
@registrant = @payment.build_registrant(:registrant_number => 123, 
    :registrant_name => "John Doe") 

, если вы хотите ассоциации заселены автоматически.

В Rails Guide есть много примеров, хотя, по моему опыту, только попытка найти наиболее подходящую для вашего фактического варианта использования покажет, есть ли ограничения, которых нельзя было ожидать. Подход Rails упростит будущие запросы и создание объектов, но если у вас есть очень свободная модель отношений для ваших объектов, вы можете обнаружить, что она становится ограничительной или неестественной, и эквивалентные ассоциации лучше кодируются вручную с помощью ваших дополнительных бизнес-правил.

Answer 2

Неправильно установить атрибуты id, так как идентификатор может не ссылаться на фактическую строку базы данных. Обычным делом здесь было бы использовать CanCan (https://github.com/ryanb/cancan), который, похоже, решит все ваши проблемы.

EDIT:

Если вы не используете проверку подлинности любого рода, то я бы либо поставить методы нагрузки в before_filter с, чтобы держать вещи в чистоте:

before_filter :load_event 

def load_event 
    @event = Event.find params[:event_id] 
end 

или определить некоторые фанки общий загрузчик (излишне мета- и сложный и не рекомендуется):

_load_resource :event 

def self._load_resource resource_type 
    before_filter do 
    resource = resource_type.constantize.find params[:"#{ resource_type }_id] 
    instance_variable_set :"@#{ resource_type }", resource 
    end 
end