[SOLUCIONADO] bucle while mas sentencia if else duda

[jscmanuel]

Código:

#include <iostream>
using namespace std;

int main(){
	float numero;
	float resultado;
	bool no_valida = false;
	
	while ( !no_valida ){
	cout<<"Indica que numero quieres calcular: "<<endl;
	cin>>numero;
	resultado = numero*numero;
	
	if (numero != 0)
	   cout<<"El cuadrado de: "<<numero<<" es: "<<resultado<<endl;
	   //no_valida = true;
	   
	   else if (numero == 0)
	     cout<<"Error, Opcion no valida"<<endl;
	     no_valida = false;
	 }
return 0;
}

hola mi duda es por que cuando activo en la sentencia if la variable bool no_valida a true el programa me da erro en el else if,

Lo que quiero es que el programa se ejecute una vez si introduces un numero distinto de 0 y se cierre , si el numero es igual a cero que haga el bucle .
El programa tal como esta ahora hace el bucle metas lo que metas.

gracias de ante mano , un saludo

[dehm]

Te falta englobar con llaves lo que sigue a if y a else.
Tal como lo tienes, la sentencia:

Código C++:

no_valida = false;
 

se ejecutará siempre.

[jscmanuel]

#include <iostream>
using namespace std;

int main(){
float numero;
float resultado;
bool no_valida = false;

while ( !no_valida ){
cout<<"Indica que numero quieres calcular: "<<endl;
cin>>numero;
resultado = numero*numero;

if (numero != 0){
cout<<"El cuadrado de: "<<numero<<" es: "<<resultado<<endl;
no_valida = true;}

else if (numero == 0){
cout<<"Error, Opcion no valida"<<endl;
no_valida = false;}

}
return 0;
}
gracias dehm por la ayuda , ya lo e entendido , dejo el código por si alguien quiero verlo.

[dehm]

De nada.
Por cierto, te puedes ahorrar la última sentencia de

Código C++:

no_valida=false;
 

ya que no está actualizando ningún valor. Sólamente te interesa que si el número es !=0, poner "no_valida=true", como tienes puesto en la parte del "if" para salir del bucle.
Saludos!

[jscmanuel]

gracias dehm , por el sugerencia. todo sea por optimizar el código. :)