Question

class Myclass
{
    static string[] user_Name = { "admin", "user1", "user2" };
    static string[] user_Password ={ "admin", "123", "789" };

    public static void Check_Method(string u_name, string u_password)
    {

        for (int i = 0; i < user_Name.Length; i++)
        {
            if (u_name == user_Name[i] && u_password == user_Password[i])
            {
                MessageBox.Show("login successful");
                break;
            }
            else
            {
                if (i == (user_Name.Length - 1))
                    MessageBox.Show("Badshow");
            }
        }
    }
    public static void add_user(string name, string password)
    {
        i=user_Name.Length;
        user_Name[i]=name;
        user_Password[i]=password;
        //here i want to add another user but im unable to find the way
    }
}

但是它给出了一个错误，它超出了数组的边界。

执行此操作最方便的方法是什么？

Answer 1

如果您需要可变大小的存储空间，请不要使用数组。

使用List<string>代替 - 它允许您Add项。

在您的情况下，您选择的两个阵列是有问题的，因为每个用户都有相应的密码 - 总是如此。这表明你应该有一个自定义类来保存用户/密码对。

使用这样的课程（比如User），您可以保留List<User>并简化代码。

Answer 2

尝试使用List<>。

class Myclass
{
    static List<string> user_Name = new List<string>{ "admin", "user1", "user2" };
    static List<string> user_Password = new List<string>{ "admin", "123", "789" };

    public static void Check_Method(string u_name, string u_password)
    {

        for (int i = 0; i < user_Name.Length; i++)
        {
            if (u_name == user_Name[i] && u_password == user_Password[i])
            {
                MessageBox.Show("login successful");
                break;
            }
            else
            {
                if (i == (user_Name.Length - 1))
                    MessageBox.Show("Badshow");
            }
        }
    }
    public static void add_user(string name, string password)
    {
        user_Name.Add(name);
        user_Password.Add(password);
    }
}

这是重构版本：

用户包含在用户类中。

它们是IEquatable<>，用于比较用户名/密码（您可能需要考虑查看Guid以保持其唯一性。）

轻松添加/删除用户。

public class User : IEquatable<User>
{
    public User(string name, string password)
    {
        Name = name;
        Password = password;
    }

    public string Name { get; set; }
    public string Password { get; set; }

    public bool Equals(User other)
    {
        if (other == null) return false;

        return other.Name == Name && other.Password == Password;
    }
}

public class AuthenticationManager
{
    private List<User> LoggedInUsers = new List<User>
    { new User("Admin", "admin"), new User ("user1", "123"), new User ("user2", "789") };

    public bool Authenticate(string userName, string password)
    {
        var user = new User(userName, password);

        //if the user is in the list it will return false otherwise true.
        return !LoggedInUsers.Any(u => user.Equals(user)); 
    }

    public void Login(string name, string password)
    {
        LoggedInUsers.Add(new User(name, password));
    }

    public void Logout(string name, string password)
    {
        LoggedInUsers.Remove(new User(name, password));
    }
}

Answer 3

好吧，我认为你可能会以错误的方式思考这个问题。你使用数组的方式是尖叫出一个对象。

我会让User成为像这样的对象

public class User 
{
  public string UserName { get; set;}
  public string Password { get; set;}
}

然后我会维护一个用户列表。这样您就不需要维护数组索引，并且可以轻松地将新用户添加到列表中。

Answer 4

为什么不使用和List并应用DTO而不是倍数string[]？

尝试这样的事情：

1）为您的用户创建DTO：

public class UserDTO
{
    public string UserName { get; set; }
    public string Password { get; set; }    
}

2）使用和List<DTO>

class Myclass
{
    static List<UserDTO> users = new List<UserDTO>()
    {
        new UserDTO() { UserName= "admin", Password = "admin" } ,
        new UserDTO() { UserName= "user1", Password = "123" } ,
        new UserDTO() { UserName= "user2", Password = "789" } ,
    }

    public static void Check_Method(string u_name, string u_password)
    {
        if (users.Exists(x => x.UserName == u_name && x.Password == u_password)
        {
               MessageBox.Show("login successful");
        }
        else
        {
            MessageBox.Show("Badshow");
        }
    }
    public static void add_user(string name, string password)
    {
        users.Add(new UserDTO() { UserName= name, Password = password });
    }
}

Answer 5

尝试使用List<string>课程代替string[]

使用object.Add()方法

将项目添加到数组中

向字符串数组添加字符串

5 个答案: