Hola de nuevo comunidad,
estoy aprendiendo entity framework y hago un ejercicio donde guardo un expediente con x documentos y ese documento tiene x imágenes.
Al querer insertar nuevos documentos a un expediente ya existente, tengo el siguiente error:
An unhandled exception of type 'System.ObjectDisposedException' occurred in EntityFramework.dll
Additional information: The ObjectContext instance has been disposed and can no longer be used for operations that require a connection.
Tengo un formulario en donde listo los expedientes, luego al seleccionar uno y dar clic a editar, paso el expediente seleccionado a una nueva ventana de detalle, así:
Detalles exp = new Detalles(bExpedientes[index]);
bExpedientes es una binding list que contiene todos los expedientes existentes, la lleno así:
public static List<Expediente> GetExpedientes()
{
Entities context = new Entities();
List<Expediente> exp = new List<Expediente>();
using (context)
{
try
{
exp = context.Expediente.ToList();
}
catch (Exception)
{
return null;
}
return exp;
}
}
Luego, en detalles, tengo un botón para agregar nuevos documentos así:
expediente.Documentos.Add(new Documento { NombreDocumento = txtNomDoc.Text, FechaCreacion = DateTime.Now, idExpediente = idExpediente })
donde expediente, es una instancia de la clase Expediente generada por EF. ahí es donde entra el problema, al ejecutar esa linea tira la excepción antes mencionada. Tengo entendido que puede ser por el using que pongo al extraer los expedientes. Pero alguna idea de como solucionarlo?
Luego guardo todos esos documentos nuevos en una binding list llamada bDocumentos, así:
Doc = expediente.Documentos.ToList();
dgvDocumentos.AutoGenerateColumns = false;
bDocumentos = new BindingList<Documento>(Doc);
dgvDocumentos.DataSource = bDocumentos;
para luego al dar clic a un botón guardar, pasar las listas a una función que hará el guardado. Cuando inserto nuevos Expedientes con nuevos documentos y nuevas imágenes no tengo problemas. Pero al agregar nuevos documentos a un expediente existente si!
Lo que hace el botón guardar es lo siguiente:
General.GuardarDatos(bDocumentos, bArchivo);
Aquí, paso las listas. La de archivo por el momento no es relevante. El metodo GuardarDatos hace lo siguiente:
public static void GuardarDatos(Expediente expediente,BindingList<Documento> documentos, BindingList<Archivo> archivos)
{
Entities context = new Entities();
using (context)
{
foreach(var documento in documentos)
{
if(documento.idDocumento == 0)
{
context.Documento.Add(documento);
}
else
{
DocumentoNG.actualizarDocumento(documento);
}
foreach(var archivo in archivos)
{
if (documento.idDocumento == 0)
{
context.Archivo.Add(archivo);
}
else
{
ArchivoNG.actualizarArchivo(archivo);
}
}
}
context.SaveChanges();
}
}
Si quito el using, el error lo tira al ejecutar la linea actualizarDocumento sin embargo es otro es decir:
An unhandled exception of type 'System.InvalidOperationException' occurred in EntityFramework.dll
Additional information: An entity object cannot be referenced by multiple instances of IEntityChangeTracker.
y ese metodo lo que hace es lo siguiente:
public static void actualizarDocumento(Documento documento)
{
Entities context = new Entities();
using (context)
{
context.Documento.Attach(documento);
var registro = context.Entry(documento);
registro.Property(t => t.NombreDocumento).IsModified = true;
context.SaveChanges();
}
}
La entidad expediente es así:
public partial class Expediente
{
[System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Usage", "CA2214:DoNotCallOverridableMethodsInConstructors")]
public Expediente()
{
this.Documentos = new HashSet<Documento>();
}
public int idExpediente { get; set; }
public string NombreExpediente { get; set; }
public System.DateTime FechaCreacion { get; set; }
public string DuenioExpediente { get; set; }
public int CantidadDocumento { get; set; }
[System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Usage", "CA2227:CollectionPropertiesShouldBeReadOnly")]
public virtual ICollection<Documento> Documentos { get; set; }
}
La de documento es así:
public partial class Documento
{
[System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Usage", "CA2214:DoNotCallOverridableMethodsInConstructors")]
public Documento()
{
this.Archivo = new HashSet<Archivo>();
}
public int idDocumento { get; set; }
public int idExpediente { get; set; }
public string NombreDocumento { get; set; }
public System.DateTime FechaCreacion { get; set; }
[System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Usage", "CA2227:CollectionPropertiesShouldBeReadOnly")]
public virtual ICollection<Archivo> Archivo { get; set; }
public virtual Expediente Expediente { get; set; }
}
Gracias de ante mano.
catch
no es buena idea.null
e ignorar la excepción quje salta no es una buena idea. Esto solo resultará en unNullReferenceException
mas adelante en tu código sin tener los detalles de la excepción original que causó tu problema. Mejor es quitar eltry-catch
, y si salta una excepción, pues que salte. Por lo menos no habrá misterio en cuanto a la causa del problema y como corregirlo.