Help with output

[tallguy]

void Graph::printPath( const Vertex & dest ) const
{
   string m;
    if( dest.prev != NULL )
    {
         vector<Edge> e = (dest.prev)->adj;
        for(unsigned int i = 0; i< e.size(); i++)
          {
            Edge edge = e[i];
            if( edge.dest == &dest)
            {
                m = (*edge.movieTitle).title();
                break;
            }
        
          }
          
          printPath( *dest.prev );
          
          cout << " was in " << m << " with ";
          cout << ((*data)[(dest.a)]).name() << endl;
     	  cout << ((*data)[(dest.a)]).name();
    	
   
    }
    else
    {
    	cout << ((*data)[(dest.a)]).name();	
    } 
}

I have this function and currently with the input I give it the output is:

Damon, Matt was in Saving Private Ryan with Hanks, Tom
Hanks, Tom was in Apollo 13 with Harris, Ed
Harris, Ed

It is displaying the final name an extra time. I want the output to just be:

Damon, Matt was in Saving Private Ryan with Hanks, Tom
Hanks, Tom was in Apollo 13 with Harris, Ed

Anyone know how I can fix this?

[Desolation]

void Graph::printPath( const Vertex & dest ) const
{
   string m;
    if( dest.prev != NULL )
    {
         vector<Edge> e = (dest.prev)->adj;
        for(unsigned int i = 0; i< e.size(); i++)
          {
            Edge edge = e[i];
            if( edge.dest == &dest)
            {
                m = (*edge.movieTitle).title();
                break;
            }
        
          }
          
          printPath( *dest.prev );
          
          cout << " was in " << m << " with ";
          cout << ((*data)[(dest.a)]).name() << endl;
     	  cout << ((*data)[(dest.a)]).name();
    	
   
    }
    else
    {
    	cout << ((*data)[(dest.a)]).name();	
    } 
}

I guess it's time to bang your head on your desk =P

[tallguy]

But I'm doing that because I want to display it again on the next line as the first name.

Damon, Matt was in Saving Private Ryan with Hanks, Tom
Hanks, Tom was in Apollo 13 with Harris, Ed

[Desolation]

Oh, sorry, I didn't see that.

[Daved]

At the point in the function after you call printPath, you know that *dest.prev refers to the first name in the line and dest refers to the second name, right? So just use that information to write out an entire single line by itself right there.

At least I think that will work. You will want to get rid of the else, since I assume that is meant only to print the very first name the first time, but the new method won't need that.