C# WinForm更新服务型数据库出现命名空间及变量不存在错误求助

报错1：A namespace cannot directly contain members such as fields or methods #

• 触发原因：你的UpdateService方法被放在了Technician类的外部，直接归属在ASSIGNMENT命名空间下。C#要求命名空间下只能直接定义类、结构体、枚举等类型，不能直接放字段、方法这类成员。你在Technician类定义完后多写了一个闭合大括号}，直接把UpdateService方法隔到了类外面。

报错2：the name "status", "description", "collect", "requestid" does not exist in the current context #

• 触发原因1：UpdateService不在Technician类内部，自然无法访问类里定义的私有字段status/description/collect/requestid
• 触发原因2：你没有为Technician类定义接收3个参数的构造函数，但按钮点击事件里直接传了3个参数实例化Technician对象
• 触发原因3：requestid字段没有对外公开的赋值入口，更新SQL语句中用到了requestid但你全程没有给它赋值，就算方法挪到类里也会因为值为空导致更新失败

其他潜在问题 #

• UpdateService方法定义要求传入3个参数，但你在按钮点击事件里调用时没有传任何参数，语法不匹配
• 更新逻辑中调用了两次ExecuteNonQuery，会重复执行两次更新操作，属于冗余代码
• 直接拼接SQL语句存在SQL注入风险，建议用你注释里写的参数化查询方案
• 目前你只取了MonthCalendar的Day属性（仅当月第几天的整数），存到数据库的Collection_Date字段里会丢失年月信息，建议存储完整日期

修复后的代码 #

1. Technician类代码 #

using System;
using System.Text;
using System.Data.SqlClient;
using System.Data;
using System.Configuration;
using System.Windows.Forms;

namespace ASSIGNMENT
{
    public class Technician
    {
        private string username;
        private string password;
        private string role;
        private int requestid;
        private string description;
        private string status;
        private int collect;

        // 新增RequestId属性，用于赋值要更新的服务请求ID
        public int RequestId { get => requestid; set => requestid = value; }
        public string Description { get => description; set => description = value; }
        public string Status { get => status; set => status = value; }
        public int Collect { get => collect; set => collect = value; }

        // 新增构造函数，接收你实例化时传入的三个参数
        public Technician(int collectDate, string desc, string status)
        {
            Collect = collectDate;
            Description = desc;
            Status = status;
        }

        // 把UpdateService方法挪到类内部，调整为无参数方法直接用类的属性值
        public string UpdateService()
        {
            string UpdateStatus;
            // 加using自动释放连接资源，不用手动关
            using(SqlConnection con = new SqlConnection(ConfigurationManager.ConnectionStrings["myCS"].ToString()))
            {
                con.Open();
                // 改用参数化查询，避免注入风险
                SqlCommand cmd = new SqlCommand("update RequestHistory set Collection_Date = @dateCollect, Description = @description, Service_Status= @status where ServiceID = @requestid", con);
                cmd.Parameters.AddWithValue("@dateCollect", Collect);
                cmd.Parameters.AddWithValue("@description", Description);
                cmd.Parameters.AddWithValue("@status", Status);
                cmd.Parameters.AddWithValue("@requestid", RequestId); // 这里要确保你提前给RequestId赋了要更新的服务ID

                int i = cmd.ExecuteNonQuery();
                UpdateStatus = i != 0 ? "Request Updated!" : "Unable to Update.";
            }
            return UpdateStatus;
        }
    }
}

2. 更新按钮点击事件代码 #

private void btnUpdate_Click(object sender, EventArgs e)
{
    // 这里要注意，你需要先获取要更新的服务请求ID，赋值给objUpdate.RequestId
    // 示例：如果你界面上有个存服务ID的文本框txtRequestId，就加一行 objUpdate.RequestId = int.Parse(txtRequestId.Text);
    Technician objUpdate = new Technician(mthCalCompletionDate.SelectionEnd.Day, txtboxDescription.Text, cboServiceStatus.SelectedValue.ToString());
    // 记得给objUpdate.RequestId赋值！否则更新会匹配不到数据
    objUpdate.RequestId = 1; // 这里替换成你实际要更新的服务ID的取值逻辑
    MessageBox.Show(objUpdate.UpdateService());
}

内容的提问来源于stack exchange，提问作者user123