提问人：dehuff 提问时间：10/24/2023 更新时间：10/24/2023 访问量：26
从控制器调用静态方法并更新该类的实例 - 获取 AR：：Association：：HasOneAssociation 的未定义方法
Calling a Static Method from a Controller and Updating instance of that class - Getting undefined method for AR::Association::HasOneAssociation
问：
我的问题：
正确的结构应该是什么？我已经尝试过多次重新设计，但一直遇到紧密耦合问题。
相关信息：
我正在为某些第三方软件编写一个新的端点。
它将接收一个有效负载，然后我将该有效负载传递到我的 Subscription 类的静态方法中。
从那里，我想查找与有效负载相关的订阅，建立它的实例，然后根据我的类的其余信息执行更新。我遇到一个错误，内容如下：undefined method update for ActiveRecording::Association::HasOneAssociation
端点控制器：
class Api::V2::EndpointController < Api::V5::BaseController
def connect
data = decode(request.body.read)
Subscription.static_method(data)
end
end
订阅模式：
class Subscription < ActiveRecord::Base
def self.static_method(data)
@subscription = Subscription.find_by_id(data.subscription_id)
@subscription.update(data)
end
end
订阅控制器：
class SubscriptionsController < ApplicationController
def update
#execute update
end
end
答：
1赞
max
10/24/2023
#1
一开始将代码移动到模型中并不是一个好主意。控制器需要检查更新是否成功并做出相应的响应。除了你传入的内容之外，模型也没有上下文的概念，并且已经被责任感所淹没。
我会以最直接的方式在控制器中编写此代码。
# @todo setup the correct inflections in config/initializers/inflections.rb
# @see https://github.com/rubocop/ruby-style-guide#camelcase-classes
# @see https://github.com/rubocop/ruby-style-guide#namespace-definition
module API
module V2
# @todo Write a high level description of the what the responsibility of this class is.
# the name itself is pretty smelly.
# "Endpoint" is very vague - every controller method is an endpoint.
# Why is this inheriting from a different version of the API? Very smelly.
class EndpointController < API::V5::BaseController
# @todo write a description of the purpose of this method
# @todo document the route that this method corresponds to
def connect
data = decode(request.body.read) # WTF is this?
# .find will raise an exception if the id is not valid
# Rails will catch this and return a 404.
subscription = Subscription.find(data.subscription_id)
# When you assume, you make an ass out of you and me.
# Always check if the update is successful
if subscription.update(data)
head :ok # or provide some other response
else
head :unprocessable_entity
end
end
end
end
end
如果后来发现重复此代码两次以上，请添加抽象，例如服务对象。
评论
@subscription = Subscription.find_by_id(data.subscription_id)