2

我正在使用CFWheels在 ColdFusion 中开发应用程序。

我有一个模型叫Vote.cfc. 在创建、更新或删除 Vote 对象之前，我需要从另一个模型中获取 post 对象：Post.cfc. 投票属于帖子。一个帖子有很多票。

使用来自post对象的数据，我需要to validate the vote跨多个标准和多个功能的对象。我能想到的唯一方法是持久化 post 对象以便这些函数可以使用它，就是将它存储在请求范围内。

其他人说这是不好的做法。但我无法找出原因。我认为请求范围是线程安全的，在这种情况下使用它是有意义的。

我的另一种选择是在每个需要它的函数中加载 post 对象的新实例。尽管 Wheels 使用缓存，但这样做会导致请求时间增加 250%。

更新

这是一些示例。首先，控制器处理查看投票对象是否已经存在。如果是，它会删除它，如果不是，它会创建它。控制器功能本质上是一个切换功能。

Votes.cfc 控制器

   private void function toggleVote(required numeric postId, required numeric userId)
    {
    // First, we look for any existing vote
    like = model("voteLike").findOneByPostIdAndUserId(values="#arguments.postId#,#arguments.userId#");

    // If no vote exists we create one
    if (! IsObject(like))
    {
        like = model("voteLike").new(userId=arguments.userId, postId=arguments.postId);
    }
    else
    {
        like.delete()
    }           
}

模型 VoteLike.cfc

之后，模型中注册的回调会在验证之前触发。它调用一个函数来检索投票所属的 post 对象。函数 getPost() 将帖子存储在请求范围内。它现在可用于模型中的一堆验证函数。

// Load post before Validation via callback in the Constructor
beforeValidation("getPost");

private function getPost()
{
    // this.postId will reference the post that the vote belongs to
    request.post = model("post").findByKey(this.postId);
}

// Validation function example
private void function validatesIsNotOwnVote()
{
    if (this.userId == request.post.userId)
    {
       // addError(message="You can't like your own post.");
    }
}

getPost() 函数的替代方法是使用范围调用“ this.post().userId”来获取 post 对象，如下所示：

private void function validatesIsNotOwnVote()
{
    if (this.userId == this.post().userId)
    {
        addError(message="can't vote on your own post")
    }
}

但是我必须this.post().userId对每个函数重复这个范围调用，这就是我认为减慢请求的原因！

4

2 回答 2

2

根据 OP 中的评论线程更新新答案

由于您正在从 VoteLike.cfc 扩展基本 Vote 对象，因此 CFC 共享一个本地线程安全变量范围（只要您不将其存储在应用程序或服务器范围内，以供其他人使用）。这意味着您只需设置一个值，例如 Variables.Post，一次，然后在 CFC 堆栈中的任何函数中引用该值。

因此，将您的 getPost() 函数更改为：

beforeValidation("getPost");

private function getPost(){
    Variables.Post = model("post").findByKey(this.postID);
}

现在，在 VoteLike.cfc 中的任何函数中，您都可以引用 Variables.Post。这意味着您在本地 CFC 变量范围内有一个 Post 实例，您不必通过参数或其他范围传递它。

原答案如下

处理此问题的“最正确”方法是将对象作为参数传递给每个单独的函数。或者将 Post 添加为 Vote 对象的属性，以便 Vote 对象可以访问完整的 Post 对象。

<cffunction name="doTheValidation">
    <cfargument name="ThePost" type="Post" required="true" />
    <cfargument name="TheVote" type="Vote" required="true" />
    <!--- do stuff here --->
</cffunction>

这是一个不好的做法的原因是因为您要求您的对象可以访问可能存在或可能不存在的外部范围，以便完成它们的工作。如果您决定要在一个页面上处理多个投票或帖子，那么您必须摆弄外部范围以使事情正常进行。

你最好传递你的对象，或者使用组合将你的对象以这样一种方式拼凑在一起，这样它们就可以相互感知。

更新

关于性能问题，如果你使用组合将你的对象绑定在一起，你最终会在内存中只有两个对象，所以你不必实例化一堆不必要的 Post 对象。此外，将 CFC 作为参数传递给函数将通过引用传递 CFC，这意味着它不会在内存中创建 CFC 的副本，这也应该考虑到您的性能问题。

代码示例更新

为了说明上面的“引用”注释，设置以下文件并将它们单独放在一个目录中。

测试对象.cfc：

<cfcomponent output="false">
    <cfproperty name="FirstName" displayname="First Name" type="string" />
    <cfproperty name="LastName" displayname="Last Name" type="string" />

</cfcomponent>

索引.cfm：

<!--- Get an instance of the Object --->
<cfset MyObject = CreateObject("component", "TestObject") />

<!--- Set the initial object properties --->
<cfset MyObject.FirstName = "Dan" />
<cfset MyObject.LastName = "Short" />

<!--- Dump out the object properties before we run the function --->
<cfdump var="#MyObject#" label="Object before passing to function" />

<!--- Run a function, sending the object in as an argument, and change the object properties --->
<cfset ChangeName(MyObject) />

<!--- Dump out the object properites again, after we ran the function --->
<cfdump var="#MyObject#" label="Object after passing to function" />

<!--- Take a TestObject, and set the first name to Daniel --->
<cffunction name="ChangeName">
    <cfargument name="TheObject" type="TestObject" required="true" hint="" />
    <cfset Arguments.TheObject.FirstName = "Daniel" />
</cffunction>

当您运行 index.cfm 时，您会注意到，第一个转储的 FirstName 为 Dan，而第二个转储的 FirstName 为 Daniel。这是因为 CFC 通过引用传递给函数，这意味着在该函数中所做的任何更改都会对内存中的原始对象进行。因此没有对象的再创造，所以没有性能影响。

担

于 2011-02-02T18:53:36.490 回答
1

request是一个全局范围，仅限于页面请求（即线程安全），在该页面请求期间存在。所以这是不好的做法，就像全局变量是不好的做法一样，但持续时间很短。我认为它是在您描述的情况下将数据扔到空中或栅栏上，任何其他代码都可以从半空中拔出。

因此，对于您的情况，它可能很好 - 在消费点添加一些有用的参考，也许是关于数据request首先放置在范围中的位置。也就是说，如果您每次都返回相同的源方法，请考虑在负责在其中创建该对象的任何函数中缓存（例如getVote()）并且，您可以使用请求范围：

<cfparam name="request.voteCache" default="#structNew()#"/>
<cffunction name="getVote" output="false" access="public" returntype="any">
    <cfargument name="voteId" type="string" required="true"/>
    <cfif structKeyExists(request.voteCache, arguments.voteId)>
        <cfreturn request.voteCache[arguments.voteId]>
    </cfif>

    <!--- otherwise get the vote object --->
    <cfset request.voteCache[arguments.voteId] = vote>
    <cfreturn vote>
</cffunction>

不利的一面是，如果在请求期间有其他东西更改了数据，您将拥有一个陈旧的缓存，但看起来您不希望在执行期间有任何更改。

于 2011-02-02T18:58:52.910 回答