1

Сейчас реализовано так (ControlType - enum value):

private readonly ControlType[] _types;

private ComboBoxItem[] GetItems()
{
    ComboBoxItem[] items = new ComboBoxItem[_types.Length];
    int i = 0;

    foreach (ControlType type in _types)
    {
        ComboBoxItem item = new ComboBoxItem(type.ToString());
        items[i++] = item;
    }

    return items;
}

Как-то громоздко, можно же, наверно, и в одну строчку это сделать с помощью LINQ? Плюс если можно, то как в этом случае будет с производительностью?

2 ответа 2

5

Видимо, так:

return _types.Select(t => new ComboBoxItem(t.ToString())).ToArray();

Однако использовать ToArray не рекомендуется, потому что выделений памяти больше, чем могло бы быть. Можно вызвать ToList. А можно вообще вернуть IEnumerable<ComboBoxItem>, особенно если это приватный метод, и вы контролируете его использование.

На фоне создания ComboBoxItem и прочего производительностью LINQ можно пренебречь. Не занимайтесь преждевременной оптимизацией.

6
  • про оптимизацию у меня был только сопутствующий вопрос. Основное - как быстрее просто написать код. 2 мая 2015 в 11:55
  • @YShinkarev Максимально кратко — это _types.Select(t => new ComboBoxItem(t.ToString()) и вернуть IEnumerable<ComboBoxItem>.
    – Kyubey
    2 мая 2015 в 11:56
  • Но мне просто интересно для справки. Возвращаются же часто такие коллекции. Почему именно массив? Пишется это применительно для Unity3D. А в сети часто наталкивался на утверждение, что используете лучше массив, чем список (это касается в первую очередь объектов, порядок/состав которых после не будет меняться. Не могу использовать IEnumerable (а может просто не знаю), потому что, в данном случае, на выходе я будут иметь некоторый SelectedIndex и мне же его надо будет вернуть потом... 2 мая 2015 в 12:01
  • @YShinkarev Опечатка, пропустил скобку, см. исправленный ответ. Для подавляющего большинства приложений разница в производительности пренебрежительно мала. То есть разница-то есть, но на "оптимизации" вы не выиграете чего-то, что можно заметить.
    – Kyubey
    2 мая 2015 в 12:08
  • машинально смотрел, потом переписал вручную, всё стало нормально. И тогда последний вопрос - вернул я IEnumerable, а как мне вернуть быстро третий элемент? 2 мая 2015 в 12:10
0

Можно короче даже без Linq:

private readonly ControlType[] _types;

private ComboBoxItem[] GetItems()
{
    var items = new ComboBoxItem[_types.Length];
    for(var i = 0; i < _types.Length; i++)
        items[i] = new ComboBoxItem(_types[i].ToString())

    return items;
}

Еще красивее без Linq, но с серьезным дополнительным оверхедом:

private readonly ControlType[] _types;

private ComboBoxItem[] GetItems()
{
    var items = new List<ComboBoxItem>();
    foreach(var type in _types)
        items.Add(new ComboBoxItem(type.ToString()));

    return items.ToArray();
}
2
  • по большому счёту это только разновидность, суть остаётся такой же. У меня только вопрос - почему вы сделали локальную переменную length? 2 мая 2015 в 11:49
  • @YShinkarev Привычка. Поменял. Если вижу свойство, стараюсь вызывать его только один раз, по понятным причинам. В данном случае скорее всего будет оптимизировано и так. Но, вообще, рекомендуют проверять скорость работы, если это критично (ищите там пункт Performance).
    – Vladimir
    2 мая 2015 в 11:54

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.