Is it wrong to put that amount of code into a button click event?
private void CPeBTNSalvar_Click(object sender, EventArgs e)
{
try
{
if (CPeTBCodigo.Text == "")
{
throw new ExcecaoModificada("Determine um codigo para o pedido!");
}
int codigo = int.Parse(CPeTBCodigo.Text);
int ano = CpeData.Value.Year;
int mes = CpeData.Value.Month;
int dia = CpeData.Value.Day;
bool semPedido = true;
Pedido p = new Pedido(codigo, ano, mes, dia);
foreach (DataGridViewRow x in dG_CPeListarProdutos.Rows)
{
if (Convert.ToInt32(x.Cells[4].Value) > 0)
{
ItemPedido item = new ItemPedido(FormularioPrincipal.lProduto[x.Index], Convert.ToInt32(x.Cells[4].Value), Convert.ToDouble(x.Cells[3].Value));
p.itens.Add(item);
semPedido = false;
x.Cells[3].Value = 0;
x.Cells[4].Value = 0;
}
else
{
x.Cells[3].Value = 0;
}
}
if (semPedido == true)
{
throw new ExcecaoModificada("Escolha algum produto!");
}
else
{
CPeLabelAlerta.Visible = true;
CPeLabelAlerta.Text = "Novo pedido adicionado!";
CPeLabelAlerta.ForeColor = Color.DarkSlateGray;
FormularioPrincipal.lPedido.Add(p);
}
}
catch (ExcecaoModificada erro)
{
CPeLabelAlerta.Visible = true;
CPeLabelAlerta.Text = erro.Message;
CPeLabelAlerta.ForeColor = Color.Red;
}
catch (Exception)
{
CPeLabelAlerta.Visible = true;
CPeLabelAlerta.Text = "Coloque apenas numeros em código!";
}
}
If you think it's not wrong, quote something that would not do as I did.