I would do something like this
public class FormFactory
{
readonly ILifetimeScope scope;
public FormFactory(ILifetimeScope scope)
{
this.scope = scope;
}
public TForm CreateForm<TForm>() where TForm : Form
{
var formScope = scope.BeginLifetimeScope("FormScope");
var form = formScope.Resolve<TForm>();
form.Closed += (s, e) => formScope.Dispose();
return form;
}
}
Register your ISession
as InstancePerLifetimeScope
and Autofac will dispose of it when its scope is disposed. In this example, I am using the "FormScope" tag so that if I accidentally try to resolve an ISession
out of another scope (maybe the top-level container scope) Autofac will throw an exception.
builder.Register(c => SomeSessionFactory.OpenSession())
.As<ISession>()
.InstancePerMatchingLifetimeScope("FormScope");
Your code will have to commit the transaction explicitly (probably when the user clicks Save or something), and it probably should rollback the transaction if the user clicks Cancel. Implicit rollback is not recommended.