not all control paths return a value

[kfaday]

i loathe that warning.
****!!. i can't see why!! can you?

cvehiculo *citerador::obtener_siguiente()
{
	if (this->nodo_actual==NULL)
	{
		this->nodo_actual=this->lista.get_primero();
		cvehiculo *vtemp=(this->nodo_actual->get_vehiculo());
		return vtemp;
	}
	else
	{
		if (this->nodo_actual!=NULL)
		{
			while (this->nodo_actual!=NULL)
			{
				this->nodo_actual=(this->lista.obtener_siguiente(this->nodo_actual));
				if (this->nodo_actual)
				{	
					cvehiculo *vtemp=(this->nodo_actual->get_vehiculo());
					return vtemp;			
				} 
				else 
					return NULL;
			}
		}
		else
			return NULL;
	}
}

thanks

[Guysl]

yeah, compilers paranoia, go figure...
the compiler expects even a return outside your if-else...

[feelite]

the culprit is at the while loop. if execution never enter the while loop, then there is nothing to return

[bashish]

Hi,

Ya, the problem is while loop, if it dose not enters the while loop, ur code dose not have anything to return. Add return statement after while loop.

Ashish

[kfaday]

i solved it with an unnecessary return

[jflegert]

i solved it with an unnecessary return

I'm curious, what does the C++ standard say. If I have the following code:

int foo(int i)
{
    if (i == 1)
        return 1;
    else
        return -1;

    return 0;
}

Is the "return 0;" statement needed (according the the C++ language standard)?

Thanks,
John Flegert

[TheCPUWizard]

No, in that case all paths return a value.