Блог

У меня есть некоторые данные, хранящиеся в словаре, где значения в основном представляют собой список объектов с небольшим количеством атрибутов в них. Прямо сейчас я просматриваю следующее, чтобы получить данные, хранящиеся в определенном атрибуте. Затем эти данные добавляются в выпадающий список (unity UI dropdown)

 foreach (KeyValuePair<string, List<NameIDValuePair>> kvp in TeamValuePair)
            {
                List<NameIDValuePair> list = kvp.Value;

                if(kvp.Key == teamNames.options[teamNames.value].text)
                {
                    foreach (var rec in list)
                    {
                        screenNamesDropDown.options.Add(new TMP_Dropdown.OptionData { text = rec.ScreenName });
                    }
                }
            }
 

Имена команд и имена экранов — это выпадающие элементы, являющиеся частью моего пользовательского интерфейса Unity.

Структура NameIdValuePair выглядит следующим образом:

 public class NameIdValuePair 
        {
            public string ScreenName { get; private set; }

            public string ScreenId { get; private set; }
        }
 

Я хотел бы оптимизировать этот фрагмент кода лучшим способом, используя linq — чтобы он был немного более читабельным. Поскольку я довольно новичок в linq, я не совсем уверен, правильно ли я использую ключевые слова при поиске предложений, но пока у меня не было большого успеха в поиске какого-либо полезного предложения.

Спасибо

Как упоминалось ранее, вместо цикла a Dictionary — где мы уже знаем, что ключи уникальны — вы могли бы просто использовать Dictionary.TryGetValue

 // do this only once! 
var key = teamNames.options[teamNames.value].text;

if (TeamValuePair.TryGetValue(key, out var list))
{
    foreach(var item in list)
    {
        screenNamesDropDown.options.Add(new TMP_Dropdown.OptionData(item.ScreenName));
    }
}
 

и тогда на самом деле единственное место, где вы могли бы использовать Linq, если вы действительно этого хотите, возможно, будет в

 var key = teamNames.options[teamNames.value].text;

if (TeamValuePair.TryGetValue(key, out var list))
{
    screenNamesDropDown.options.AddRange(list.Select(item => new TMP_Dropdown.OptionData(item.ScreenName)));
}
 

хотя, если это улучшает чтение, это сомнительно.

И в целом вопрос также будет заключаться в том, всегда ли вы хотите Add ( AddRange ) к screenNamesDropDown.options или, если вы, возможно, захотите фактически заменить параметры. Тогда вместо AddRange того, чтобы вы могли бы сделать

 screenNamesDropDown.options = list.Select(item => new TMP_Dropdown.OptionData(item.ScreenName)).ToList();