простой логин C# и веб-приложение MySQL

Question

У меня есть несколько ошибок в моем коде, по какой-то причине, когда я пытаюсь поймать в конце, он вызывает ошибки, заявляя, что отсутствует множество скобок, хотя я не думаю, что это так. Может кто-нибудь, пожалуйста, дайте мне знать, где я поступил неправильно.

Код:

namespace login 
{ 
    public partial class _Default : Page 
    { 
     // decleration of tabels and dataadapters including my connection string for my MySQL databse 
     DataSet ds = new DataSet(); 
     MySqlConnection cs = new MySqlConnection(@"SERVER= ********;username=******;password=******;Allow Zero Datetime=true; Initial Catalog = benoatsc_GreenFilm"); 

     MySqlDataAdapter da = new MySqlDataAdapter(); 
     DataTable dt = new DataTable(); 
     String totalDonations = string.Empty; 

     protected void Button1_Click(object sender, EventArgs e) 
     { 
     try 
     { 
      MySqlCommand SelectCommand = new MySqlCommand("select * from films.user where user_name='" + this.username.Text + "; and password='" + this.password.Text + "';", cs); 
      MySqlDataReader myreader; 
      cs.Open(); 
      myreader = SelectCommand.ExecuteReader(); 

      int count = 0; 
      while (myreader.Read()) 
      { 
       count = count + 1; 
      } 

      if (count == 1) 
      { 
       Response.Write(@"<script language='javascript'>alert('wow your in !!');</script>"); 
      } 

      else if (count > 1) 
      { 
       Response.Write(@"<script language='javascript'>alert('duplicate');</script>"); 
      } 

      else Response.Write(@"<script language='javascript'>alert('wrong password');</script>"); 

      cs.Close(); 
     } 

     catch (Exception ex) 
     { 
      Response.Write(@"<script language='javascript'>alert(ex.message);</script>"); 
     } 
     } 
    } 
} 

Answer 1

Проблема 1: вы открыли дополнительный Керли скобку { после try block.
Проблема 2: Вы открыли параметр user_name с помощью single quotes, но вы не закрыли с single quotes.

Решение 1: Вам необходимо удалить дополнительную скобу для завивки, открытую после блока try.
Решение 2: необходимо приложить user_name параметр с single quotes должным образом.

Предложение: ваш запрос открыт для SQL Injection attacks, я хотел бы предложить использовать parameterised queries, чтобы избежать этого.

Полный код: с помощью parameterised queries

namespace login 
{ 
public partial class _Default : Page 
{ 
    // decleration of tabels and dataadapters including my connection string for my MySQL databse 
    DataSet ds = new DataSet(); 
    MySqlConnection cs = new MySqlConnection(@"SERVER= ********;username=******;password=******;Allow Zero Datetime=true; Initial Catalog = benoatsc_GreenFilm"); 

    MySqlDataAdapter da = new MySqlDataAdapter(); 
    DataTable dt = new DataTable(); 
    String totalDonations = string.Empty; 

    protected void Page_Load(object sender, EventArgs e) 
    { 

    } 

    protected void Button1_Click(object sender, EventArgs e) 
    { 
     try 
     { 

       MySqlCommand SelectCommand = new MySqlCommand("select * from films.user where user_name=@username and password=@password;", cs); 
       MySqlDataReader myreader; 
       SelectCommand.Parameters.AddWithValue("@username",this.username.Text); 
       SelectCommand.Parameters.AddWithValue("@password",this.password.Text); 
       cs.Open(); 

       myreader = SelectCommand.ExecuteReader(); 

       int count = 0; 
       while (myreader.Read()) 
       { 
        count = count + 1; 
       } 

       if (count == 1) 
       { 
        Response.Write(@"<script language='javascript'>alert('wow your in !!');</script>"); 
       } 

       else if (count > 1) 
       { 
        Response.Write(@"<script language='javascript'>alert('duplicate');</script>"); 
       } 

       else Response.Write(@"<script language='javascript'>alert('wrong password');</script>"); 

       cs.Close(); 
      } 

      catch (Exception ex) 
       { 
       Response.Write(@"<script language='javascript'>alert(ex.message);</script>"); 
       }//end of catch block 

     }//end of try block 
    }//end of class 
}//end of namespace 

Answer 2

Помимо отсутствующих скобок и неправильного SQL запроса (содержит точку с запятой) вы можете улучшить свой код много. Вы можете использовать ExecuteScalar и изменить ваш запрос на COUNT(*). Таким образом, вам не нужно учитывать код. Также используйте инструкцию using, которая обеспечит закрытие соединения даже в случае исключения. Так что ваш код должен быть на следующей linesL

namespace login 
{ 
    public partial class _Default : Page 
    { 
     // decleration of tabels and dataadapters including my connection string for my MySQL databse 
     DataSet ds = new DataSet(); 
     MySqlConnection cs = new MySqlConnection(@"SERVER= ********;username=******;password=******;Allow Zero Datetime=true; Initial Catalog = benoatsc_GreenFilm"); 

     MySqlDataAdapter da = new MySqlDataAdapter(); 
     DataTable dt = new DataTable(); 
     String totalDonations = string.Empty; 

     protected void Page_Load(object sender, EventArgs e) 
     { 

     } 

     protected void Button1_Click(object sender, EventArgs e) 
     { 
      try 
      { 
       using (MySqlCommand SelectCommand = new MySqlCommand("select count(*) from films.user where user_name=@username AND password = @password", cs)) 
       { 
        SelectCommand.Parameters.AddWithValue("@username", username.Text); 
        SelectCommand.Parameters.AddWithValue("@password", password.Text); 
        cs.Open(); 
        int count = (int)SelectCommand.ExecuteScalar(); 
        if (count == 1) 
        { 
         Response.Write(@"<script language='javascript'>alert('wow your in !!');</script>"); 
        } 
        else if (count > 1) 
        { 
         Response.Write(@"<script language='javascript'>alert('duplicate');</script>"); 
        } 

        else Response.Write(@"<script language='javascript'>alert('wrong password');</script>"); 
       } 
      } 

      catch (Exception ex) 
      { 
       Response.Write(@"<script language='javascript'>alert(ex.message);</script>"); 
      } 

     } 
    } 
} 

Использование параметров с помощью команды избавит вас от SQL Injection