新手小白请教大神！自己研究做登录页面，不但登录按钮的cs代码报错而且数据库还连不上，不知咋改！求大神指导

[憨憨少年]

using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using System.Web.UI;
using System.Web.UI.WebControls;
using System.Data;
using System.Data.SqlClient;
using System.Data.OleDb;

public partial class Admin_AdminLogin : System.Web.UI.Page
{
    protected void Page_Load(object sender, EventArgs e)
    {

    }


    protected void AdminButton1_Click(object sender, EventArgs e)
    {
        string us = TextBox1.Text;
        string pw = TextBox2.Text;


        SqlConnection conn = new SqlConnection();
        conn.ConnectionString= "Data Source=.;Initial Catalog=ASP.NET_project;Persist Security Info=True;User ID=sa;Password=123456";
        conn.Open();

        using (OleDbConnection cmd = new OleDbConnection())
        {
            cmd.Connection = conn;
            string sql = "SELECT   ASP.NET_project  FROM AdminTable WHERE username" + TextBox1 + "AND password" + TextBox2 + "'";
            cmd.CommandText = sql;



            if (Convert.ToInt32(cmd.Executescalar()) > 0)
            { Label1.Text = "登录成功"; }
            else
            { Label1.Text = "登录失败"; }
           }
        }
    }

[E去不回]

定义的string 类型的us,pw都没用到，textbox1是控件名称，相当于类，所以sql里有问题或者直接用textbox1.text

[E去不回]

图片看不了，不知道为什么

[E去不回]

还有为什么用sql server，又用oracle的连接字符串，一个就够了

[憨憨少年]

引用 3 楼 吃也没关系 的回复:

还有为什么用sql server，又用oracle的连接字符串，一个就够了

谢谢大佬！已修正了！！！

[一个低调的码农]

将TextBox1改成us TextBox2改成pw
并且最好是对us和pw进行非空判断等有效性校验。

[qq_40778501]

OleDbConnection 改成SqlCommand sql这样写 string sql = "SELECT count(1) FROM AdminTable WHERE username = @username AND password = @password"; cmd.CommandText = sql; cmd.Parameters.Add(new SqlParameter("@username,us")); cmd.Parameters.Add(new SqlParameter("@password,pw")); 不要直接把Textbox.text拼到字符串里面，多的也不说，你自己想一下我如果在密码这栏里输入的是hacker or 1=1会怎么样

[qq_40778501]

引用 6 楼 qq_40778501 的回复:

OleDbConnection 改成SqlCommand sql这样写 string sql = "SELECT count(1) FROM AdminTable WHERE username = @username AND password = @password"; cmd.CommandText = sql; cmd.Parameters.Add(new SqlParameter("@username,us")); cmd.Parameters.Add(new SqlParameter("@password,pw")); 不要直接把Textbox.text拼到字符串里面，多的也不说，你自己想一下我如果在密码这栏里输入的是hacker or 1=1会怎么样

cmd.Parameters.Add(new SqlParameter("@username",us)); cmd.Parameters.Add(new SqlParameter("@password",pw)); 手打的没在vs编辑器里打，更正