Remove object from a collection

  • Thread starter Thread starter Jibesh
  • Start date Start date
J

Jibesh

Tony,

You can also try Predicate method of List<T>.Remove(...), This method also
searching through the list elements. Read MSDN for more details.

eg:
public class ParametermappingObj
{
private string _name;
public ParametermappingObj(string name)
{
_name = name;
}
public string Name
{
get { return this._name; }
set { this._name = value; }
}
}

public partial class MyClass

{

static string id = "aa";

public MyClass()

{

List<ParametermappingObj> myList = new List<ParametermappingObj>();

ParametermappingObj aa = new ParametermappingObj("aa");

ParametermappingObj bb = new ParametermappingObj("bb");

myList.Add(aa);

myList.Add(bb);

ParametermappingObj sublist = myList.Find(FindId); // myList.Remove(FindId)

}

public static bool FindId(ParametermappingObj obj){

if (obj.Name == id){ return true; }

return false;

}

}

Thanks
Jibesh
 
Thanks Pete.

sorry i miss typed it.

Its RemoveAll and not Remove. Remove only accepts reference and not
Predicate as you said.


Regards,
Jibesh
 
Hello!

I have a collection with a lot of object.
Each object has the following definition.
public class ParametermappingObj
{
private string Name {get;set;}
private string Id {get;set;}
some other field defined here
....
}

If I know for example want to remove object with the following key values
Name = "CC" and Id = "123-CHISI" how is that done ?

Is there a better way then to loop through the whole collection

//Tony
 
Hello!

I forgot to say that I use VS2008 and the collection consist of a generic
List<ParametermappingObj>

//Tony
 
Tony said:
Hello!

I forgot to say that I use VS2008 and the collection consist of a
generic List<ParametermappingObj>

//Tony
Click to expand...

No, there is no other way to locate an item in the list other than
looping through the items in the list.

You can use the RemoveAll method to remove specific items:

theMappingList.RemoveAll(delegate(ParametermappingObj p){ return p.Name
== "CC" && p.Id == "123-CHISI"; });

However, if you know that there is only one item to remove, it's more
efficient to do the looping yourself, so that you can exit from the loop
as soon as you have found the item to remove.
 
Hello!

I have a collection with a lot of object.
Each object has the following definition.
public class ParametermappingObj
{
private string Name {get;set;}
private string Id {get;set;}
some other field defined here
...

}

If I know for example want to remove object with the following key values
Name = "CC" and Id = "123-CHISI" how is that done ?

Is there a better way then to loop through the whole collection

//Tony
Click to expand...

No, you have to iterate on the collection but you can do it in an
elegant way :)
List<ParametermappingObj> list = new .....
......
ParametermappingObj element = null;
element = list.Find( x=>x.Name == "CC" && x.Id="123-CHISI");
if (element!= null)
list.Remove(element);
 
No, there is no other way to locate an item in the list other than
looping through the items in the list.

You can use the RemoveAll method to remove specific items:

theMappingList.RemoveAll(delegate(ParametermappingObj p){ return p.Name
== "CC" && p.Id == "123-CHISI"; });

However, if you know that there is only one item to remove, it's more
efficient to do the looping yourself, so that you can exit from the loop
as soon as you have found the item to remove.
Click to expand...

You can use LINQ in a very elegant way. it will iterate until the
element is found for the first time.
 
In what way is that an improvement over calling Find() and then Remove()?
And is it as efficient as doing the enumeration explicitly, keeping a
counter, and then removing the element by index?
Click to expand...

More code?
Other than that there should be no difference, there is no other way
than iterate until find it, break the loop and then remove it. As I
said there is no other way around it
 
Back
Top