Рекомендации по использованию Rails - контроллер или модель?

Question

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

@stripe_cards = Stripe::Customer.retreive(self.stripe_customer_id).cards.all

Дело в том, что я не точно, где (с точки зрения лучшей практики Rails) он подходит. Моей первой задачей является поместить ее в демонстрационный метод пользовательского контроллера, поскольку он не является бизнес-логикой и не вписывается в модель. Я также рассмотрел вспомогательные методы, но они кажутся (по моему мнению) строго используемыми при работе с HTML.

Может ли кто-нибудь из вас использовать Rails?

Спасибо! Francis

Answer 1

Хороший вопрос. Всякий раз, когда вы видите переменную экземпляра в рельсах (начиная с @), она обычно является битом представления/контроллера.

@stripe_cards = Stripe::Customer.retreive(self.stripe_customer_id).cards.all 

Однако, глядя на хвостовом конце этого

Stripe::Customer.retreive(self.stripe_customer_id).cards.all 

Это может соответствовать лучше не в модели, где вы можете повторное та же линия, но есть безопасность добавляемого обработки ошибок и предсказуемое поведение. Например,

# user.rb 

def stripe_customer_cards 
    Stripe::Customer.retreive(self.stripe_customer_id).cards.all 
    rescue Stripe::InvalidRequestError 
    false # You could use this to render some information in your views, without breaking your app. 
end 

Также обратите внимание на использование self. Обычно это подразумевает использование модели Rails, потому что вызов self в контроллере фактически относится к контроллеру, что делает его практически бесполезным, если вы действительно не знаете, что делаете.

EDIT

Оказывать сообщение об ошибке, просто написать вызов для перенаправления или визуализации, с возможностью alert.

if @stripe_cards = current_user.stripe_customer_cards 
    # Your being paid, sweet! 
else 
    # Render alert info :(
    render 'my_view', alert: 'This is an alert' 
    redirect_to other_path, alert: 'Another alert' 
end 

Я также хотел бы подчеркнуть, что вы не должны обрабатывать ошибки только потому, что можете. Не обрабатывайте ошибки, которых вы не ожидаете. Если обрабатывать ошибки вы не ожидаете, что будет

• пользователи путайте
• сделать ошибки в коде сложнее исправить
• Преувеличивайте время до признана ошибка

Answer 2

Я бы рекомендуется добавить виртуальный атрибут в вашей User модели:

# app/models/user.rb 
def cards 
    Stripe::Customer.retrieve(stripe_customer_id).cards.all # note the spelling of `retrieve` 
end 

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

user = User.first 
#=> #<User id:1> 

user.cards 
#=> [Array of all cards]