Autor
|
Tema: Que esta mal en este codigo (Leído 6,020 veces)
|
mr.blood
Desconectado
Mensajes: 150
|
Animo! Un consejo, las variables decláralas dentro de main, no fuera, ya entenderás por qué. Y en vez de pedir los números en cada if, los pides antes de los if y te ahorras lineas. Solo tendrías que implementar un "control de errores" por si introducen una letra que no sea válida. #include <iostream> using namespace std; int main() { char a; int uno; int dos; cout<<"(S)Suma (R)Resta (M)Multiplicacion (D)Division\n\n"; cin>>a; if(a!='r' && a!='R' && a!='s' && a!='S' && a!='d' && a!='D' && a!='m' && a!='M') { cout<<"Ingresa una opcion valida\n"; return 1; } cout<<"Ingresa un numero\n\n"; cin >>uno; cout<<"Ingresa otro numero\n\n"; cin >>dos; if(a == 's'||a == 'S') { cout<<"Total Suma = \n"<< uno + dos ; } else if(a == 'r'||a == 'R') { cout<<"Total Resta = \n"<< uno - dos ; } else if(a == 'm'|| a == 'M') { cout<<"Total Multiplicacion = \n"<< uno * dos ; } else if(a == 'd'||a == 'D') { cout<<"Total Division = \n"<< uno / dos ; } return 0; }
Sa1uDoS
|
|
« Última modificación: 16 Abril 2013, 19:04 pm por mr.blood »
|
En línea
|
|
|
|
Daniel_Ruiz
Desconectado
Mensajes: 28
|
Animo! Un consejo, las variables decláralas dentro de main, no fuera, ya entenderás por qué. Y en vez de pedir los números en cada if, los pides antes de los if y te ahorras lineas. Solo tendrías que implementar un "control de errores" por si introducen una letra que no sea válida. #include <iostream> using namespace std; int main() { char a; int uno; int dos; cout<<"(S)Suma (R)Resta (M)Multiplicacion (D)Division\n\n"; if(a!='r' && a!='R' && a!='s' && a!='S' && a!='d' && a!='D' && a!='m' && a!='M') { cout<<"Ingresa una opcion valida\n"; return 1; } cin>>a; cout<<"Ingresa un numero\n\n"; cin >>uno; cout<<"Ingresa otro numero\n\n"; cin >>dos; if(a == 's'||a == 'S') { cout<<"Total Suma = \n"<< uno + dos ; } else if(a == 'r'||a == 'R') { cout<<"Total Resta = \n"<< uno - dos ; } else if(a == 'm'|| a == 'M') { cout<<"Total Multiplicacion = \n"<< uno * dos ; } else if(a == 'd'||a == 'D') { cout<<"Total Division = \n"<< uno / dos ; } return 0; }
Sa1uDoS Vale gracias Anotado y Listo ya use las etiquetas Ponganme una Tare haber si la puedo hacer
|
|
|
En línea
|
Importante
Bueno Necesito de su ayuda estoy montando una microempresa, Y estoy reuniendo Quemadoras de Dvd - Cd , Soy de Colombia Bogota si quieres apoyarme enviame un Mp Por Favor!.
|
|
|
amchacon
Desconectado
Mensajes: 1.211
|
cout<<"(S)Suma (R)Resta (M)Multiplicacion (D)Division\n\n"; if(a!='r' && a!='R' && a!='s' && a!='S' && a!='d' && a!='D' && a!='m' && a!='M') { cout<<"Ingresa una opcion valida\n"; return 1; } cin>>a;
Esto... No deberías hacer la comprobación después de introducir los datos?
|
|
« Última modificación: 16 Abril 2013, 19:13 pm por amchacon »
|
En línea
|
|
|
|
mr.blood
Desconectado
Mensajes: 150
|
Esto... No deberías hacer la comprobación después de introducir los datos?
EPIC FAIL! jajaja Lo edite a toda prisa en el propio post y ni me di cuenta, buena vista . EDITADO. Sa1uDoS
|
|
|
En línea
|
|
|
|
Daniel_Ruiz
Desconectado
Mensajes: 28
|
Bueno miren hice este pequeño programa dice la fecha en formato dia mes año #include <iostream> using namespace std; main() { int dia; int mes; int ao; cout<<"Ingrese El dia : "; cin>>dia; cout<<"Ingrese El Mes : "; cin>>mes; cout<<"Ingrese El Año : "; cin>>ao; if(mes == 1) { cout<<"Dia:"<<dia<<" Mes: Enero"<<" ano:"<<ao; } else if(mes == 2) { cout<<"Dia:"<<dia<<" Mes: Febrero"<<" ano:"<<ao; } else if(mes == 3) { cout<<"Dia:"<<dia<<" Mes: Marzo"<<" ano:"<<ao; } else if(mes == 4) { cout<<"Dia:"<<dia<<" Mes: Abril"<<" ano:"<<ao; } else if(mes == 5) { cout<<"Dia:"<<dia<<" Mes: Mayo"<<" ano:"<<ao; } else if(mes == 6) { cout<<"Dia:"<<dia<<" Mes: Junio"<<" ano:"<<ao; } else if(mes == 7) { cout<<"Dia:"<<dia<<" Mes: Julio"<<" ano:"<<ao; } else if(mes == 8) { cout<<"Dia:"<<dia<<" Mes: Agosto"<<" ano:"<<ao; } else if(mes == 9) { cout<<"Dia:"<<dia<<" Mes: Septiembre"<<" ano:"<<ao; } else if(mes == 10) { cout<<"Dia:"<<dia<<" Mes: Octubre"<<" ano:"<<ao; } else if(mes == 11) { cout<<"Dia:"<<dia<<" Mes: Noviembre"<<" ano:"<<ao; } else if(mes == 12) { cout<<"Dia:"<<dia<<" Mes: Diciembre"<<" ano:"<<ao; } else { cout<<"No ingresaste el Mes Correcto Max 12!"; } return 0; }
Y me gustaria hacer este pero no se como :s Ejercicio 308 Realizar un programa que permita elegir el formato (windows o linux) y permita escribir un determinado texto de cualquier tamaño y guardarlo en un archivo ascii con el formato especificado llamado c:\texto.txt. Los archivos de texto windows tienen como marcador de final de línea la secuencia CR (retorno de carro ascii=13) y LF (line feed ascii=10). Los de Linux solo tienen LF.
|
|
« Última modificación: 16 Abril 2013, 19:16 pm por Daniel_Ruiz »
|
En línea
|
Importante
Bueno Necesito de su ayuda estoy montando una microempresa, Y estoy reuniendo Quemadoras de Dvd - Cd , Soy de Colombia Bogota si quieres apoyarme enviame un Mp Por Favor!.
|
|
|
leosansan
Desconectado
Mensajes: 1.314
|
EPIC FAIL! jajaja Lo edite a toda prisa en el propio post y ni me di cuenta, buena vista . EDITADO. Sa1uDoS Y además darle la opción a que ingrese otro valor en lugar de "cerrarle" el programa:#include <iostream> #include <cstdlib> using namespace std; int main() { char a; int uno,dos; cout<<"(S)Suma (R)Resta (M)Multiplicacion (D)Division\n\n"; cin>>a; while(a!='r' && a!='R' && a!='s' && a!='S' && a!='d' && a!='D' && a!='m' && a!='M') { cout<<"Ingresa una opcion valida\n"; cin>>a; } cout<<"Ingresa un numero\n\n"; cin >>uno; cout<<"Ingresa otro numero\n\n"; cin >>dos; if(a == 's'||a == 'S') { cout<<"Total Suma = "<< uno + dos ; } else if(a == 'r'||a == 'R') { cout<<"Total Resta = "<< uno - dos ; } else if(a == 'm'|| a == 'M') { cout<<"Total Multiplicacion = "<< uno * dos ; } else if(a == 'd'||a == 'D') { cout<<"Total Division = "<< uno / dos ; } return EXIT_SUCCESS; }
Saluditos!. ...
|
|
|
En línea
|
|
|
|
leosansan
Desconectado
Mensajes: 1.314
|
Bueno miren hice este pequeño programa dice la fecha en formato dia mes año
Bien, aunque un poco largo.
Antes de meterte en ficheros te aconsejo practiques con bucles, switch y arrays.
Por ejemplo tu código de día, mes y año podrías abreviarlo con el uso de array:#include <iostream> #include<clocale> using namespace std; int main() { setlocale(LC_ALL, "Spanish"); char Mes [12][11]={"ENERO","FEBRERO","MARZO","ABRIL","MAYO","JUNIO","JULIO","AGOSTO","SEPTIEMBRE","OCTUBRE","NOVIEMBRE","DICIEMBRE"}; int dia; int mes; int ao; cout<<"Ingrese El día : "; cin>>dia; cout<<"Ingrese El Mes : "; cin>>mes; cout<<"Ingrese El Año : "; cin>>ao; cout<<"Día:"<<dia<<" Mes: "<<Mes [mes-1]<<" año:"<<ao; return 0; }
Ya sé que es simple, no implemento condiciones de verificación de día y mes, pero es como un simple ejemplo de uso de un array.
Por cierto, te falto "int" antes de main, ten en cuenta que retorna cero, luego main es una función de tipo int, así a lo simple.
Y el uso de la librería clocale y luego lo de "spanish" es para que en lo que se va a imprimir salgan los caracteres hispanos en lugar de símbolos raros. Permite el uso de acentos y de la "ñ", eso sí ¡cuidado¡ porque usa como caracter para los decimales la coma en lugar del punto que es más anglosajón.
Y lo dicho, mi consejo es que te centres en lo que vas viendo hasta ahora, los ficheros ya tendrán su momento.
Saluditos!. ....P.D: Sorry por no haber editado el anterior post y meter este. He metido la pata
|
|
« Última modificación: 16 Abril 2013, 20:09 pm por leosansan »
|
En línea
|
|
|
|
mr.blood
Desconectado
Mensajes: 150
|
Para que no hagas tantos if. #include <stdio.h> int main() { char listameses[][11]={"Enero", "Febrero", "Marzo", "Abril", "Mayo", "Junio", "Julio", "Agosto", "Septiembre", "Octubre", "Noviembre", "Diciembre"}; int dia, mes, anio; puts("Introduce un dia: "); puts("Introduce un mes: "); puts("Introduce un anio: "); /* Puedes saltarte el siguiente control de errores */ if(mes>12 || dia<1 || ((mes<6 && mes%2==1) && dia>31) || ((mes<6 && mes%2==0 && mes!=2) && dia>30) || (mes==2 && dia>28 && anio%4!=0) || (mes==2 && dia>29 && anio%4==0) || ((mes>6 && mes%2==0) && dia>31) || ((mes>6 && mes%2==1) && dia>31)) { puts("Introduce los datos correctamente\n"); return 1; } printf("%i de %s de %i\n", dia , listameses [mes -1], anio ); return 0; }
Sa1uDoS
|
|
|
En línea
|
|
|
|
Senior++
Desconectado
Mensajes: 957
Ama y haz lo que te de la gana
|
No hace falta usar la librería Iostream y usar la funcion cout y Cin yo personalmente prefiero usar la librería Stdio.h como ati te resulte mas comodo
|
|
|
En línea
|
Vendo cuenta de LEAGUE OF LEGENDS lvl 30, EU west, muy barata
|
|
|
mr.blood
Desconectado
Mensajes: 150
|
No hace falta usar la librería Iostream y usar la funcion cout y Cin yo personalmente prefiero usar la librería Stdio.h como ati te resulte mas comodo Pues estas usando librerias de C en C++ xD. Sa1uDoS
|
|
|
En línea
|
|
|
|
|
Mensajes similares |
|
Asunto |
Iniciado por |
Respuestas |
Vistas |
Último mensaje |
|
|
comprobar si este codigo en batch esta bien
Scripting
|
blha
|
0
|
2,305
|
11 Septiembre 2008, 16:19 pm
por blha
|
|
|
ayuda que esta mal en este codigo C
« 1 2 »
Programación C/C++
|
lupitapro
|
12
|
6,150
|
15 Junio 2010, 13:17 pm
por M3LiNdR1
|
|
|
¿Qué está mal en este código?
Desarrollo Web
|
Aikanáro Anário
|
5
|
2,560
|
15 Julio 2011, 02:53 am
por m0rf
|
|
|
necesito modificar este código de triqui esta C++ lo corro en koding linux
Programación C/C++
|
rapsolostyle
|
3
|
2,938
|
1 Octubre 2015, 04:35 am
por 0xFer
|
|
|
Este codigo esta en C/C++ o ambos?
Programación C/C++
|
Beginner Web
|
4
|
1,792
|
2 Octubre 2018, 22:34 pm
por OmarHack
|
|