Начнём с того, что <UserPass>Пароль1</UserPass>
- это элемент, а не атрибут.
Я бы предложил следующую структуру xml:
<Users>
<User>
<Name>Человек1</Name>
<Pass>Пароль1</Pass>
</User>
<User>
<Name>Человек2</Name>
<Pass>Пароль2</Pass>
</User>
</Users>
Элемент Users
(во множественном числе) содержит множество элементов User
(в единственном числе), каждый из которых представляет информацию об одном пользователе. Имена вложенных элементов сокращаем до Name
и Pass
, т. к. и так понятно, что они относятся к юзеру.
Названия всех методов и переменных в коде должны хорошо описывать своё предназначение. XMLRes
- что делает этот метод? Из названия непонятно. Переименуем его, например, в LoadData
.
Имена phoneElement
, companyElement
, priceElement
вероятно остались от какого-то прежнего кода. Копипаста - зло! Переименуем их.
Сделаем класс, который будет содержать информацию о юзере. Таким образом данные об имени и пароле хранятся в одном объекте.
public class User
{
public string Name { get; set; }
public string Pass { get; set; }
}
Для хранения данных о пользователях сделаем поле:
private List<User> _users = new List<User>();
Код загрузки и привязки данных:
void LoadData()
{
var xdoc = XDocument.Load("test.xml");
foreach (XElement userElement in xdoc.Element("Users").Elements("User"))
{
string name = userElement.Element("Name").Value;
string pass = userElement.Element("Pass").Value;
var user = new User { Name = name, Pass = pass };
_users.Add(user);
}
comboBoxUsers.DisplayMember = nameof(User.Name);
comboBoxUsers.ValueMember = nameof(User.Pass);
comboBoxUsers.DataSource = _users;
textBoxPass.DataBindings.Add(nameof(TextBox.Text), _users, nameof(User.Pass));
}
На основе данных из xml создаём классы User и заносим их в коллекцию.
Делаем привязку этого списка к обоим контролам.
Комбобоксу задаём в DisplayMember
, что он будет показывать имя пользователя. А в ValueMember
задаём, какое значение он будет выдавать при выборе строки.
Текстбоксу задаём привязку, по которой он будет показывать свойство Pass
.
Вместо парсинга xml с помощью LINQ можно применить десериализацию. Но в данном случае это не имеет большого смысла, т. к. код и так лаконичен.
Вместо List<User>
, возможно, следует испоьзовать BindingList<User>
- при добавлении данных в эту коллекцию будет автоматически обновляться GUI.
Users
, заведите в нем 2 свойстваUserName
иUserPass
и десереализуйте в массив. Операция настолько простая, что я удивлен, что вы не нашли решения. Я как минимум несколько раз их здесь постил. Например вот. Не бойтесь использовать ООП.