views:

77

answers:

4

I need to save instances of type HANDLE to an array container and iterate over it, finally remove some of them when necessary,

Which container should I use for the purpose of easiness?

A: 

I'd probably use std::vector<HANDLE>. Even better would be to make a class around HANDLE whose destructor deals with closing it -- you'll need to reference count it.

Lou Franco
A: 

what about using a list?
http://www.cplusplus.com/reference/stl/list/

christian
A list will work, but if HANDLE is just at typical pointer, then for each element you have quite a bit of overhead in relation to the size of the object you're storing. This because each element will contain at least pointers to the previous and next elements in the list as lists are implemented as doubly linked lists. This space usage does allow quick removal from the middle and doesn't invalidate other iterators b/c underneath it's just reassigning some pointers. (i.e next and previous of the surrounding elements)
RC
+3  A: 

The best one will depend on how your going to access it and where you're going to remove elements from.

std::vector should usually be your default choice. Here you can iterate over the HANDLE elements and storage is efficient. The issue with vector may arise when you're erasing. If you are removing elements from the middle of the container, vector is not as efficient as something like a list or set and it will invalidate all iterators beyond the element you are erasing. Note that lists are more performant than a vector when erasing from the middle, but list is not terribly space efficient. I typically use list only as a last resort. A good option over the use of a list is the deque container if you need only efficient removal from the ends of the container.

If you need the data sorted (i.e. without using something like std::sort to sort a vector), you want to remove from anywhere without invalidating other iterators, then a set may be just what you're looking for.

It should be noted that I'm assuming your HANDLE class is either a pointer or is a class that meets the criteria for storage in a C++ std container. (Copy semantics, etc.) If using a pointer, then you will be responsible for closing and/or deallocating the resource that the pointer is referring too. This should be done via some RAII class. You SHOULD NOT use auto_ptr for this as it does not meet the requirements to be stored in a standard container. You can however use the shared_ptr class provided by the Boost libraries.

Here is some general info all these containers and the other container options available in C++.

Here is a simple example of how to use a vector from the reference pages. You can easily take this and produce what you want with your HANDLE class/handle. I've added an example loop printing the contents via iterators also.

// erasing from vector
#include <iostream>
#include <vector>
using namespace std;

int main ()
{
  unsigned int i;
  vector<unsigned int> myvector;

  // set some values (from 1 to 10)
  for (i=1; i<=10; i++) myvector.push_back(i);

  // erase the 6th element
  myvector.erase (myvector.begin()+5);

  // erase the first 3 elements:
  myvector.erase (myvector.begin(),myvector.begin()+3);

  cout << "myvector contains:";
  for (i=0; i<myvector.size(); i++)
  {
      cout << " " << myvector[i];
  }
  cout << endl;

  // Using iterators
  cout << "myvector contains (using iterators):";
  for (vector<unsigned int>::const_iterator it = myvector.begin();
       it != myvector.end();
       ++it)
  {
      cout << " " << *it;
  }
  cout << endl;
  return 0;
}
RC
Can you provide a sample how to use `vectore` for `HANDLE`? Is it thread safe?
Alan
Great tip! Seems I don't need an iterate to loop through all the items ,right ? Also, is it thread safe or not?
Alan
std::vector is thread safe, but not reentrant. http://en.wikipedia.org/wiki/Thread_safety
Clark Gaebel
I've always regard thread safe and reentrant the same thing,what's the difference between these two concepts?
Alan
No STL container is guaranteed by the standard to be thread safe. (including vector) This is implementation specific. You can find more information on thread safety in std containers here: http://stackoverflow.com/questions/1362110/is-the-c-stl-stdset-thread-safe
RC
A: 

As you are removing the handle from the middle of the container as well, so it is better to use List for your usecase.

bjskishore123