3
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
    }
}

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

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

4

5 回答 5

7

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

改为使用List<string>- 它允许您使用Add物品。


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

有了这样一个类(比如User),您将拥有List<User>并简化您的代码。

于 2013-03-05T19:51:46.443 回答
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));
    }
}
于 2013-03-05T19:53:43.983 回答
0

尝试使用List<string>类而不是string[]

object.Add()并使用方法将项目添加到数组

于 2013-03-05T19:56:15.520 回答
0

好吧,我想你可能想错了。您使用数组的方式是为一个对象而尖叫。

我会让 User 成为这样的对象

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

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

于 2013-03-05T19:54:34.970 回答
0

为什么不使用 andList并应用 DTO 而不是 multiples 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 });
    }
}
于 2013-03-05T19:55:51.213 回答