Не работает удаление из списка(Remove). Что делать???

Question

[Artem Nim]

public static List< Note > list_note = new List< Note >();
................
private void button2_Click(object sender, EventArgs e)// удалить
{

Note note = new Note();
DateTime.TryParse(dataGridView1.Rows[dataGridView1.SelectedCells[0].RowIndex].Cells[0].Value.ToString(), out note.date);
note.text = dataGridView1.Rows[dataGridView1.SelectedCells[0].RowIndex].Cells[1].Value.ToString();

XmlSerializer x = new XmlSerializer(typeof(List));

list_note.Remove( note); // тут должно удалить элемент

Stream st = File.OpenWrite("data.xml");
x.Serialize(st, list_note);
st.Close();
dataGridView1.Rows.RemoveAt(dataGridView1.SelectedCells[0].RowIndex);
}

Comments

[Artem Nim]

Вот код с решением, если кому-то надо:
class Note
{
public DateTime date;
public string text;

public override string ToString()
{
return date + " " + text;
}
public override bool Equals(Object obj)
{
bool rez = false;
if (obj != null)
{
Note n = obj as Note;
if ((date == n.date) && (text == n.text))
rez = true;
}

return rez;
}
}

class Program
{

static void Main(string[] args)
{
List note = new List();

note.Add(new Note { date = DateTime.Now.Date, text = "q" });
note.Add(new Note { date = DateTime.Now.Date, text = "1111" });

for (int i = 0; i < note.Count; i++)
Console.WriteLine(note[i].ToString());
Console.WriteLine();

note.Remove(new Note { date = DateTime.Now.Date, text = "1111" });

for (int i = 0; i < note.Count; i++)
Console.WriteLine(note[i].ToString());
Console.ReadKey();
}
}

Answer 1

[MrDywar Pichugin]

1) Вы создаете элемент.
2) Вы удаляете элемент не добавив его в список.

List что бы удалить элемент Note, сравнивает их.
https://msdn.microsoft.com/ru-ru/library/cd666k3e(...

Если тип T реализует IEquatable — универсальный интерфейс компаратор проверки на равенство Equals метода этого интерфейса; в противном случае — функцию сравнения по умолчанию является Object.Equals.
Этот метод выполняет линейный поиск; Таким образом, этот метод является операцией O (n) операция, где n — Count.

Note должен уметь сравнивать себя с другими.
https://msdn.microsoft.com/ru-ru/library/ms173147(...

Пройдитесь по коду в режиме отладки, шаг за шагом.
Точно узнайте что находится в List, и что происходит после remove, где именно ошибка.

Comments

[Artem Nim]

private void button1_Click(object sender, EventArgs e)//добавить
{

Note note = new Note();
DateTime.TryParse(label2.Text + " " + textBox1.Text, out note.date);
note.text = textBox2.Text;

XmlSerializer x = new XmlSerializer(typeof(List));
Stream s = File.OpenRead("data.xml");
list_note = (List)x.Deserialize(s);
s.Close();//считываем с xml

list_note.Add(note);//добавляет в список


Stream st = File.OpenWrite("data.xml");
x.Serialize(st, list_note);
st.Close();
// записываем
}
Я добавляю.

[Artem Nim]

Насколько я понимаю вот такой код должен удалить обьект № 2 из списка:
class Note
{
public DateTime date;
public string text;

public override string ToString()
{
return date + " " + text;
}
}

class Program
{
public static List note = new List();
static void Main(string[] args)
{

note.Add(new Note { date = DateTime.Now,text ="q " });
note.Add(new Note { date = DateTime.Now, text = "1111" });

for (int i = 0; i < note.Count; i++)
Console.WriteLine(note[i].ToString());
Console.WriteLine();

note.Remove(new Note { date = DateTime.Now, text = "1111" });

for (int i = 0; i < note.Count; i++)
Console.WriteLine(note[i].ToString());
Console.ReadKey();
}
}
Я правильно понимаю?

[Artem Nim]

А я понял. Спасибо Вам огромное :)