C# Windows 窗体应用程序更新查询外键 VS2010

Question

我需要帮助的人需要更新我的课程名称。
表1：课程
-CurriculumID (PK, Auto_Increment)
-课程名称
表 2：年级
-GradeLevelID（PK，自动增量）
-等级
-CurriculumID（连接到课程表的外键）

结果：

年级 |课程名称
````````````1级````````````K-12`````````
更新到
`````````````2级``````````K-11

问题是每当我按下更新按钮时，它都不会更新，因为我的更新查询不起作用。这是我的代码。请帮忙~

private void btnGLEdit_Click(object sender, EventArgs e)
        {
            try
            {
                //update
                SqlConnection scon = new SqlConnection();
                scon.ConnectionString = "Data Source=.;Initial Catalog=HSPAEnrollmentSytem;Integrated Security=True";
                //scon.ConnectionString = "Data Source=D630 ;Initial Catalog=NLEDB; User ID=approj1; Password=approj1";
                scon.Open();

                SqlCommand cmd = new SqlCommand("UPDATE GradeLevel set GradeLevel = @GradeLevel, CurriculumID = @CurriculumID where GradeLevelID = '" + txtGradeLevelID.Text + "'", scon);



                //cmd.Parameters.Add("@GradeLevel", SqlDbType.Int).Value = txtGradeLevel.Text.Trim();
                //cmd.Parameters.Add("@CurriculumID", SqlDbType.Int).Value = comboBox1.Text.ToString();
                cmd.Parameters.AddWithValue("@GradeLevel", txtGradeLevel.Text);
                cmd.Parameters.AddWithValue("@CurriculumID", comboBox1.Text);

                cmd.ExecuteNonQuery();
                MessageBox.Show("update");
            }
            catch (SqlException ex)
            {

            }
        } 

Answer 1

也许这个答案看起来更像是一个评论，但它需要文本格式并且会很长，所以我决定把它作为一个答案。希望不会造成任何麻烦

让我看看我能不能让你好起来……

你有下一个架构：

Curricuculum
--------------------------------
CurriculumID ( PK, Auto_Increment )
CurriculumName

GradeLevel
--------------------------------
GradeLevelID( PK, Auto_Increment )
GradeLevel
CurriculumID( Foreign Key )

还有这个数据：

Curriculum
----------------------------------------
CurriculumId | CurriculumName
----------------------------------------
1            | K-12
2            | K-13

GradeLevel
----------------------------------------
GradeLevelId | GradeLevel | CurriculumId
----------------------------------------
1            | Grade 1    | 1

执行此查询：

SELECT g.GradeLevelId, g.GradeLevel, c.CurriculumName
FROM GradeLevel g
INNER JOIN Curriculum c
ON g.CurriculumId = c.CurriculumId

返回这个结果：

------------------------------------------
GradeLevelId | GradeLevel | CurriculumName
------------------------------------------
1            | Grade 1    | K-12

现在您的代码中有此更新：

UPDATE GradeLevel set GradeLevel = @GradeLevel, CurriculumID = @CurriculumID where GradeLevelID = ' txtGradeLevelID.Text '

你想更新GradeLevel，这样你就会得到这样的数据：

GradeLevel
----------------------------------------
GradeLevelId | GradeLevel | CurriculumId
----------------------------------------
1            | Grade 2    | 2

再次运行我之前输入的相同查询现在会返回这个结果：

------------------------------------------
GradeLevelId | GradeLevel | CurriculumName
------------------------------------------
1            | Grade 2    | K-13

现在... 这是您真正想要的吗？您在问题中说过您想更新 CurriculumName，但您从未更新该字段，您的 cmets 和答案告诉我您只想引用Curriculum 中具有CurriculumID = 2 的记录。如果我说的是正确的，那么到目前为止，您的代码看起来应该并且应该这样做。所以我可以告诉你的几件事是：

• 在为查询参数赋值时放置一个断点，并确保它们具有您想要的值（txtGradeLevelID.text = "1"、txtGradeLevel.text = "Grade 2" 和 ComboBox1.Text = "2"）
• 不要将catch 留空；至少放一个消息框告诉你出了什么问题。也许有一个错误，你永远不会注意到它。
• （与您的问题本身无关，但需要考虑）如果您已经在使用参数化查询...为什么还要进行字符串连接？您还可以在 WHERE 条件中使用参数；没有理由混合使用两种方式来运行查询，而且字符串连接会导致 SQL 注入和性能问题。

现在，如果我所说的不是您真正想要实现的，请重新表述您的问题。添加您在 cmets 中编写的任何相关信息，并正确格式化（老实说，我很难相信您的问题没有被搁置，甚至没有收到一个反对票）

Answer 2

您在更新声明中没有引用您的列 Curriculum Name：

SqlCommand cmd = new SqlCommand("UPDATE GradeLevel set GradeLevel = @GradeLevel, CurriculumID = @CurriculumID where GradeLevelID = '" + txtGradeLevelID.Text + "'", scon);

您希望Curriculum Name 更新到什么？您没有提供任何更改。事实上，如果您要更改 Curriculum Name，因为它存在于您的 Curriculum 表中，而不是您的 GradeLevel 表中，那么您正在更新错误的表。

您的更新语句应该看起来更像这样：

UPDATE Curriculum
SET 'Curriculum Name' = @curriculumName
WHERE curriculumId = @curriculumId

我不明白您为什么要根据年级 ID 更新课程名称...这将使用许多潜在的不同值多次更新表中的行。所以最后，你不一定知道最后会叫什么名字。

一旦您的课程名称更新，任何从年级 -> 课程在正确列上的连接都将显示新更新的课程名称。

老实说，我不了解您的架构。如果要同时更新这两个值，则将它们作为两个单独的表似乎不可行。他们两个总是要一起改变吗？他们有1对1的关系吗？如果是这样的话，它们可能属于同一张桌子。要么，要么将您的 PK 放在您的成绩表上，然后将课程表设置为两个表上的主键。这样，您将拥有一对一的关系，而不是一对多的关系。