0

我的查询是完美的（我已经在 SQL Server Management Studio 中验证了它）。我的代码很完美，但我仍然收到此语法错误：

'=' 附近的语法不正确。说明：执行当前 Web 请求期间发生未处理的异常。请查看堆栈跟踪以获取有关错误及其源自代码的位置的更多信息。异常详细信息：System.Data.SqlClient.SqlException：“=”附近的语法不正确。

public partial class Temporaryche : System.Web.UI.Page
{
    protected void Page_Load(object sender, EventArgs e)
    {
        ddlTDept.Items.Clear();
        ddlTBranch.Items.Clear();

        string connectionString = GlobalVariables.databasePath;
        SqlConnection sqlCon = new SqlConnection(connectionString);
        string query = "select fac.fac_name, dp.dp_name, br.br_name from STUDENT s, DIVISON dv, BRANCH br, DEPT dp, FACULTY fac, CLASS cls, DEGREE dg where dg.dg_id = cls.dg_id and cls.cls_id = s.cls_id and fac.fac_id = dp.fac_id and dp.dp_id = br.dp_id and br.br_id = dv.br_id and s.dv_id = dv.dv_id and s.prn_no = " + txtSearch.Text;

        sqlCon.Open();
        SqlCommand cmd = new SqlCommand(query, sqlCon);
        SqlDataReader reader = cmd.ExecuteReader();

        string facultyName = reader.GetValue(0).ToString();
        string deptName = reader.GetValue(1).ToString();
        string branchName = reader.GetValue(2).ToString();

        ddlTFaculty.SelectedValue = facultyName;

        query = "select dp_name from DEPT where fac_id=(select fac_id where fac_name='" + facultyName + "')";
        cmd = new SqlCommand(query, sqlCon);
        reader = cmd.ExecuteReader();
        ddlTDept.Items.Clear();

        while (reader.Read())
        {
            ddlTDept.Items.Add(reader.GetValue(0).ToString());
        }

        ddlTDept.SelectedValue = deptName;
        sqlCon.Close();
    }
}
4

2 回答 2

4

您的代码中没有任何 ExecuteNonQuery，因此唯一可能发生语法错误的地方就是这一行

query = @"select dp_name from DEPT 
         where fac_id=(select fac_id where fac_name='" + facultyName + "')";

在这一行中，您错过了子查询中的 FROM，因此您应该写为

query = @"select dp_name 
          from DEPT 
          where fac_id= (select fac_id 
                         FROM DEPT where fac_name= '" + facultyName + "')";

当然，这只是解决了您眼前的问题，但正如其他人所说，您应该立即开始使用参数化查询。

例如

query = @"SELECT dp_name FROM DEPT 
          wHERE fac_id = (SELECT TOP 1 fac_id 
                          FROM DEPT 
                          WHERE fac_name=@faculty";
cmd = new SqlCommand(query, sqlCon);
cmd.Parameters.Add("@faculty", SqlDbType.NVarChar).Value = facultyName;
....

编辑
我应该重写我的第一个陈述。还有一点可能会发生语法错误。如果txtSearch.Text为空，则位于第一行。
在这种情况下，查询仍然不完整并触发语法错误。
这可能是因为Page_Load在文本框中仍然没有任何内容的事件中调用了代码。所以可能我们应该添加一些东西来防止整个代码块的执行如果txtSearch为空

protected void Page_Load(object sender, EventArgs e)
{
    if(!string.IsNullOrWhiteSpace(txtSearch.Text))
    {
        .... code that executes the queries ...
    }
}

总而言之，这段代码不太适合Page_Load事件，也许当用户要求执行搜索时，这段代码应该移动到按钮点击事件中......

最后，正如评论中提到的，您应该使用 JOIN 语法来更好地将 WHERE 条件与表之间的关系分开。查询可以写成

SELECT  fac.fac_name,
        dp.dp_name,
        br.br_name 
FROM    STUDENT s INNER JOIN DIVISION dv ON s.dv_id = dv.dv_id
        INNER JOIN BRANCH br ON dv.br_id = br.br_id
        INNER JOIN DEPT dp ON br.dp_id = dp.dp_id
        INNER JOIN FACULTY fac ON dp.fac_id = fac.fac_id
        INNER JOIN CLASS cls ON s.cls_id = cls.cls_id
        INNER JOIN DEGREE dg ON cls.dg_id = dg.dg_id
WHERE   s.prn_no=@search";
于 2015-10-24T20:36:07.420 回答
1

您没有提到错误发生的那一行，很难 100% 确定，还缺少一些其他信息。但是，在第一个 SQL 文本中，最后有以下内容：

s.prn_no=" + txtSearch.Text;

因为那是文本，所以应该用引号 ''

s.prn_no='" + txtSearch.Text +"'"

也就是说，由于SQL Injection ，您永远不应该构建这样的 SQl 查询。相反，您应该使用Parameter 对象。

于 2015-10-24T20:33:36.803 回答