更新分数以覆盖以前的分数

Question

我有一个页面，列出了诗歌标题、平均分数，并列出了对诗歌进行评分的评委。这些项目分为 3 列。当评委点击诗歌标题时，他会进入诗歌概览页面，该页面显示诗歌内容，并允许评委对这首诗进行从 1 到 3 的评分。如果评委已经对这首诗打分，则他的分数会加载到诗歌中下拉列表中会出现一条消息，表明他已经得分，如果他更改分数，则会更新他的分数。我的更新查询不起作用，但我想知道这是否是由于我的逻辑问题造成的。

有问题的完整代码在这里。更新查询从第 86 行开始。我的想法是是否已经有分数，arrDuplicate 数组将显示一条消息，说明并加载下拉列表中的上一个分数。如果需要，裁判可以更改分数。然而，按照现在的情况，分数每次都会写入新记录，而不是更新现有分数。

PoemScores表由poemID、judgeID和score组成。没有pk。应该有吗？

Answer 1

您需要添加

AND judgeID = ?

到您的查询中，否则它将尝试更新该特定诗歌的每个分数。

Answer 2

将cfqueryparam添加到duplicateCheck、updateScore和qRatePoem查询中，主要是为了避免SQL注入。

您可以将其

<cfif duplicateCheck.recordCount GT '0'>

（字符串比较）更改为此（数字比较）

<cfif duplicateCheck.recordCount GT 0>

，甚至只是此（布尔值）。

<cfif duplicateCheck.recordCount>

这两个 IF 语句是相同的，因此如果没有错误，您将同时执行更新和插入。你需要重新思考你的逻辑。

<cfif NOT ArrayLen( arrErrors ) >
        <cfquery name="updateScore" datasource="#request.dsn#">
                UPDATE PoemScores
                SET score = '#form.rating#'
                WHERE poemID = <cfqueryparam cfsqltype="cf_sql_varchar" value="#url.id#">
        </cfquery>
</cfif>


<cfif NOT ArrayLen( arrErrors )>
        <!--- INSERT DATABASE QUERY--->
        <cfquery name="qRatePoem" datasource="#request.dsn#">
                INSERT INTO PoemScores (poemID, score, judgeID)
                VALUES ('#url.id#', '#form.rating#', '#session.username#')
        </cfquery>

        <!---<cflocation url="dash.cfm" addtoken="no">--->
</cfif>

您可能想在此处添加额外的 IF 语句，检查 arrDuplicates 是否也有长度。如果是，则他们之前已对此进行评分，请执行更新查询。否则进行插入。

<cfif NOT ArrayLen( arrErrors ) >
   <cfif ArrayLen(arrDuplicate)> <!--- already submitted a score for this poem --->
        <cfquery name="updateScore" datasource="#request.dsn#">
                UPDATE PoemScores
                SET score = '#form.rating#'
                WHERE poemID = <cfqueryparam cfsqltype="cf_sql_varchar" value="#url.id#">
        </cfquery>
   <cfelse>
        <!--- INSERT DATABASE QUERY--->
        <cfquery name="qRatePoem" datasource="#request.dsn#">
                INSERT INTO PoemScores (poemID, score, judgeID)
                VALUES ('#url.id#', '#form.rating#', '#session.username#')
        </cfquery>
   </cfif>
        <!---<cflocation url="dash.cfm" addtoken="no">--->
</cfif>