使用实体框架注册

本文关键字:注册 框架 实体 | 更新日期: 2023-09-27 18:06:36

所以我尝试使用实体框架和c#创建一个注册方法,这是方法:

//Button create new account
    private void BtnSignUp_Click(object sender, EventArgs e)
    {
        IEnumerable<DriverAcount> list = from a in context.DriverAcounts select a;
        foreach (var Ac in list)
        {
                if (TxBoxNewUserName.Text != Ac.Login)
                {
                    if (TxtBoxPASS1.Text == TxBoxPass.Text)
                    {
                        Ac.Login = TxBoxNewUserName.Text;
                        Ac.Password = TxtBoxPASS1.Text;
                        context.DriverAcounts.Add(Ac);
                        MessageBox.Show("the account is create succefuly");
                        TxBoxNewUserName.Text = "";
                        TxtBoxPASS1.Text = "";
                        TxBoxPass.Text = "";
                        break;
                    }
                    else
                    {
                        MessageBox.Show("the two passwords didn't matched");
                    }
                    TxBoxNewUserName.Text = "";
                    TxtBoxPASS1.Text = "";
                    TxBoxPass.Text = "";
                    continue;
                }
            else
            {
                MessageBox.Show("this username is already exist, please choose another one");
                TxBoxNewUserName.Text = "";
                TxtBoxPASS1.Text = "";
                TxBoxPass.Text = "";
                break;
            }
        }
        context.SaveChanges();
    }

问题是,当我想添加一个新用户通常应该看看它是否存在于数据库中,但它没有做得对,例如,如果我们有两个名字在DB name1name2和进入文本框我们有name2它会添加name2在DB即使它已经存在。所以,如果有人有一个想法,我会非常感激。

使用实体框架注册

您需要再次查看您的代码并确切地了解它在做什么-您是否附加了调试器并逐步通过它?如果检索到的第一个名称与文本框中的名称不匹配,则总是在foreach循环的第一次迭代时创建驱动程序名称。其他的都将被忽略。

你可以试试这个。当你只想检查一条记录是否已经存在时,你不需要查询所有的记录(就像你现在所做的那样)。

private void BtnSignUp_Click(object sender, EventArgs e)
{
    // This performs a case sensitive match on the login name, you'll need to change it if you want to ignore case
    DriverAcount existingAccount = context.DriverAcounts.FirstOrDefault(d => d.Login == TxBoxNewUserName.Text);
    if (existingAccount != null)
    {
            MessageBox.Show("This username already exists, please choose another one.");
    }
    else
    {
            if (TxtBoxPASS1.Text == TxBoxPass.Text)
            {
                    Ac.Login = TxBoxNewUserName.Text;
                    Ac.Password = TxtBoxPASS1.Text;
                    context.DriverAcounts.Add(Ac);
                    // Only need to call this if you've made changes, so I've moved it here
                    context.SaveChanges();
                    MessageBox.Show("The account was created successfully");
            }
            else
            {
                    MessageBox.Show("The two passwords didn't match each other.");
            }
    }
    TxBoxNewUserName.Text = "";
    TxtBoxPASS1.Text = "";
    TxBoxPass.Text = "";
}