No mezcles I/O de C y C++
Lo primero: procura no mezclar mecanismos de entrada/salida propios de C (printf
, scanf
) con los propios de C++ (cout
, cin
). La razón es que cada mecanismo tiene su propia forma de hacer las cosas.
Si en tu ejemplo funciona es porque alguien pensó que sería buena idea activar por defecto la sincronización entre ambos mecanismos. Esta sincronización es una de las razones por las que, por defecto, interacción con la consola es tan lenta en C++. Sin embargo, esta sincronización puede deshabilitarse en cualquier momento y, si esto sucede, tu programa (o tu parte de un proyecto más grande) podría empezar a comportarse de forma errática.
Para desactivar la sincronización basta con ejecutar la siguiente línea de código:
std::ios_base::sync_with_stdio(false);
Una vez ejecutada esta línea, La I/O propia de C++ (cout
, cin
) será bastante más rápida que la homóloga heredada de C (printf
, scanf
).
Ordenar ideas
En tu código hay una mezcla de diferentes enfoques y acercamientos al problema. El resultado es un código que dificilmente te va a funcionar.
Para empezar, las variables pagoE
y pagoT
no son necesarias. Para gestionar el tipo de pago solo necesitas una única variable. La razón es que tu, en principio, no sabes si el usuario va a elegir un método de pago u otro. Es justamente lo que le estás preguntando al usuario. Lo que tienes que hacer es pedirle al usuario que introduzca el método de pago (almacenas un valor en una variable) y luego tu verificas cual es el tipo de pago introducido (evalúas esa variable).
Dicho con un ejemplo:
int numero;
// Fase 1: Se almacena la opcion del usuario en una variable
std::cout << "Introduce un número par o impar: ";
std::cin << numero;
// Fase 2: Se evalua el contenido de la variable
if (numero % 2 == 0)
std::cout << "Has introducido un número par";
else:
std::cout << "Has introducido un número impar";
A partir de este punto asumiré que pagoE
se ha renombrado a pago
y que se ha eliminado la variable pagoC
:
char pago[] = {'E'};
Inicialización de arrays
char pago[] = {'E'};
Tanto en C como en C++, las cadenas de caracteres deben terminar con el caracter nulo \0
. De no ser así no podrás usar las funciones que permiten gestionar y manipular cadenas de caracteres.
En tu caso estás inicializando un array de tipo char
con un único elemento (el caracter 'E'
. Esto quiere decir que tu array de caracteres no puede ser tratado como una cadena de caracteres ya que carece del finalizador de cadena.
De la misma forma, tampoco podrías usar ese array para almacenar cadenas de caracteres ya que para ello necesitarías poder añadir, al menos, dos elementos (un caracter y el terminador de la cadena) y tu solo admites uno.
Aquí tienes dos posibles soluciones:
- Usas cadenas de caracteres
- Usas caracteres
Para usar cadenas de caracteres basta con emplear comillas dobles en vez de comillas simples:
char pago[] = "E";
Aunque, por otro lado, estás programando en C++, así que sería más cómodo para ti usar std::string
:
#include <string>
string pago = "E";
Si, en cambio, prefieres el uso de caracteres, adelante:
char pago = 'E';
Todo esto sería si necesitases inicialiar la cadena de caracteres, pero es algo que no necesitas. El valor de pago
debe darlo el usuario, no tu.
Por comodidad, yo te recomiendo usar std::string
:
std::string pago;
Ahora, ya puedes dejar que el usuario introduzca el método de pago. Una vez superado ese trámite es hora de aplicar los descuentos/incrementos correspondientes.
Llegados a este punto, no entiendo todos esos valores que le pides al usuario:
- venta -> numero
- cantidad de productos - can_pro_ven
- total ventas -> total_ven
Tu solo necesitas saber cuántas unidades se venden y el precio de cada una de ellas. El resto lo puedes calcular tu mismo (total = unidades * precio).
Por cierto, nota, que el tipo int
no admite decimales. Si tienes que aplicar descuentos/recargos que implique generar decimales, o usas tipos con decimales float
o double
o el programa te arrojará resultados incorrectos
El código final podría quedar así:
#include <iostream>
#include <string>
int main()
{
int unidades;
std::cout << "Numero de articulos: ";
std::cin >> unidades;
float precio;
std::cout << "precio unitario: ";
std::cin >> precio;
std::string tipo_pago;
std::cout << "Tipo de pago (E) efectivo o (C) credito: ";
std::cin >> tipo_pago;
float total;
if (tipo_pago == "E")
{
// Aplicas el descuento del 10%
}
else if (tipo_pago == "C")
{
// Aplicas recargo del 15%
}
else
{
// metodo de pago no valido
}
std::cout << "Total a pagar: " << total << '\n';
}
if
. Pero al ver la forma en la que lo implementas, creo que la mejor respuesta que te puedo dar es que regreses ya sea al material de clase o a la serie de tutoriales que estes siguiendo, y repases los fundamentos. No solo copies y pegues en tu editor, entiendelos. No suelo responder este tipo de preguntas ya que considero que dar una respuesta no es de mucha ayuda si no tienes los fundamentos bien cimentados.@Pablochaches
al final. Al menos estare seguro de que tendras lo necesario para que la respuesta te sea util.