我是 C# 的初学者，我遇到了变量范围的问题。所以...这是我的代码，希望有人能帮助我找到一种方法，使我的变量“s”在我的开关/案例中都可以访问，然后在底部附近使用“SqlCommand cmd = new SqlCommand(s, conn) ;"陈述。如果那不可能，我想就如何重组此代码和/或添加什么来获得我在 switch/case 语句中提出的“s”提出建议 - 将该值降低到 SqlCommand 语句. 我应该使用 session 变量还是类？如果是这样，如何/在哪里？ 哦，你会说我不需要这个:“使用(StreamReader sr = new StreamReader(Request.InputStream，Encoding.UTF8))” 谢谢!
这是我的 ASPX 页面中的代码:
<%@ Page Language="C#" %>
<%@ Import Namespace="System.Collections.Generic" %>
<%@ Import Namespace="System.Data" %>
<%@ Import Namespace="System.Data.SqlClient" %>
<%@ Import Namespace="System.IO" %>
<%@ Import Namespace="System.Web.Script.Serialization" %>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<script runat="server">
protected void Page_Load(object sender, EventArgs ec)
{
//level 01
using (SqlConnection conn = new SqlConnection("Database=db; User Id=user; password=pw"))
{
//level 02
using (StreamReader sr = new StreamReader(Request.InputStream, Encoding.UTF8))
{
//level 03
Response.ContentType = "text/plain";
string s;
string u = Request.QueryString["u"];
if (u == "specificstring")
{
//level 04
string sCmd = Request.QueryString["sCmd"];
string IDCategory;
string sCategory;
string sDefaultEventSort;
//string s;
switch (sCmd)
{
case "GetCategoryNames":
s = "SELECT";
//s += " Id";
s += " name";
//s += ", defaultSort";
s += " FROM Category";
s += " WHERE";
s += " (";
s += " ShowOnHomePage=1";
s += " AND IncludeInTopMenu=1";
s += " AND Published=1";
s += " )";
s += " ORDER BY name";
//s = "SELECT name FROM Category ORDER BY name";
break;
case "GetCategoryId":
sCategory = Request.QueryString["sCategory"];
s = "SELECT";
s += " Id";
s += " FROM Category";
s += " WHERE";
s += " (";
s += " [name]='" + sCategory + "'";
s += " )";
break;
}
try
{
SqlCommand cmd = new SqlCommand(s, conn);
conn.Open();
SqlDataReader rdr = cmd.ExecuteReader(CommandBehavior.CloseConnection);
List<Dictionary<string, object>> list = new List<Dictionary<string, object>>();
while (rdr.Read())
{
Dictionary<string, object> d = new Dictionary<string, object>(rdr.FieldCount);
for (int i = 0; i < rdr.FieldCount; i++)
{
d[rdr.GetName(i)] = rdr.GetValue(i);
}
list.Add(d);
}
JavaScriptSerializer j = new JavaScriptSerializer();
Response.Write(j.Serialize(list.ToArray()));
}
catch (Exception e)
{
Response.TrySkipIisCustomErrors = true;
Response.StatusCode = 500;
Response.Write("Error occurred. Query=" + s + "\n");
Response.Write(e.ToString());
}
//level 04
}
Response.End();
//level 03
}
//level 02
}
//level 01
}
</script>最佳答案
您需要先初始化 ，因为您的 switch 语句不能保证初始化 ；
string s = string.empty;
或
string s = null;
或者您可以在 switch 语句中添加一个 default case 以确保初始化 s:
default:
s = string.Empty; //or some default sql query
break;
更好的结构如下:
将 try/catch 从第二级 using 语句中取出，并将 using 语句包装在 try/catch 中 block ，换句话说，使您的 try/catch 成为父级别。这并没有错，try/catch block 现在在哪里，只是您没有专门使用它来捕获 SqlCommand 本身的异常，您捕获任何 Exception 并在您的 HTTP 响应中返回其错误消息。因此，让 try/catch 成为包装您的 using 语句的最外层 block 。
try
{
StringBuilder sb = new StringBuilder();
//use sb to build the SQL string query
using (SqlConnection conn = CreateSqlConnection(connString))
{
using (SqlCommand command = CreateSqlCommand(sb.ToString(), conn)
{
//open connection + execute command
}
}
{
catch(Exception ex)
{
}
为什么使用 StringBuilder 而不是连接字符串变量？ 因为 C# 字符串是不可变的，这意味着每次连接到字符串时，C# 都必须创建该字符串的副本。 StringBuilder 是一个快速、可变的字符串对象，可以更快地构建您的 sql 字符串。
您不需要StreamReader，您甚至不需要在任何地方使用sr 并将您的代码包装在StreamReader using 语句对 using 语句中的代码没有任何作用。
请注意@LukeBriggs 在他的回答中提到的安全问题，如果您使用存储过程来传递参数或 Entity Framework ，您可以减轻 Sql 注入(inject)攻击，但我会说，因为您只是在学习，安全方面可能有点不知所措，并且在实际构建人们会使用的东西之前您不必担心安全问题，您的软件必须“受到关注”以担心安全漏洞是您现在可以查看的一种方式，这样您就可以继续学习而不会不知所措。
关于c# - C# 中的变量作用域 - 使用 session 还是类？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/40822931/