Лучший способ вызвать исключения при попытке запроса c #

Question

Привет! Я разрабатываю приложение winform с использованием C # и платформы сущностей (linq to entity).Предположим, что следующий сценарий:

В методе некоторого класса я устанавливаю и объект значений со значениями формы

 private void agrega_cliente_Click(object sender, EventArgs e)
 {
        cliente = new _Cliente();

        try
        {
            cliente.nombres = nom_cliente.Text;
            cliente.apellidoP = apellidoP_cliente.Text;
            cliente.apellidoM = apellidoM_cliente.Text;
            cliente.fechaNacimiento = fechaNacimientoPicker.Value.Date;

            if (operaciones.AgregaCliente(cliente, referencias))
            {
                MessageBox.Show("Cliente Agregado");
                this.Close();
            }
        }
        catch(Exception ex)
        {
            MessageBox.Show(ex.ToString());
        }
 }

Обратите внимание, что назначение и вызов метода "AgregaCliente" находится междупопробуйте и поймайте, поэтому, если вызвано исключение, MessageBox покажет его.

Затем в другом классе у меня есть метод AgregaCliente, который вставляет значения в базу данных.

 public bool AgregaCliente(_Cliente cliente, ArrayList refes)
 {
        try
        {
            Cliente cli = new Cliente()
            {
                Nombres = cliente.nombres,
                ApellidoP = cliente.apellidoP,
                ApellidoM = cliente.apellidoM,
                FechaNac = cliente.fechaNacimiento
            };
            if (NombreExiste(cli))
                context.clientes.AddObject(cli);
            else
                throw new System.ArgumentException("El usuario ya existe");
            if (refes.Count != 0)
            {
                foreach (_Referencia elem in refes)
                    context.referencias_personales.AddObject(AgregaReferencia(elem));
            }
            context.SaveChanges();
        }
        catch (Exception ex)
        {
            return false;
        }
        return true;
 }

Внутри этого методаесть вызов "NombreExiste()", который проверяет, что пользователь еще не вставлен, если пользователь существует, генерируется исключение.

Таким образом, проблема заключается в том, что, если исключение выдается в методе "AgregaCliente"Я хочу, чтобы это исключение было перехвачено методом "agrega_cliente_Click()", чтобы пользователь знал, из-за чего возникла проблема.Я надеюсь, вы понимаете, что я пытаюсь сделать.

Спасибо

Answer 1

Просто избавьтесь от вашего try / catch внутри метода AgregaCliente (), и исключение автоматически всплывет.

public bool AgregaCliente(_Cliente cliente, ArrayList refes) 
{ 
    Cliente cli = new Cliente() 
    { 
        Nombres = cliente.nombres, 
        ApellidoP = cliente.apellidoP, 
        ApellidoM = cliente.apellidoM, 
        FechaNac = cliente.fechaNacimiento 
    }; 
    if (NombreExiste(cli)) 
        context.clientes.AddObject(cli); 
    else 
        throw new System.ArgumentException("El usuario ya existe"); 
    if (refes.Count != 0) 
    { 
        foreach (_Referencia elem in refes) 
            context.referencias_personales.AddObject(AgregaReferencia(elem)); 
    } 
    context.SaveChanges(); 

    return true; 
} 

Answer 2

Проблема в том, что ваш метод AgregaCliente () перехватывает все исключения и просто проглатывает их.Вместо того, чтобы перехватывать все исключения с помощью:

    catch (Exception ex)
    {
        return false;
    }

Вы должны перехватывать только определенные исключения, которые вы можете обработать, и позволить другим пропускать цепочку вызовов.Однако вы должны знать, что создание исключений очень «дорого» для программы.C # много работает за кулисами, когда выдается исключение.Лучшим решением может быть использование кода возврата для указания вызывающим лицам метода AgregaCliente () статуса.Например:

public enum AgregaClienteStatus
{
  Success = 0;
  ClientAlreadyExists = 1;
  Other = ??;  // Any other status numbers you want
}

 public AgregaClienteStatus AgregaCliente(_Cliente cliente, ArrayList refes)
 {

            Cliente cli = new Cliente()
            {
                Nombres = cliente.nombres,
                ApellidoP = cliente.apellidoP,
                ApellidoM = cliente.apellidoM,
                FechaNac = cliente.fechaNacimiento
            };
            if (NombreExiste(cli))
                context.clientes.AddObject(cli);
            else
                return AgregaClienteStatus.ClientAlreadyExists
            if (refes.Count != 0)
            {
                foreach (_Referencia elem in refes)
                    context.referencias_personales.AddObject(AgregaReferencia(elem));
            }
            context.SaveChanges();


        return AgregaClientStatus.Success;
 }

Конечно, эта функциональность также может быть достигнута с использованием константных целых чисел, если вам не нравятся перечисления.

Затем вы можете использовать этот статус возврата для указания информациипользователь без учета исключения:

  var result = AgregaClient(cliente, refes);
  switch (result)
  {
    case AgregaClientStatus.Success:
         // Perform success logic
         break;
    case AgregaClientStatus.ClientAlreadyExists:
         MessageBox.Show("Client already exists");
         break;
    // OTHER SPECIAL CASES
    default:
         break;
   }

}