private void bttnAddInventary_Click(object sender, EventArgs e)
{
if (txtProductNameInventary.Text.Equals("") &&
txtDescriptionInventary.Text.Equals("") &&
txtPriceInventary.Text.Equals("") &&
txtStockInventary.Text.Equals(""))
{
MessageBox.Show("No puede dejar campos vacios");
}
else
{
ControladorInventario.AnadirProducto(txtProductNameInventary.Text, txtDescriptionInventary.Text,
txtPriceInventary.Text, txtStockInventary.Text);
ActualizarInventario();
}
}
private void bttnAddInventary_Click(object sender, EventArgs e)
{
if (txtProductNameInventary.Text.Equals("") &&
txtDescriptionInventary.Text.Equals("") &&
txtPriceInventary.Text.Equals("") &&
txtStockInventary.Text.Equals(""))
MessageBox.Show("No puede dejar campos vacios");
else
{
//Convertir a un double, y a un entero
ControladorInventario.AnadirProducto(txtProductNameInventary.Text, txtDescriptionInventary.Text,
Convert.ToDouble(txtPriceInventary.Text), Convert.ToInt32(txtStockInventary.Text));
ActualizarInventario();
}
}
private void bttnAddInventary_Click(object sender, EventArgs e)
{
if (txtProductNameInventary.Text.Equals("") || // se cambió los && por || porque esta es la validación correcta
txtDescriptionInventary.Text.Equals("") ||
txtPriceInventary.Text.Equals("") ||
txtStockInventary.Text.Equals(""))
{
MessageBox.Show("No puede dejar campos vacios");
}
else
{
ControladorInventario.AnadirProducto(txtProductNameInventary.Text, txtDescriptionInventary.Text,
Convert.ToDecimal(txtPriceInventary.Text), Convert.ToInt32(txtStockInventary.Text)); // se convirtió el precio y stock en su respectivo tipo
ActualizarInventario();
}
}
//Correción: Se cambiaron los operadores && por ||, porque si no, solo se cumpliria la condicion cuando todos estan vacios.
private void bttnAddInventary_Click(object sender, EventArgs e)
{
if (txtProductNameInventary.Text.Equals("") ||
txtDescriptionInventary.Text.Equals("") ||
txtPriceInventary.Text.Equals("") ||
txtStockInventary.Text.Equals(""))
{
MessageBox.Show("No puede dejar campos vacios");
}
else
{ //Corrección: Se detecto una mala practica de programacion sobrepasandose la linea.
ControladorInventario.AnadirProducto(txtProductNameInventary.Text,
txtDescriptionInventary.Text,
txtPriceInventary.Text, txtStockInventary.Text);
ActualizarInventario();
}
}
private void bttnAddInventary_Click(object sender, EventArgs e)
{
//CORRECION: se cambiaron las && por || en la condicion para evitar que se dejen campos vacios.
if (txtProductNameInventary.Text.Equals("") ||
txtDescriptionInventary.Text.Equals("") ||
txtPriceInventary.Text.Equals("") ||
txtStockInventary.Text.Equals(""))
{
MessageBox.Show("No puede dejar campos vacios");
}
else
{
ControladorInventario.AnadirProducto(txtProductNameInventary.Text, txtDescriptionInventary.Text,
txtPriceInventary.Text, txtStockInventary.Text);
ActualizarInventario();
}
}
private void bttnAddInventary_Click(object sender, EventArgs e)
{
//Corrección: Se cambió AND por OR como argumento de if
if (txtProductNameInventary.Text.Equals("") ||
txtDescriptionInventary.Text.Equals("") ||
txtPriceInventary.Text.Equals("") ||
txtStockInventary.Text.Equals(""))
{
MessageBox.Show("No puede dejar campos vacios");
}
else
{
ControladorInventario.AnadirProducto(txtProductNameInventary.Text, txtDescriptionInventary.Text,
txtPriceInventary.Text, txtStockInventary.Text);
ActualizarInventario();
}
}
private void bttnAddInventary_Click(object sender, EventArgs e)
{
//Se debe cambiar los && por || debido a que solo entraría al if en dado caso se dejen todos los campos vacíos, pero es necesario entrar en caso deje al menos uno, para lo cual nos beneficia el ||.
if (txtProductNameInventary.Text.Equals("") ||
txtDescriptionInventary.Text.Equals("") ||
txtPriceInventary.Text.Equals("") ||
txtStockInventary.Text.Equals(""))
{
MessageBox.Show("No puede dejar campos vacios");
}
else
{
ControladorInventario.AnadirProducto(txtProductNameInventary.Text, txtDescriptionInventary.Text,
txtPriceInventary.Text, txtStockInventary.Text);
ActualizarInventario();
}
}
private void bttnAddInventary_Click(object sender, EventArgs e)
{
//Correccion: cambiando and por or para que ningun campo se pueda dejar vacio
if (txtProductNameInventary.Text.Equals("") ||
txtDescriptionInventary.Text.Equals("") ||
txtPriceInventary.Text.Equals("") ||
txtStockInventary.Text.Equals(""))
{
MessageBox.Show("No puede dejar campos vacios");
}
else
{
ControladorInventario.AnadirProducto(txtProductNameInventary.Text, txtDescriptionInventary.Text,
txtPriceInventary.Text, txtStockInventary.Text);
ActualizarInventario();
}
}
// correcion: error en ves de && ponemos || ya que con && se estaria cumpliendo solo si es vacia todas y queremos saber
//si es vacia alguna
private void bttnAddInventary_Click(object sender, EventArgs e)
{
if (txtProductNameInventary.Text.Equals("") ||
txtDescriptionInventary.Text.Equals("") ||
txtPriceInventary.Text.Equals("") ||
txtStockInventary.Text.Equals(""))
{
MessageBox.Show("No puede dejar campos vacios");
}
else
{
//correcion: mala practica es sobrepasarse de la linea lo menciono walter en un labo
ControladorInventario.AnadirProducto(txtProductNameInventary.Text,
txtDescriptionInventary.Text,
txtPriceInventary.Text, txtStockInventary.Text);
ActualizarInventario();
}
}
//CORRECCION: Se cambiaron los operadores && por ||. Con && estaría cumpliéndose solo si todos fueran vacíos.
private void bttnAddInventary_Click(object sender, EventArgs e)
{
if (txtProductNameInventary.Text.Equals("") ||
txtDescriptionInventary.Text.Equals("") ||
txtPriceInventary.Text.Equals("") ||
txtStockInventary.Text.Equals(""))
{
MessageBox.Show("No puede dejar campos vacios");
}
else
{
//CORRECCION: Alinear codigo y que no pase de la linea, para evitar mala practica.
ControladorInventario.AnadirProducto(txtProductNameInventary.Text,
txtDescriptionInventary.Text,
txtPriceInventary.Text, txtStockInventary.Text);
ActualizarInventario();
}
}
private void bttnAddInventary_Click(object sender, EventArgs e)
{
// cambiar operadores && por || para que valide que ningun campo este vacio
if (txtProductNameInventary.Text.Equals("") ||
txtDescriptionInventary.Text.Equals("") ||
txtPriceInventary.Text.Equals("") ||
txtStockInventary.Text.Equals(""))
{
MessageBox.Show("No puede dejar campos vacios");
}
else
{
//comvertir a entero txtStockInventary.Text
int stock = Convert.ToInt32(txtStockInventary.Text);
ControladorInventario.AnadirProducto(txtProductNameInventary.Text, txtDescriptionInventary.Text,
txtPriceInventary.Text, stock);
ActualizarInventario();
}
}
private void bttnAddInventary_Click(object sender, EventArgs e)
{
//En este if al poner && solo estaria veriricando que todos los campos esten parcialmente llenos y si hay uno vaío nos saltaría error
//pero no tomaría acción mientras que al cambiarlo por un || nos saltaría el mensaje de que todos los campos deben de estar llenos
//siendo mas óptimo ocupar || sobre &&
if (txtProductNameInventary.Text.Equals("") ||
txtDescriptionInventary.Text.Equals("") ||
txtPriceInventary.Text.Equals("") ||
txtStockInventary.Text.Equals(""))
{
MessageBox.Show("No se puede dejar campos vacios");
}
else
{
ControladorInventario.AnadirProducto(txtProductNameInventary.Text, txtDescriptionInventary.Text,
txtPriceInventary.Text, txtStockInventary.Text);
ActualizarInventario();
}
}
private void bttnAddInventary_Click(object sender, EventArgs e)
{
//5 Correcion: Si almenos uno esta vacio debe saltar el mensaje, no si todos estan vacios.
if (txtProductNameInventary.Text.Equals("") ||
txtDescriptionInventary.Text.Equals("") ||
txtPriceInventary.Text.Equals("") ||
txtStockInventary.Text.Equals(""))
{
MessageBox.Show("No puede dejar campos vacios");
}
else
{
//9 Correcion: Buenas practicas (pasa la linea)
ControladorInventario.AnadirProducto(txtProductNameInventary.Text,
txtDescriptionInventary.Text,
txtPriceInventary.Text, txtStockInventary.Text);
ActualizarInventario();
}
}
private void bttnAddInventary_Click(object sender, EventArgs e)
{
//#4: tienen que cotenplarse el caso de que alguno este vacio no todos
if (txtProductNameInventary.Text.Equals("") ||
txtDescriptionInventary.Text.Equals("") ||
txtPriceInventary.Text.Equals("") ||
txtStockInventary.Text.Equals(""))
{
MessageBox.Show("No puede dejar campos vacios");
}
else
{
//#5: identacion
ControladorInventario.AnadirProducto(txtProductNameInventary.Text,
txtDescriptionInventary.Text,
txtPriceInventary.Text, txtStockInventary.Text);
ActualizarInventario();
}
}
private void bttnAddInventary_Click(object sender, EventArgs e)
{
if (txtProductNameInventary.Text.Equals("") ||//Corrección: Se cambiaron operadores lógicos && por ||
txtDescriptionInventary.Text.Equals("") ||
txtPriceInventary.Text.Equals("") ||
txtStockInventary.Text.Equals(""))
{
MessageBox.Show("No puede dejar campos vacios");
}
else
{
ControladorInventario.AnadirProducto(txtProductNameInventary.Text, txtDescriptionInventary.Text,
txtPriceInventary.Text, txtStockInventary.Text);
ActualizarInventario();
txtProductNameInventary.Clear(); //Corrección estética: Se agregó método '.Clear()' para limpiar
txtDescriptionInventary.Clear(); //las textBox después de agregar el producto
txtPriceInventary.Clear();
txtStockInventary.Clear();
}
}
private void bttnAddInventary_Click(object sender, EventArgs e)
{
//Correpcion:cambiando de string al varibale double e int a stock y precio
int stock = 0;
double precio = 0;
precio = Convert.ToDouble(txtPriceInventary.Text);
stock = Convert.ToInt32(txtStockInventary.Text);
if (txtProductNameInventary.Text.Equals("") ||
txtDescriptionInventary.Text.Equals("") ||
txtPriceInventary.Text.Equals("") ||
txtStockInventary.Text.Equals(""))
{
MessageBox.Show("No puede dejar campos vacios");
}
else
{
ControladorInventario.AnadirProducto(txtProductNameInventary.Text
, txtDescriptionInventary.Text, precio, stock);
ActualizarInventario();
}
}