Question

得到一些非常奇怪的东西，我无法解决。

我在CFC（guest.cfc）中有3个方法：

- save
- create
- update

我将一个argumentCollection传递给save方法。

saveGuest = objGuest.save(argumentcollection=guestStruct)

如果它包含一个Identier，它会将argumentCollecion传递给update方法，如果它没有将它传递给create方法。

当从app中的一个地方调用时，这工作正常，但是当我创建一个对save方法的新调用时，create方法工作正常，但是如果在update方法中传递了一个ID，则会收到错误UPDATE变量不存在。

但是，如果我通过创建cfc的对象来改变我调用它的方式，它就可以工作..

所以..

saveObject =  update(argumentCollection = arguments;

不起作用。收到更新变量不存在的错误。

saveObject =  createObject("component",'guest').update(argumentCollection = arguments);

开始工作了。

请注意，这两个调用都发生在guest.cfc本身。

当我将表单结构传递给save方法时，这个问题不会发生，但是当我传递一个标准结构（从XML导入构造）时，确实会发生这个问题。

很奇怪。

任何人对可能导致这种情况的原因有任何想法吗？

4月24日编辑 - 为guest.cfc添加了代码

<cffunction name="save" output="false" access="remote" hint="save guest">
    <cfargument name="title" type="any" required="false" default="" />
    <cfargument name="first_name" type="any" required="false" default="" />
    <cfargument name="surname" type="any" required="false" default="" />
    <cfargument name="dob" type="any" required="false" default="NULL" />
    <cfargument name="partner_first_name" type="any" required="false" default="" />
    <cfargument name="partner_surname" type="any" required="false" default="" />
    <cfargument name="partner_dob" type="any" required="false" default="NULL" />
    <cfargument name="address_1" type="any" required="false" default="" />
    <cfargument name="address_2" type="any" required="false" default="" />
    <cfargument name="address_3" type="any" required="false" default="" />
    <cfargument name="city" type="any" required="false" default="" />
    <cfargument name="state" type="any" required="false" default="" />
    <cfargument name="postcode" type="any" required="false" default="" />
    <cfargument name="country" type="any" required="false" default="" />
    <cfargument name="phone_bh" type="any" required="false" default="" />
    <cfargument name="phone_ah" type="any" required="false" default="" />
    <cfargument name="phone_mob" type="any" required="false" default="" />
    <cfargument name="fax" type="any" required="false" default="" />
    <cfargument name="email" type="any" required="false" default="" />
    <cfargument name="business" type="any" required="false" default="" />
    <cfargument name="notes" type="any" required="false" default="" />
    <cfargument name="referer" type="any" required="false" default="" />
    <cfargument name="prospect" type="any" required="false" default="" />
    <cfargument name="occasion" type="any" required="false" default="" />
    <cfargument name="occasion_date" type="any" required="false" default="NULL" />


    <!---pass to Create or Save--->
    <cfif NOT isdefined("arguments.guest_id") OR arguments.guest_id EQ "0">
        <cfset saveObject =  create(argumentCollection = arguments) />
    <cfelse>
        <cfset saveObject =  update(argumentCollection = arguments) />
    </cfif>

    <cfreturn saveObject>

</cffunction>


<!---CREATE--->
<cffunction name="create" output="false" access="private" returntype="struct" hint="Create a New Item">
    <cfargument name="provider_id" type="any" required="false" default="#session.providerID#" />
    <cfargument name="ext_ref_id" type="any" required="false" default="NULL" />
    <cfargument name="tstamp" type="any" required="false" default="#session.tStamp#" />

    <cfif isValid('date',arguments.occasion_date)>
        <cfset iOccasionDate = createODBCDateTime(arguments.occasion_date)>
    <cfelse>
        <cfset iOccasionDate = "NULL">
    </cfif>

    <cfset returnStruct = StructNew()>

    <cfquery name="insertGuest" datasource="#Application.ds#">
        INSERT INTO guest (provider_id, ext_ref_id, title, first_name, surname, full_name, partner_first_name, partner_surname, partner_full_name, address_1, address_2, address_3, city, state, postcode, country, phone_bh, phone_ah, phone_mob, fax, email, company, notes, referer, prospect, occasion, occasion_date, tstamp)
        VALUES (#provider_id#, #ext_ref_id#, '#arguments.title#', '#arguments.first_name#', '#arguments.surname#', '#arguments.first_name# #arguments.surname#', '#arguments.partner_first_name#', '#arguments.partner_surname#', '#arguments.partner_first_name# #arguments.partner_surname#', '#arguments.address_1#', '#arguments.address_2#', '#arguments.address_3#', '#arguments.city#', '#arguments.state#', '#arguments.postcode#', '#arguments.country#', '#arguments.phone_bh#', '#arguments.phone_ah#', '#arguments.phone_mob#', '#arguments.fax#', '#arguments.email#', '#arguments.company#', '#arguments.notes#', '#arguments.referer#', '#arguments.prospect#', '#arguments.occasion#', #iOccasionDate#, #CreateODBCDateTime(tstamp)#)
    </cfquery>

    <cfquery name="guest" datasource="#Application.ds#">
        SELECT max(guest_id) as id
        FROM guest
        WHERE provider_id = #provider_id#
    </cfquery>

    <cfset returnStruct.id = #guest.id#>
    <cfreturn returnStruct>

</cffunction>



<!---UPDATE--->
<cffunction name="update" output="false" access="private" returntype="struct" hint="Update an existing item">

    <!---general details--->
    <cfquery name="update" datasource="#Application.ds#">
        UPDATE guest
        SET provider_id = provider_id

            <cfif isdefined("arguments.title")>
                ,title = '#arguments.title#'
            </cfif>
            <cfif isdefined("arguments.first_name")>
                ,first_name = '#arguments.first_name#'
            </cfif>
            <cfif isdefined("arguments.surname")>
                ,surname = '#arguments.surname#'
            </cfif>
            <cfif isdefined("arguments.full_name")>
                ,full_name = '#arguments.full_name#'
            </cfif>
            <cfif isdefined("arguments.dob")>
                ,dob = #formDate2odbcDate(arguments.dob)#
            </cfif>

            <cfif isdefined("arguments.partner_first_name")>
                ,partner_first_name = '#arguments.partner_first_name#'
            </cfif>
            <cfif isdefined("arguments.partner_surname")>
                ,partner_surname = '#arguments.partner_surname#'
            </cfif>
            <cfif isdefined("arguments.partner_full_name")>
                ,partner_full_name = '#arguments.partner_full_name#'
            </cfif>
            <cfif isdefined("arguments.partner_dob")>
                ,partner_dob = #formDate2odbcDate(arguments.partner_dob)#
            </cfif>

            <cfif isdefined("arguments.address_1")>
                ,address_1 = '#arguments.address_1#'
            </cfif>
            <cfif isdefined("arguments.address_2")>
                ,address_2 = '#arguments.address_2#'
            </cfif>
            <cfif isdefined("arguments.address_3")>
                ,address_3 = '#arguments.address_3#'
            </cfif>
            <cfif isdefined("arguments.city")>
                ,city = '#arguments.city#'
            </cfif>
            <cfif isdefined("arguments.state")>
                ,state = '#arguments.state#'
            </cfif>
            <cfif isdefined("arguments.postcode")>
                ,postcode = '#arguments.postcode#'
            </cfif>
            <cfif isdefined("arguments.country")>
                ,country = '#arguments.country#'
            </cfif>
            <cfif isdefined("arguments.phone_bh")>
                ,phone_bh = '#arguments.phone_bh#'
            </cfif>
            <cfif isdefined("arguments.phone_ah")>
                ,phone_ah = '#arguments.phone_ah#'
            </cfif>
            <cfif isdefined("arguments.phone_mob")>
                ,phone_mob = '#arguments.phone_mob#'
            </cfif>
            <cfif isdefined("arguments.fax")>
                ,fax = '#arguments.fax#'
            </cfif>
            <cfif isdefined("arguments.email")>
                ,email = '#arguments.email#'
            </cfif>
            <cfif isdefined("arguments.subscribe_email_broadcast")>
                ,subscribe_email_broadcast = '#arguments.subscribe_email_broadcast#'
            </cfif>
            <cfif isdefined("arguments.company")>
                ,company = '#arguments.company#'
            </cfif>
            <cfif isdefined("arguments.notes")>
                ,notes = '#arguments.notes#'
            </cfif>
            <cfif isdefined("arguments.prospect")>
                ,prospect = '#arguments.prospect#'
            </cfif>
            <cfif isdefined("arguments.occasion")>
                ,occasion = '#arguments.occasion#'
            </cfif>
            <cfif isdefined("arguments.occasion_date")>
                ,occasion_date = #formDate2odbcDate(arguments.occasion_date)#
            </cfif>

        WHERE guest_id = #arguments.guest_id#
    </cfquery>

    <cfset returnStruct = structNew()>
    <cfset returnStruct.id = arguments.guest_id>
    <cfreturn returnStruct>

</cffunction>

Answer 1

如果没有CFC代码和调用代码的样本，更难以更精确，但是我倾向于建议您检查组件的init（）。你有一个init（）方法，你在调用它吗？例如，组件定义的概述

<cfcomponent>

    <cffunction name="init" access="public">
        <cfreturn this>
    </cffunction>

    <cffunction name="save" access="public">
        <!--- logic --->
    </cffunction>

    <cffunction name="create" access="public">
        <!--- logic --->
    </cffunction>

    <cffunction name="update" access="public">
        <!--- logic --->
    </cffunction>

</cfcomponent>

然后由以下任何一个

调用

<cfscript>
    // This will work in CF9 upwards
    objCFC = new guest(/* add any arguments you have in the init() method here */);
    objCFC.update(......);

    // This also works
    objCFC = CreateObject('component','guest').init(/* add any arguments you have in the init() method here */);
    objCFC.update(......);
</cfscript>

按照您的示例进行修改

CFC很讨厌。有各种明显的问题。

您依赖于CFC中的应用程序/会话范围，从而迫使它依赖于全局变量。这就是init（）方法用于其他事情，因为你可以init(datasource, provider_id, ext_ref_id, tstamp)将它们存储在variables.范围内，因此它不再容易受到在CFC外部未定义的变量的影响< / LI>
您正在使用大量的VARCHAR字段，并且只允许使用具有远程访问权限的cfc进行转义。这很容易受到SQL注入的影响。 Bobby先生可以告诉你为什么这很糟糕（http://bobby-tables.com/）
您继续使用字符串值NULL。如果将其传递到varchar / quotes中的数据库中，则它将存储为“NULL”而不是数据库NULL值

总而言之，最好的做法是重写这个组件。使用该组件的任何代码的代码更改将最小化到必须添加到init（）并且可能删除一些参数，但是你会发现它的稳定性得到改善，并且一切都很好，这个奇怪的问题将会消失。

Answer 2

在CF中，当您在函数内部编写<cfset iOccasionDate = ... >和<cfquery name="insertGuest" ...>之类的内容时，您将在该CFC /模板的全局变量范围内创建这些内容。

您需要编写<cfset var iOccasionDate = ... >或<cfset local.iOccasionDate = ... >或<cfquery name="local.insertGuest" ...>以确保在函数的本地变量范围内创建它们，而不是覆盖其他变量。

你的具体问题是因为你有这个：

<cfquery name="update" datasource="#Application.ds#">

所以你用更新查询覆盖你的更新功能。

一些快速的注意事项：

本地范围适用于CF9及以上版本。如果你在CF8或以下，你需要在你的功能顶部写<cfset var local = StructNew() />来模仿它。
如果您使用Railo或OpenBD，则有一些设置可以将默认行为更改为始终在本地范围内创建（这样可以避免使用var / local范围），但ACF不会有这个选项（还有？）
使用cfqueryparam！ - Simon已经提到了这一点，但重要的是重要。您应始终根据数据的使用位置处理数据 - 防止有意和无意的注入攻击 - 以及使用cfqueryparam的cfquery。

在CFC中调用方法失败，但将其创建为对象工作正常

2 个答案: