将大型表格保存到多个表格，规范化和动态查询

Question

去年，我发布了一个问题here关于提交一个50字段的表单，以及最好的方法来做到这一点。该解决方案仍在使用中，运行良好。然而，为了构建这些动态查询，我最终得到了很多重复的<cfif>，我想知道是否有更好的方法来处理这个问题。虽然代码结束了“杂乱”，但由于这一点，db当然非常干净，并且写入次数也保持在最低限度，但有没有更好的方法来执行以下操作？

<cfif StructKeyExists(arguments.form,"data1") or StructKeyExists(arguments.form,"data2") or StructKeyExists(arguments.form,"data3")> 
    <cfquery> 
    insert into table1 (
     <cfif StructKeyExists(arguments.form,"data1")>data1,</cfif> 
     <cfif StructKeyExists(arguments.form,"data2")>data2,</cfif> 
     <cfif StructKeyExists(arguments.form,"data3")>data3,</cfif> 
     userid 
    ) 
    values (
     <cfif StructKeyExists(arguments.form,"data1")><cfqueryparam cfsqltype="cf_sql_varchar" value="#arguments.form.data1#" maxlength="30">,</cfif> 
     <cfif StructKeyExists(arguments.form,"data2")><cfqueryparam cfsqltype="cf_sql_varchar" value="#arguments.form.data2#" maxlength="10">,</cfif> 
     <cfif StructKeyExists(arguments.form,"data3")><cfqueryparam cfsqltype="cf_sql_varchar" value="#arguments.form.data3#" maxlength="25">,</cfif> 
     <cfqueryparam cfsqltype="cf_sql_smallint" value="#arguments.form.userid#" maxlength="5"> 
    ) 
    on duplicate key update 
     <cfif StructKeyExists(arguments.form,"data1")>data1=values(data1),</cfif> 
     <cfif StructKeyExists(arguments.form,"data2")>data2=values(data2),</cfif> 
     <cfif StructKeyExists(arguments.form,"data3")>data3=values(data3),</cfif> 
     userid=values(userid) 
    </cfquery> 
</cfif> 

这种'感觉'出于某种原因是错误的。难道例如更加聪明，能，而有更多的写操作，每个值分割到自己的更新，就像这样：

<cfif StructKeyExists(arguments.form,"data1")> 
    <cfquery> 
    insert into table1 (data1,userid) 
    values (<cfqueryparam cfsqltype="cf_sql_varchar" value="#arguments.form.data1#" maxlength="30">,<cfqueryparam cfsqltype="cf_sql_smallint" value="#arguments.form.userid#" maxlength="5">) 
    on duplicate key update data1=values(data1),userid=values(userid) 
    </cfquery> 
</cfif> 

等

还是有更好的方式来做到这一点，我完全可以俯瞰？！

Answer 1

如果你知道你有多达50个字段，并且它们的名字都是一致的（data1 ... dataN），我只需要一个循环。例如：

<cfloop index="i" from="1" to="50"> 
    <cfif StructKeyExists(arguments.form, "data#i#")> 
     <cfquery> 
      insert into table1 (data#i#,userid) 
      values (<cfqueryparam cfsqltype="cf_sql_varchar" value="#arguments.form['data' & i]#" maxlength="30">,<cfqueryparam cfsqltype="cf_sql_smallint" value="#arguments.form.userid#" maxlength="5">) 
      on duplicate key update data#i#=values(data#i#),userid=values(userid) 
     </cfquery> 
    </cfif> 
</cfloop> 

Answer 2

是的，还有一种更好的方法来做到这一点，无论如何，我认为。

首先，不是检查是否存在所有50个字段，只是检查表单是否已提交。一旦你完成了，唯一可能不存在的字段是复选框和单选按钮。

接下来，在进入cfquery标签之前先完成所有的条件逻辑。除复选框和单选按钮之外，您不检查是否存在某个值，而是检查有效性。如果需要表单字段，请确保它不是空字符串。另外请确保您具有数字和日期字段的正确数据类型。

在此阶段，您可能需要设置一些用于cfqueryparam的null属性的变量。例如，如果您有一个可选的数字字段并且用户没有填写它，则需要一个等于true的变量，以便数据库接收一个空值。空字符串仅适用于文本字段。

另请注意，数据的质量比代码的外观更重要。尽管可读代码很重要，但您不会在10行或更少的代码中处理50个表单域。

Answer 3

我觉得你这样做的方式很好。

我不会将你的插入/更新分解为每个数据字段的单独查询。当提交很多字段时，这会导致更多的开销。想象一下，所有50个表单域都被提交了，50个查询，yike。

如果需要处理空值，这里有办法把你的语法内做到这一点的例子：

<cfif structKeyExists(arguments.form, "data1"><cfqueryparam value="#arguments.form.data1#" null="#(NOT len(arguments.form.data1))#" cfsqltype="cf_sql_integer">,</cfif>