A better way??

Started by
2 comments, last by kelaklub 20 years, 3 months ago
I wrote the following function as way to eliminate duplicate points that I stored inside an STL vector. Would anyone please be able to give me some ideas on ways that I could optimize this? Thank you.

template <typename tType>
void fn_eliminateDuplicatePoints(vector<cl_3dPoint<tType> >& vertices);

template <typename tType>
void fn_eliminateDuplicatePoints(vector<cl_3dPoint<tType> >& vertices)
{
    int i = 0;
    int j = 0;
    bool* duplicateFound = NULL;

	// GET THE TOTAL NUMBER OF POINTS

    int number_of_points = vertices.size();

    duplicateFound = new bool[number_of_points];

    // INITIALIZE OUR ARRAY OF FLAGS

    for( i = 0 ; i < number_of_points ; i = i + 1 )
    {
        duplicateFound[i] = false;
    }
    // TAG THE INDEXES OF THE ARRAY WHEREVER DUPLICATE POINTS OCCUR BY SETTING OUR BOOLEAN FLAG

    for( i = 0 ; i < number_of_points ; i = i + 1 )
    {
        for( j = i + 1 ; j <= number_of_points - 1 ; j = j + 1  )
        {
            if( ( vertices[i] == vertices[j] ) && ( duplicateFound[j] == false  ) )
            {
                duplicateFound[j] = true;
            }
        }
    }

	// GO THRU THE VECTOR CONTAINER ELIMINATING ALL DUPLICATE POINTS

    for( i = number_of_points - 1 ; i >= 0 ; i = i - 1 )
    {
        if( duplicateFound[i] == true )
        {
			vertices.erase(&vertices[i]);
        }
    }
    delete [] duplicateFound;  // DEALLOCATING OUR DYNAMICALLY ALLOCATED ARRAY

    duplicateFound = NULL;
}
[edited by - kelaklub on January 17, 2004 9:42:11 PM] [edited by - kelaklub on January 17, 2004 9:43:52 PM]
Advertisement
Is the order of your points important? If it isn''t, it might just be easier to std::sort() the vector then use std::unique() to get rid of duplicate elements.
Hmmmmm, that sounds like a plan. Thanks.
or use std::set if you want to store a set of unique things

This topic is closed to new replies.

Advertisement