程序在IF / Else SQL语句期间挂起

时间:2013-09-12 09:52:28

标签: c# if-statement

我正在创建一个注册应用程序,它有两种工作方式:在线和离线。我们使用条形码将自己扫描到应用程序中。当您扫描在线条形码时,它将为您提供6个数字和一个“L”,例如242565L,程序将选择此项并在SQL数据库中搜索相应的ID号。并且离线条形码保存带有“”的名称文本值,例如史密斯,约翰

当我运行它并扫描在线条形码时,它工作正常。 但是,当我从互联网断开连接,因此它没有找到数据库,并扫描我的代码,它收起1个字母并冻结,只是挂起,直到我停止进程。

请问任何想法? C#的新手

这是主要代码:

            {
            SqlConnection DBConnection = new SqlConnection("DataSource=DATABASE;Initial Catalog=imis;Integrated Security=True");
            SqlCommand cmd = new SqlCommand();

            Object returnValue;

            string txtend = textBox1.Text;
            try
            {
                DBConnection.Open();
            }
            catch
            {
                DBConnection.Close();
            }
            if (DBConnection.State == ConnectionState.Open)
            {
                if (textBox1.Text.Length != 7) return;
                cmd.CommandText = "SELECT last_name +', '+ first_name +'\t ('+major_key+')\t' from name where id ='" + textBox1.Text.Replace(@"L", "") + "'";
                // MessageBox.Show(textBox1.Text.Replace(@"L", ""));
                cmd.CommandType = CommandType.Text;
                cmd.Connection = DBConnection;
                // sqlConnection1.Open();
                returnValue = cmd.ExecuteScalar() + "\t (" + textBox1.Text.Replace(@"L", "") + ")";
                DBConnection.Close();


                if (listBox1.Items.Contains(returnValue))
                {

                    for (int n = listBox1.Items.Count - 1; n >= 0; --n)
                    {
                        string removelistitem = returnValue.ToString();
                        if (listBox1.Items[n].ToString().Contains(removelistitem))
                        {
                            // listBox1.Items.RemoveAt(n);
                        }

                    }
                }
                else

                    listBox1.Items.Add(returnValue);

                textBox1.Text = null;

                if (listBox1.Items.Count != 0) { DisableCloseButton(); }
                else
                {

                    EnableCloseButton();
                }
                label6.Text = "Currently " + listBox1.Items.Count.ToString() + " in attendance.";
            }

            else
            {
                try
                {
                    string lastTwoChars = txtend.Substring(txtend.Length - 1);

                    returnValue = textBox1.Text.Replace(@"*", "");

                    if (lastTwoChars != "*") return;
                    {
                        if (listBox1.Items.Contains(returnValue))
                        {
                            for (int n = listBox1.Items.Count - 1; n >= 0; --n)
                            {
                                string removelistitem = returnValue.ToString();
                                if (listBox1.Items[n].ToString().Contains(removelistitem))
                                {
                                    //listBox1.Items.RemoveAt(n);
                                }
                            }
                        }
                        else
                            listBox1.Items.Add(returnValue);
                        textBox1.Text = null;
                        if (listBox1.Items.Count != 0) { DisableCloseButton(); }
                        else
                        {
                            EnableCloseButton();
                        }
                        label6.Text = "Currently " + listBox1.Items.Count.ToString() + " in attendance.";
                    }
                }
                catch
                {
                }
            }

    }

2 个答案:

答案 0 :(得分:2)

尝试调试它。挂起程序时暂停程序并查看程序挂起的位置。如果它挂起在SQL查询上,你可能想对它进行超时。

你也不应该像这样创建

cmd.CommandText = "SELECT last_name +', '+ first_name +'\t ('+major_key+')\t' from name where id ='" + textBox1.Text.Replace(@"L", "") + "'";

它可以进行sql注射。

使用parameterized sql或其他形式的保护:

SqlCommand command = new SqlCommand("SELECT last_name +', '+ first_name +'\t ('+major_key+')\t' from name where id =@Name"

command.Parameters.Add(new SqlParameter("Name", textBox1.Text.Replace(@"L", ""))); 

修改

如果您想在连接上设置超时,可以在MSDN处查看: 您可以在连接字符串中设置超时参数,或者只是

DBConnection.ConnectionTimeout = yourTime;

会在等待超时参数后使连接失败。

答案 1 :(得分:0)

不使用try..catch来确定您是否有活动数据库连接,而是使用“使用”块。

它更高效,并且在不再需要时会自动关闭/销毁数据库连接。

通过这种方式,您可以在using语句中移动try catch块并捕获与代码相关的错误,而无需担心是否有开放数据库连接或Internet连接状态。

在这个例子中,我从我自己的代码库中获取了代码,并且我正在使用oracle db连接。你需要用'Oracle'代替'SQL'这个词 注意参数声明在db之间略有不同。 Oracle使用“:”,MsSql使用“@”指定参数。

例如

  DataSet myDataSet= new DataSet();

  using (OracleConnection conn = new OracleConnection(connectionString))
   {
     using (OracleCommand cmd = new OracleCommand())
       {
        cmd.BindByName = true;

        #region SQL Core
           StringBuilder sql = new StringBuilder();
           sql.Append(" <Your SQL Here> ");
        #endregion

        #region - ADD SQLParameter
          // use Parameters to avoid SQL Injection like @param1
          sql.Append(" AND (columnname = @param1 ");
          OracleParameter param1 = new OracleParameter("@param1", OracleDbType.Varchar2); 
          param1.Value = yourValue;

          cmd.Parameters.Add(param1);
        #endregion

        cmd.CommandText = sql.ToString();

        cmd.Connection = conn;

        try
          {
             conn.Open();
             try
             {
               OracleDataAdapter adapter = new OracleDataAdapter(cmd);
               adapter.Fill(myDataSet, "myDataSetName");
             }
             catch (Exception myex)
             {
               //Handle General Exception
                Console.Write(myex);
             }
          }
          Catch (OracleException myex)
          {
            //Handle Oracle Exception
              Console.Write(myex);
          }
       }//end of inner 'using'
    }//end of outer 'using'

//Now you have a populated Data Set, you can loop over it extracting values.
//This acts like an offline record set
//Or you can Bind MyDataSet to a GridView to see the data in your offline table.

   //Output Result set in expected XML format and return
   DataTable dataTable = myDataSet.Tables[0];

   if (dataTable.Rows.Count > 0)
     {
     for (int i = 0; i < dataTable.Rows.Count; i++)
      {
       DataRow dataRow = dataTable.Rows[i];
       for (int x = 0; x < dataTable.Columns.Count; x++)
       {
       //dataTable.Rows[i].Table.Columns[x].ColumnName.ToString()
       if (dataTable.Rows[i].Table.Columns[x].ColumnName == "YourColumnName") 
       { 
         //Get value with dataRow[x]
       }
      }//end 2nd for
     }//end 1st for
    }//end outer if.

如果有任何语法错误,请道歉,这都是在没有intellisense的情况下编辑的。 它只是为了给你一个要点。就是这样。

通过使用离线数据集,您可以减少应用程序挂起。