Saludos chicos abro este tema solo para ver sus opiniones sobre este código que hice y saber como lo hubieran hecho ustedes acá el código.....
/*
Name: Mostrar_talbas_matematicas
Author: H4ckw1X
Date: 18/03/17 14:35
*/
#include <iostream>
using namespace std;
int main(){
int menu,i,t,result;
int respuesta=1;
while(respuesta!=0){
cout<<"\n*************** Que tabla matematica quiere revisar ***************"<<endl;
cout<<"\n--MENU--"<<endl;
cout<<"\n1.SUMA"<<endl;
cout<<"\n2.RESTA"<<endl;
cout<<"\n3.MULTIPLICACION"<<endl;
cout<<"\n4.DIVICION"<<endl;
cin>>menu;
switch(menu){
case 1:
if(menu=1);
cout<<"\nEscribre el numero de la tabla"<<endl;
cin>>t;
for(i=0;i<=12;i++){
result=t+i;
cout<<t<<" + "<<i<<" = "<<result<<endl;
}
break;
case 2:
if(menu=2);
cout<<"\nEscribre el numero de la tabla"<<endl;
cin>>t;
for(i=0;i<=12;i++){
result=t-i;
cout<<t<<" - "<<i<<" = "<<result<<endl;
}
break;
case 3:
if(menu=3);
cout<<"\nEscribre el numero de la tabla"<<endl;
cin>>t;
for(i=0;i<=12;i++){
result=t*i;
cout<<t<<" * "<<i<<" = "<<result<<endl;
}
break;
case 4:
if(menu=4);
cout<<"\nEscribre el numero de la tabla"<<endl;
cin>>t;
for(i=0;i<=12;i++){
result=t/i;
cout<<t<<" / "<<i<<" = "<<result<<endl;
}
break;
default:
cout<<"\nOpcion invalida"<<endl;
break;
}
cout<<"Desea ver otra tabla s.1/n.0"<<endl;
cin>>respuesta;
}
return 0;
}
Hola
Así de primeras, estas usando un switch para las opciones y dentro un if, sobra ese if, además, las comprobaciones en el if es con == (doble signo igual), tu estás haciendo una asignación.
Un saludo
el caso 4 será poco efectivo
si lo ejecutas te darás cuenta a lo que me refiero
para que se vea bien, result debería ser de tipo float, castear como float la división, debe ser i/t y no t/i
procura hacer un poco mejor la identación
tu haces
int main(){
int menu,i,t,result;
int respuesta=1;
while(respuesta!=0){
cout<<"\n*************** Que tabla matematica quiere revisar ***************"<<endl;
/* ... */
cin>>menu;
switch(menu){
case 1:
if(menu=1);
cout<<"\nEscribre el numero de la tabla"<<endl;
cin>>t;
for(i=0;i<=12;i++){
yo creo que sería más legible
no ideantas dentro del while, ni el for
int main(){
int menu,i,t,result;
int respuesta=1;
while(respuesta!=0){
cout<<"\n*************** Que tabla matematica quiere revisar ***************"<<endl;
/* ... */
cin>>menu;
switch(menu){
case 1:
if(menu=1);
cout<<"\nEscribre el numero de la tabla"<<endl;
cin>>t;
for(i=0;i<=12;i++){
result=t+i;
por otro lado los if de las lineas 28, 37,46 y 55, no tienen sentido y en general están mal formulados (usan "=" en lugar de "==") a demás terminan en ";" lo que los finaliza apenas se evalúan quitandole todo el sentido
para mi es preferible correr sobre seguro y el while de la linea 16 no lo haría != 0 sino == 1, porque si la persona marca cualquier cosa que no sea 0 se repetirá y eso no sería correcto (para mi), desde mi punto de vista debería repetirse si y solo si es 1, que es la condición de repetición... pero es decisión de cada quien eso
Gracias por sus respuestas me son de gran ayuda soy nuevo en el mundo de la programación :D :D :D :D ;-) ;-)
Hola:
El Switch sustituye el if, para no poner tantos if, se pone Switch. ;)
https://msdn.microsoft.com/es-es/library/k0t5wee3.aspx
[youtube=640,360]https://www.youtube.com/watch?v=G50bwqLapaM[/youtube]
Por cierto. ¿Qué IDE usas?
Recuerda, no es lomismo IDE que compilador. ;)
Saludos.