Removing listbox item

Question

Im at beginner level and have tried for a while now. Im trying to remove an item from a ListBox with the help of a remove button. The code is not giving away any errors but the items is not disappearing from the list. This is the part im struggling with

void taBort()
{
    listboxKontakter.SelectedItems.Remove(listboxKontakter.SelectedItems);
    textboxAnteckningar.Clear();
    textboxGatuadress.Clear();
    textboxNamn.Clear();
    textboxPostnummerOrt.Clear();
    textboxEmail.Clear();
    textboxFödelsedag.Value = DateTime.Now;
}

Here is my entire code:

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.IO;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace adressboken
{
    public partial class Form1 : Form
    {

        List<Kontakter> kontaktLista = new List<Kontakter>();
        Kontakter person;
        string path = "kontakter.txt";



        public Form1()
        {
            InitializeComponent();
        }

        private void LäggTill_Click(object sender, EventArgs e)
        {
            person = new Kontakter();
            person.FullständigtNamn = textboxNamn.Text;
            person.Gatuadress = textboxGatuadress.Text;
            person.PostnummerOrt = textboxPostnummerOrt.Text;
            person.Födelsedag = textboxFödelsedag.Value;
            person.Email = textboxEmail.Text;
            person.Anteckningar = textboxAnteckningar.Text;

            kontaktLista.Add(person);

            listboxKontakter.DataSource = null;
            listboxKontakter.DisplayMember = "FullständigtNamn";
            listboxKontakter.DataSource = kontaktLista;

            textboxAnteckningar.Clear();
            textboxGatuadress.Clear();
            textboxNamn.Clear();
            textboxPostnummerOrt.Clear();
            textboxEmail.Clear();
            textboxFödelsedag.Value = DateTime.Now;

            textboxAntal.Text = kontaktLista.Count.ToString();
        }

        private void Rensa_Click(object sender, EventArgs e)
        {
            textboxAnteckningar.Clear();
            textboxGatuadress.Clear();
            textboxNamn.Clear();
            textboxPostnummerOrt.Clear();
            textboxEmail.Clear();
            textboxFödelsedag.Value = DateTime.Now;
        }
        void taBort()
        {

            textboxAnteckningar.Clear();
            textboxGatuadress.Clear();
            textboxNamn.Clear();
            textboxPostnummerOrt.Clear();
            textboxEmail.Clear();
            textboxFödelsedag.Value = DateTime.Now;
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            kontaktLista = new List<Kontakter>();
            string line = "";
            StreamReader sr = new StreamReader(path);
            while ((line = sr.ReadLine()) != null)
            {
                string[] listarray = line.Split(',');
                person = new Kontakter();
                person.FullständigtNamn = listarray[0];
                person.Gatuadress = listarray[1];
                person.PostnummerOrt = listarray[2];
                person.Email = listarray[3];
                person.Födelsedag = Convert.ToDateTime(listarray[4]);
                person.Anteckningar = listarray[5];
                kontaktLista.Add(person);
            }
            sr.Close();
            listboxKontakter.DataSource = kontaktLista;
            listboxKontakter.DisplayMember = "FullständigtNamn";
        }

        public void listboxKontakter_Click(object sender, EventArgs e)
        {
            person = (Kontakter)listboxKontakter.SelectedItem;
            textboxNamn.Text = person.FullständigtNamn;
            textboxGatuadress.Text = person.Gatuadress;
            textboxPostnummerOrt.Text = person.PostnummerOrt;
            textboxEmail.Text = person.Email;
            textboxFödelsedag.Value = person.Födelsedag;
            var selectedindex = listboxKontakter.SelectedItems;
        }

        private void Spara_Click(object sender, EventArgs e)
        {
            StreamWriter sw = new StreamWriter(path);
            foreach (Kontakter k in kontaktLista)
            {
                sw.WriteLine(k.FullInfo);
            }
            sw.Close();
        }

        private void taBortToolStripMenuItem_Click(object sender, EventArgs e)
        {
            taBort();
        }

        private void TaBort_Click(object sender, EventArgs e)
        {
            taBort();
        }


    }


}

Answer 1

Your code seems to remove all selected items, and does not refresh.

How about:

listboxKontakter.SelectedItems.Remove(listboxKontakter.SelectedItem);
listboxKontakter.Refresh();

You remove only the single selected item, then refresh your listbox.

If you want to remove ALL items try:

listboxKontakter.Items.Clear();

If you use a DataSource try:

listboxKontakter.DataSource = null;

If all else fails you could loop through the collection and RemoveAt:

for(int i=listboxKontakter.Items.Count; i > -1; i--) {
{
    listboxKontakter.Items.RemoveAt(i);
}

Based on a bit of chatting, this should work for you:

void taBort()
{
    var newList = (List<Kontakter>)listboxKontakter.DataSource;
    var ds = newList.Where(k => k.FullständigtNamn != ((Kontakter)listboxKontakter.SelectedItem).FullständigtNamn).ToList();
    listboxKontakter.DataSource = ds;
    listboxKontakter.DisplayMember = "FullständigtNamn";
    textboxAnteckningar.Clear();
    textboxGatuadress.Clear();
    textboxNamn.Clear();
    textboxPostnummerOrt.Clear();
    textboxEmail.Clear();
    textboxFödelsedag.Value = DateTime.Now;
}

If you want to remove several items at once try:

var ds = newList.Where(k => !listboxKontakter.SelectedItems.Contains(k.FullständigtNamn)).ToList();