[C] Problema con ejercicio

Iniciado por dmacias, 24 Febrero 2014, 16:38 PM

0 Miembros y 1 Visitante están viendo este tema.

dmacias

Hola de nuevo
Sigo estudiando c en mis ratos libres y ya voy avanzando, pero estoy teniendo problemas con un ejercicio de los primeros que estoy haciendo con switch, no termino de ver que hago mal ( que seran muchas cosas  ;D )
El ejercicio es este:



Y lo que tengo hecho luego de mil pruebas es esto otro:

/*Programa del libro, Tema 4
* ejercicio 4.19, pagina 123
* Programa de calculos de ventas usando switch*/

#include <stdio.h>

int main()
{
float a = 2.98;
float b = 4.50;
float c = 9.98;
float d = 4.49;
float e = 6.87;
int n;
int x = 0;
float total;
float a1,b1,c1,d1,e1;
float a2,b2,c2,d2,e2;

do{
printf("Ingrese el codigo de producto (-1 para salir): ");
scanf("%d", &x);
printf("Ingrese las unidades vendidas: ");
scanf("%d", &n);

while (x != -1){
switch (x){
case '1':
a1 = a*n;
a2 += a1;
printf("Llega asta aki almenos"); /*chekeo*/
break;
case '2':
b1 = b*n;
b2 += b1;
break;
case '3':
c1 = c*n;
c2 += c1;
break;
case '4':
d1 = d*n;
d2 += d1;
break;
case '5':
e1 = e*n;
e2 += e1;
break;
case '\n':
case '\t':
case ' ':
break;
default:
printf("Introdujo un codigo erroneo\n");
break;
} /*fin de switch*/
} /*fin while interno*/
} /*fin de do*/
while (x != -1); /*condicion de do*/

total = a2+b2+c2+d2+e2;
printf("El total vendido es %f \n", total);
return 0;

}


Cuando lo ejecuto, hace un ciclo infinito en printf("Itrodujo un codigo erroneo"), tal como este



Como si el valor de x no coincidiera con ningun case. La verdad que ya estoy hecho un lio  :rolleyes:

Gracias por adelantado  ::)

eferion

scanf("%d", &x);

x va a contener un número, 1, 2, 100, ...

case '1':

Aquí estás comparando x, que es un número, con '1'... que es un carácter.

La codificación ANSI tiene la mala costumbre de codificar los caracteres que representan números a partir de la posición 0x30, de tal forma que...

* '0' se codifica como 0x30
* '1' se codifica como 0x31
* '2' se codifica como 0x32

Queda claro entonces que 0 != '0'.

Si quitas las comillas simples y dejas esos case como números te debería funcionar.

dmacias

Ok, voy a probar y vengo a contar
;D

xaps

Cita de: eferion en 24 Febrero 2014, 16:42 PM
scanf("%d", &x);

x va a contener un número, 1, 2, 100, ...

case '1':

Aquí estás comparando x, que es un número, con '1'... que es un carácter.

La codificación ANSI tiene la mala costumbre de codificar los caracteres que representan números a partir de la posición 0x30, de tal forma que...

* '0' se codifica como 0x30
* '1' se codifica como 0x31
* '2' se codifica como 0x32

Queda claro entonces que 0 != '0'.

Si quitas las comillas simples y dejas esos case como números te debería funcionar.
Lo que comentas es cierto, pero el fallo del bucle infinito no es ese. El fallo es que una vez entra en el bucle while no sale nunca. ¿Porque? Fácil, no actualiza la variable x.

Por cierto, el bucle while sobra.

Saludos
"The programmers of tomorrow are the wizards of the future" - Gave Newel

dmacias

CitarLo que comentas es cierto, pero el fallo del bucle infinito no es ese. El fallo es que una vez entra en el bucle while no sale nunca. ¿Porque? Fácil, no actualiza la variable x.

Por cierto, el bucle while sobra.

Saludos

Si es verdad eso, había probado mil cosas ya y asi quedo  :rolleyes:

Solo me viene un ejemplo de codigo en el libro que estoy siguiendo y en el usan caracteres, no caí en que usaba las comillas por eso  >:(

Al final ha terminado asi :


/*Programa del libro, Tema 4
* ejercicio 4.19, pagina 123
* Programa de calculos de ventas usando switch*/

#include <stdio.h>

int main()
{
float a = 2.98;
float b = 4.50;
float c = 9.98;
float d = 4.49;
float e = 6.87;
int n;
int x = 0;
float total;
float a1,b1,c1,d1,e1;
float a2,b2,c2,d2,e2;

do{
printf("Ingrese el codigo de producto (-1 para salir): ");
scanf("%d", &x);
printf("Ingrese las unidades vendidas: ");
scanf("%d", &n);

if (x != -1){
switch (x){
case 1:
a1 = a*n;
a2 += a1;
break;
case 2:
b1 = b*n;
b2 += b1;
break;
case 3:
c1 = c*n;
c2 += c1;
break;
case 4:
d1 = d*n;
d2 += d1;
break;
case 5:
e1 = e*n;
e2 += e1;
break;
case '\n':
case '\t':
case ' ':
break;
default:
printf("Introdujo un codigo erroneo\n");
break;
} /*fin de switch*/
} /*fin if*/
} /*fin de do*/
while (x != -1); /*condicion de do*/

total = a2+b2+c2+d2+e2;
printf("El total vendido es %f \n", total);
return 0;

}


Ya solo me queda evitar que tras introducir -1 del valor centinela no me pida la cantidad de productos. Sigo al lio con el, al final aprendere y todo  ;D



Gracias a los dos por la ayuda

Un saludo

eferion

Imagino que evitar que te pregunte por el número de unidades es algo con lo que podrás lidiar tu solo, no?? :)

dmacias

Solo vengo a por ayuda cuando ya no doy mas de mi, asi para la proxima me acuerdo  ;-)

Deja que me acabe el café y sigo liado con el, sale (si || si ||  le prendo fuego) xD .


Un saludo

dmacias

Bueno, aqui os vengo con el codigo final creo, almenos me funciona como creo que debe ser.

Al final he inicializado las variables "x2" por que algunas pruebas me daban resultados erroneos, creo que ese podia ser el problema.

Seguire haciendole pruebas haber si falla.




/*Programa del libro, Tema 4
* ejercicio 4.19, pagina 123
* Programa de calculos de ventas usando switch*/

#include <stdio.h>

int main()
{
float a = 2.98;
float b = 4.50;
float c = 9.98;
float d = 4.49;
float e = 6.87;
int n = 0;
int x = 0;
float total;
float a1,b1,c1,d1,e1;
float a2,b2,c2,d2,e2;

do{
printf("Ingrese el codigo de producto (-1 para salir): ");
scanf("%d", &x);
if (x >= 1 && x <= 5){
printf("Ingrese las unidades vendidas: ");
scanf("%d", &n);
if (x != -1){
}
}
switch (x){
case 1:
a1 = a*n;
a2 += a1;
break;
case 2:
b1 = b*n;
b2 += b1;
break;
case 3:
c1 = c*n;
c2 += c1;
break;
case 4:
d1 = d*n;
d2 += d1;
break;
case 5:
e1 = e*n;
e2 += e1;
break;
case -1:
break;
case '\n':
case '\t':
case ' ':
break;
default:
printf("Introdujo un codigo erroneo\n");
break;
} /*fin de switch*/
} /*fin de do*/
while (x != -1); /*condicion de do*/

total = a2+b2+c2+d2+e2;
printf("\nEl total vendido es %f \n", total);
return 0;

}




Un saludo y muchas gracias por la ayuda  ;-)

amchacon

Código (cpp) [Seleccionar]
if (x != -1){
}

¿Para que es ese if? :huh:
Por favor, no me manden MP con dudas. Usen el foro, gracias.

¡Visita mi programa estrella!

Rar File Missing: Esteganografía en un Rar

dmacias

Cita de: amchacon en 24 Febrero 2014, 21:53 PM
Código (cpp) [Seleccionar]
if (x != -1){
}

¿Para que es ese if? :huh:
La verdad, no lo se, solo supuse que como el anterior if filtra los mayores de 1 y menores de 5, pense que cuando metiera -1 del centinerla me mandaria al cuerno pero no lo he probado sin el.

Por cierto, el codigo de arriba no es el final, lo tuve que copiar mal o al pegarlo algo falló.
es este

/*Programa del libro, Tema 4
* ejercicio 4.19, pagina 123
* Programa de calculos de ventas usando switch*/

#include <stdio.h>

int main()
{
float a = 2.98;
float b = 4.50;
float c = 9.98;
float d = 4.49;
float e = 6.87;
int n = 0;
int x = 0;
float total;
float a1,b1,c1,d1,e1;
float a2,b2,c2,d2,e2;
a2=b2=c2=d2=e2= 0;
do{
printf("Ingrese el codigo de producto (-1 para salir): ");
scanf("%d", &x);
if (x >= 1 && x <= 5){
printf("Ingrese las unidades vendidas: ");
scanf("%d", &n);
}
switch (x){
case 1:
a1 = a*n;
a2 += a1;
break;
case 2:
b1 = b*n;
b2 += b1;
break;
case 3:
c1 = c*n;
c2 += c1;
break;
case 4:
d1 = d*n;
d2 += d1;
break;
case 5:
e1 = e*n;
e2 += e1;
break;
case -1:
break;
case '\n':
case '\t':
case ' ':
break;
default:
printf("Introdujo un codigo erroneo\n");
break;
} /*fin de switch*/
} /*fin de do*/
while (x != -1); /*condicion de do*/

total = a2+b2+c2+d2+e2;
printf("\nEl total vendido es %f \n", total);
return 0;

}



Sin esta linea, como comento en el comentario de arriba, el resultado a veces no es correcto

a2=b2=c2=d2=e2= 0;

Probare a comentar el if que me dices, haber que hace.

Lo edito sobre la marcha, funciona igual sin el if que comenta amchacon