0

Имеется ComboBox, из которого пользователь выбирает строки, которые после этого отображаются на ListBox. Хочу эти выбранные элементы вывести через запятую.

Такой вариант:

string.Join(", ", listBox1.SelectedItems)

выдаёт System.Windows.Forms.ListBox+SelectedObjectCollection.
Пытаюсь обратиться к конкретному объекту ListBox по индексу:

listBox1.Items[1].ToString()

получаю простанство имен.имякласса, вместо значения, которое хранится по индексу 1.
Что не так?

7
  • Это Winforms? Добавьте тег winforms тогда.
    – aepot
    1 ноя 2020 в 19:30
  • Я так понимаю в listBox1.Items некие объекты ? т.е. не просто string ? тогда если хотите заюзать .ToString() переопределите его в ваше классе, объекты которого грузите в Items, а иначе он по дефолту выдает имя типа вашего класса 1 ноя 2020 в 19:32
  • тогда будет можно по идее string.Join(", ", listBox1.Items) 1 ноя 2020 в 19:34
  • 1
    string.Join(", ", listBox1.SelectedItems) => string.Join(", ", listBox1.SelectedItems.Cast<НужныйТипОбъекта>()) (привет WinForms'у и его ужасным костылям). В случае с listBox1.Items[1].ToString() - если у вас там класс, то так и должно быть, обращайтесь значит к свойствам этого класса (listBox1.Items[1].Свойство), если у вас там тип по типу string, то такого не должно быть.
    – EvgeniyZ
    1 ноя 2020 в 20:01
  • @PavelPopov Вы вместо удаления вопроса, лучше бы его подправили, ведь по поводу listBox1.Items[1].ToString() у вас одно из решений задачи, описали бы почему так, дописали бы как можно без переопределения ToString() поступить и норм, а про string.Join() всего лишь надо про каст упомянуть...
    – EvgeniyZ
    1 ноя 2020 в 21:00

1 ответ 1

0
  1. Нужно переопределить ToString() в вашем классе, который вы используете для заполнения Items, иначе по дефолту он выводит наименование типа.

  2. Теперь можно джойнить строку как из SelectedItems, так и из Items, НО обязтаельно через Cast к нужному типу(таковы особенности этой коллекции)

    public class MainForm
    {
         public MainForm()
         {
             listBox1.Items.Add(new DataItem
                 {
                     Name = "Петя"
                 });
             listBox1.Items.Add(new DataItem
                 {
                     Name = "Вася"
                 });
    
             var str = string.Join(",", listBox1.Items.Cast<DataItem>());
         }
     }
    
     class DataItem
     {
         public string Name { get; set; }
    
         public override string ToString()
         {
             return Name;
         }
     }
    
4
  • При чем здесь static void Main?
    – aepot
    1 ноя 2020 в 19:47
  • 1
    @aepot тестанул на всякий случай(на простом массиве), но открыл консоль. А потом просто дописал лист вместо массива 1 ноя 2020 в 19:54
  • Как это отвечает на поставленный вопрос? Вот автор написал, что делает string.Join(", ", listBox1.SelectedItems), но получает System.Windows.Forms.ListBox+SelectedObjectCollection. Вы сейчас банально повторили его код, который возвращает кстати тоже самое, что и у него, без объяснений почему так, что надо сделать, как сделать и так далее. Так в чем польза?
    – EvgeniyZ
    1 ноя 2020 в 20:21
  • @EvgeniyZ свою идею я пояснил в комменте под вопросов. Да нужно было ее скопировать в ответ и да вы правы бе каста это не работает. Я почему то думал, что так отработает, может путаю с WPF 1 ноя 2020 в 20:47

Ваш ответ

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

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