problema con sentencia switch

[++c]

Hola amigos,

tengo un problema a la hora de ejecutar adecuadamente esta sentencia y es a la hora de cuando se introduce en un caso no sale de él, no entiendo porque break no rompe cuando termina la ejecución de la instrucción.

Deseo que el usuario introduzca una opción, seguidamente introduzca un valor, se realice su cálculo de venta de ese producto y por último que muestre el total de venta de todos los productos existentes en la tienda.

También observo que no obedece cuando se introduce un carácter no válido metiéndose en un bucle infinito no acertando porque....

Código (c) [Seleccionar] Expandir



#include <stdio.h>

int main(){

    int opcion;
    float cantidad,total=0;

    printf("En esta tienda se vende 5 productos con los siguientes\n");
    printf("\nPrecios:");
    printf("\nProducto 1: 2,98$");
    printf("\nProducto 2: 4,50$");
    printf("\nProducto 3: 9,98$");
    printf("\nProducto 4: 4,49$");
    printf("\nProducto 5: 6,87$\n");

    printf("\nIntroduzca numero de producto. -1 para salir: ");
    scanf("%d",&opcion);

    while( opcion != -1){

        switch( opcion ){

        case 1:

            printf("\nIntroduzca cantidad vendida: ");
            scanf("%f",&cantidad);
            printf("El precio de esta venta es: %f",cantidad*2.98);
            total+=cantidad*=2.98;
            break;

        case 2:

            printf("\nIntroduzca cantidad vendida: ");
            scanf("%f",&cantidad);
            printf("El precio de esta venta es: %f",cantidad*4.50);
            total+=cantidad*=4.50;
            break;

        case 3:

            printf("\nIntroduzca cantidad vendida: ");
            scanf("%f",&cantidad);
            printf("El precio de esta venta es: %f",cantidad*9.98);
            total+=cantidad*=9.98;
            break;

        case 4:

            printf("\nIntroduzca cantidad vendida: ");
            scanf("%f",&cantidad);
            printf("El precio de esta venta es: %f",cantidad*4.49);
            total+=cantidad*=4.49;
            break;

        case 5:

            printf("\nIntroduzca cantidad vendida: ");
            scanf("%f",&cantidad);
            printf("El precio de esta venta es: %f",cantidad*6.87);
            total+=cantidad*=6.87;
            break;

        default:

            printf("\nIntroduzca numero de producto VALIDO: ");
            break;

        }
    }
    printf("\nLas ventas totales fueron: %f\n\n ",total);
    return 0;
}


Cualquier aclaración se lo agradecería.

Saludos y muchas gracias

++c

[amchacon]

Código (cpp) [Seleccionar] Expandir

printf("\nIntroduzca numero de producto. -1 para salir: ");
scanf("%d",&opcion);

Esas dos líneas deberían ir dentro del while, no fuera.

[++c]

Hola amchacon gracias por la respuesta, en efecto ese era el problema.

Me gustaría conocer un último matiz que aún no he llegado a aprender y espero conocer adelatándome en materia y es en relación a optimizar el menu de selección cuando se introduce otro carácter distinto a lo estipulado iniciando un bucle infinito.

Dejo el código como lo llevo elaborado:

Código (c) [Seleccionar] Expandir


#include <stdio.h>

int main(){

    int opcion;
    float cantidad,total=0;

    printf("En esta tienda se vende 5 productos con los siguientes precios:\n");
    printf("\nProducto 1: 2,98$");
    printf("\nProducto 2: 4,50$");
    printf("\nProducto 3: 9,98$");
    printf("\nProducto 4: 4,49$");
    printf("\nProducto 5: 6,87$\n");

    while( opcion != -1){

        printf("\nIntroduzca numero de producto. -1 para salir: ");
        scanf("%d",&opcion);

        switch( opcion ){

        case 1:

            printf("\nIntroduzca cantidad vendida: ");
            scanf("%f",&cantidad);
            printf("El precio de esta venta es: %f",cantidad*2.98);
            total+=cantidad*=2.98;
            break;

        case 2:

            printf("\nIntroduzca cantidad vendida: ");
            scanf("%f",&cantidad);
            printf("El precio de esta venta es: %f",cantidad*4.50);
            total+=cantidad*=4.50;
            break;

        case 3:

            printf("\nIntroduzca cantidad vendida: ");
            scanf("%f",&cantidad);
            printf("El precio de esta venta es: %f",cantidad*9.98);
            total+=cantidad*=9.98;
            break;

        case 4:

            printf("\nIntroduzca cantidad vendida: ");
            scanf("%f",&cantidad);
            printf("El precio de esta venta es: %f",cantidad*4.49);
            total+=cantidad*=4.49;
            break;

        case 5:

            printf("\nIntroduzca cantidad vendida: ");
            scanf("%f",&cantidad);
            printf("El precio de esta venta es: %f",cantidad*6.87);
            total+=cantidad*=6.87;
            break;

        }
    }
    printf("\nLas ventas totales fueron: %f\n\n ",total);
    return 0;
}



Saludos y gracias por ayudar a personas que se inician en el mundo de la programación.