將字符串添加到字符串數組

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

好吧我想你可能會想這是錯誤的方式。你使用數組的方式是尖叫出一個對象。

我會做出用戶喜歡這樣

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[]

並使用將項添加到數組方法