not all control paths return a value

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

Code:

---

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

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

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

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

i solved it with an unnecessary return

Quote:

---

i solved it with an unnecessary return

---

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

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

No, in that case all paths return a value.