[C] Problema con ejercicio

[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   )
El ejercicio es este:



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

Código (c) [Seleccionar] Expandir

/*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  

Gracias por adelantado  

[eferion]

Código (c) [Seleccionar] Expandir

scanf("%d", &x);

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

Código (c) [Seleccionar] Expandir

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

[xaps]

Código (c) [Seleccionar] Expandir

scanf("%d", &x);

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

Código (c) [Seleccionar] Expandir

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

[dmacias]

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

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

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 :

Código (c) [Seleccionar] Expandir


/*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 



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.

Código (c) [Seleccionar] Expandir


/*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] Expandir

if (x != -1){
}
¿Para que es ese if?

[dmacias]

Código (cpp) [Seleccionar] Expandir

if (x != -1){
}
¿Para que es ese if?

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

Código (c) [Seleccionar] Expandir


/*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

Código (c) [Seleccionar] Expandir

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