cfc 中的函数局部变量

Question

在我被要求研究 Coldfusion 应用程序中一些奇怪的间歇性错误之前，我没有使用过 Coldfusion。

在阅读了有关范围的信息后，我认为问题是因为我的 cfc 函数中的所有变量都没有使用 var 关键字，并且在各种函数中使用了相同的变量名。因此，据我了解，变量的范围是页面级别，调用这些函数的不同线程将覆盖导致“奇怪”问题的变量。

我的问题是这样做的正确方法是什么？

 <cfset var listCount = 0>
 <cfquery name="qGetElementsByType" dbtype="query" maxrows="#arguments.num_to_return#">
    SELECT elementId,
           title, PIhtml, Rerhtml,
           text, url, image, Rank, isPoll, pollId, subjectId
    FROM   arguments.element_query
    WHERE  <cfloop list="#arguments.element_type_id#" index="lcv">
               <cfif listCount GT 0>
                  OR
               </cfif>
               subjectid =  #lcv#
              <cfset listCount = listCount + 1>
           </cfloop>
</cfquery>

是否需要在每次设置 listCount 变量时添加var，还是仅在初始声明时添加？

Answer 1

（我希望这个答案不会太啰嗦。我认为现有答案没有提供足够的信息，但希望没有走得太远……）

在 CF 中，有各种各样的作用域可以放置变量（应用程序、会话、url、cgi 等）。

其中一些需要使用显式声明（例如，会话变量必须始终具有范围），其他可以在读取变量时自动访问（例如，可以使用无范围变量读取表单和 url 变量） - 有一个这里的优先顺序决定了在哪些范围内检查无范围的变量。

此排序中的底部范围是variables 范围，这是适用于整个当前页面/对象实例的范围。

设置新变量时，如果没有作用域，则在variables 作用域中创建。由于这是一个全局范围，因此可以从同一函数的不同实例以及不同的函数访问它，这会导致您意识到的问题。

为了防止变量进入全局变量作用域，您必须将它放在函数的local 作用域中。 （从技术上讲，您可以将其放在函数的 arguments 范围内，但这可能会使人们感到困惑。）

在早期版本的 CF 中，无法显式访问本地范围 - 您需要使用 var 关键字才能在本地范围内创建变量 - 一旦创建，它将始终优先（两者用于读取和写入）在变量范围内。

在 CF9 中，local 范围现在是“正确”范围并且可以显式访问，因此您可以编写 <cfset local.x = 0 /> 而不是使用 <cfset var x = 0 /> - 这样做的主要好处是当您创建变量时不能使用 var 关键字的地方，例如<cfquery name="local.qGetElementsByType" ...> 和 <cfloop index="local.lcv"...>

您仍然只需要在第一次创建每个变量时应用本地范围，以防止它进入变量范围 - 如果您愿意，后续读取/更新可以是无范围的，就像它们一样在执行 var 范围时。
（尽管无范围变量还有其他潜在的范围相关问题，例如在 <cfloop query="queryname"> 块内，因此有些人会争辩说您应该始终对所有变量进行范围无论如何。）

总而言之，要使您显示的代码安全，您需要范围：

• qGetElementsByType 来自 cfquery 标签
• lvc 来自 cfloop 标签

由于这些变量不是用 cfset 创建的，因此最容易通过在名称前加上 local. 前缀来确定范围

由于您已经 var 限定了 listCount 变量的范围，因此您无需在同一个函数中再次执行此操作 - 您可以选择使用 <cfset local.listCount = local.listCount + 1> （或者实际上是 <cfset local.listCount++ > ）但这又是一个问题首选项，不需要防止泄漏到变量范围内。

（旁注：理想情况下，您应该使用 #lcv# 周围的 cfqueryparam 标记来防止 SQL 注入 - 即使这是一个查询查询，这可能仍然是一个问题，并且在安全性上始终保持安全总是更好.)

当然，这只是这个功能 - 您还需要修复其他功能 - 一个简单的方法是使用 varscoper 工具扫描您的整个代码库并识别需要作用域的变量。

Answer 2

除了 Evik 在他的回答中所说的之外，你应该 VAR all 你的函数局部变量。 qGetElementsByType 和 lcv 也应该是 VARed。如果您使用的是 CF9 及更高版本，您可以简单地使用 LOCAL 范围来确定它们的范围，例如：local.qGetElementsByType 等。

Answer 3

我假设您的问题是专门针对这一行的：

<cfset listCount = listCount + 1>

不，您不需要在此行中再次使用 var。

但是，如果稍后在页面中，您尝试使用访问名为 listCount 的变量，它已经具有您刚刚运行的代码中的值，因此您需要重新创建它。