C# - combobox mnożenie się obiektów listy

Question

Witam, potrzebuje pomocy. Czy ktoś jest w stanie odpowiedzieć mi na pytanie, dlaczego z początkowej listy obiektów (4 -elementy) w combobox'ie po wciśnięciu przycisku dodaj, wszystkie te 4 elementy się mnożną i tak w kółko ? Oczywiście wszystko ładnie dodaje się do datagridview jednak w kolekcji combobox itemy się duplikują :/

 

Kod:

            Pracownik p1 = new pracownik();
            p1.Imię_Nazwisko = "Jan Kowalski";
            p1.Stawka = 20.0m;
            Mechanik.Mechanicy.Add(p1);

            comboBox_pracownicy.Items.Clear();
            foreach (Pracownik m in Pracownik.Pracownicy)
            {
    
                comboBox_pracownicy.Items.Add(p1);
            }

            foreach (Pracownik prac in Pracownik.pracownicy)

            {
                dataGridView1.Rows.Add(prac.pracownicy);
            }

 

Answer 1

"Czeski" błąd  

Iterujesz w pętli foreach

foreach (Pracownik m in Pracownik.Pracownicy)
{ 
    comboBox_pracownicy.Items.Add(p1);
}

więc chyba chodzi Tobie o ...

foreach (Pracownik m in Pracownik.Pracownicy)
{ 
    comboBox_pracownicy.Items.Add(m);
}

 

Comments

Niestety  dalej to samo :/ Po dodaniu wartości z combobox do datagridview wszystkie elementy z combobox się mnożą :/

---

Musisz udostępnić więcej kodu, bo z tego co pokazałeś, ciężko jest coś więcej wywnioskować.

---

@VBService, Mam 2 formatki. 

1. Tutaj mam tabelkę oraz jeden przycisk, którym wywołuje formatkę nr 2. I dodaje wartości z formatki 2 do tabelki.

        private void button1_Click(object sender, EventArgs e)
        {
            Form2 formularz_dodaj = new Form2();
            formularz_dodaj.ShowDialog();

            dataGridView1.Rows.Clear();
            foreach (Pracownik prac in Pracownik.Pracownicy)

            {
                dataGridView1.Rows.Add(prac);
            }
        }

2.  Tutaj zadeklarowałem Imię i nazwisko pracownika którego chce wybrac z listy combobox i dodać go do tabelki z pierwszej formatki. Wszystko się dodaje jednak, gdy  z powrotem chce dodać kolejnego pracownika, lista w combobox zwiększa się o +1 ( Zamiast jednego Jan Kowalski pojawia się 2 itd. za każdym dodaniem do tabelki)

        private void Form2_Load(object sender, EventArgs e)
        {
            Pracownik p1 = new Pracownik();
            p1.Imię_Nazwisko = "Jan Kowalski";
            Pracownik.Pracownicy.Add(p1);

            comboBox_pracownik.Items.Clear();
            foreach (Pracownik m in Pracownik.Pracownicy)
            {

                comboBox_pracownik.Items.Add(m);
            }
        }

oraz utworzyłem klase o nazwie Pracownik:

    class Pracownik
    {
        public string Imię_Nazwisko;
        static public List<Pracownik> Pracownicy = new List<Pracownik>();


        public override string ToString()
        {//popraw już coś istniejącego
            return Imię_Nazwisko;
        }
    }

 

---

@VBService, Wydaje mi się, że udało mi się naprawić ten problem. Mianowicie w formatce 2 w pętli foreach dodałem: "comboBox_pracownik.Items.Clear();" tuż przed comboBox_pracownik.Items.Add. Jednak gdy mój combobox ma więcej elementów to już nie działa.

---

Dlaczego w Form2_Load na "sztywno" dodajesz Jan Kowalski?

w ten sposób za każdym odwołaniem się do Form2 - zawsze "uruchomiane" jest zdarzenie Load i za każdym razem dodawany jest kolejny Kowalski do listy, więc czyszczenie combox-a nie daje zamierzonego przez Ciebie efektu.

---

Czyli jak chce to wrzucić na sztywno to powinienem to wrzucić np. tutaj?

        public Form1()
        {
            InitializeComponent();
        }

 

---

Moim zdaniem trzeba by było dopisać kod, który sprawdza czy na liście jest już dodany Kowalski i go nie dodawać jak już jest, albo dodać jak go jeszcze nie ma np.:

        private void Form1_Load(object sender, EventArgs e)
        {
            Pracownik p1 = new Pracownik();
            p1.Imię_Nazwisko = "Jan Kowalski";

            if (!Pracownik.Pracownicy.Contains(p1))
            {
                Pracownik.Pracownicy.Add(p1);
            }

            comboBox_pracownik.Items.Clear();
            foreach (Pracownik m in Pracownik.Pracownicy)
            {

                comboBox_pracownik.Items.Add(m);
            }
        }