MVP win forms application
There are a number of typos in your code blocks, that lead me to believe the code wasn't taken from the working code in your IDE. I will assume the following:


IBaseRcovery is really IBaseRecovery
RecoveryPresenter( IBaseRecover model, ... is really RecoveryPresenter(IBaseRecovery model, ...
SqlCommand sqlCommand = new SqlCommand("BEGIN INSERT INTO recovery ; is really SqlCommand sqlCommand = new SqlCommand("BEGIN INSERT INTO recovery ...");


That said, the naming feels awkward with an underscore followed by a capital letter:


this._Model = model;
this._View = view;
this._DataService = dataService;



I would rather see this:

_model = model;
_view = view;
_dataService = dataService; 


Or that:

this.model = model;
this.view = view;
this.dataService = dataService; 


But not a mixture of both.

The raison d'être of the underscore prefix is to avoid having to use the this qualifier; if you prefix a field name with an underscore and use this, you're cluttering up your code.

Also convention is camelCase (i.e. lowercase first letter) for private fields, local variables and parameters. PascalCase (i.e. uppercase first letter) is for type and member names (classes, interfaces, delegates, enums, events, methods, etc.).



You can declare a variable without initializing it. When it's a reference type, it's set to null until you assign it a reference. Hence, this is perfectly redundant:


IBaseRecovery recovery = null;



You could just do this:

IBaseRecovery recovery;




Looks like generics could come in handy here:


public RecoveryPresenter(IBaseRecovery model, IRecoveryForm view, IDataService dataService)



Can turn into a constructor for a generic class:

public class Presenter&lt;TView, TModel&gt; 
    where TView : IView
    where TModel : class
{
    private TModel _model;
    private TView _view;
    private IRepository&lt;TModel&gt; _repository;

    public Presenter(TModel model, TView view, IRepository&lt;TModel&gt; repository)
    {
        _model = model;
        _view = view;
        _repository = repository;
    }

    /* ... */
}


If IBaseRecovery is only a marker interface with no members, get rid of it, it smells.

The key here, is IRepository&lt;TModel&gt;: your DataService has too many responsibilities, its interface will keep getting modified as you add more features to your application (i.e. as you need to deal with more database tables), and that's not very cleanly extensible.

The IRepository&lt;TModel&gt; interface could look something like this:

public interface IRepository&lt;TModel&gt; where TModel : class
{
    void SelectAll();
    void GetById(int id);
    void Insert(TModel model);
    void Update(TModel model);
    void Delete(TModel model);
}


Look into the repository pattern for more information. The idea is that you'll have a family of specialized classes, all presenting the same interface, that perform similar operations on different parts of the underlying database:

public class LoanRepository : IRepository&lt;Loan&gt;

public class FineRepository : IRepository&lt;Fine&gt;

...


You can even derive all these classes from an abstract ReposityBase class (non-generic), and implement the common code only once (i.e. connection set-up &amp; disposal, for example).

Now, if your presenter is going to be dealing with more than a single database table / repository, you have a problem; that problem is solved with a unit of work, which the presenter can depend on:

public class Presenter&lt;TView&gt; 
    where TView : IView
{
    private IUnifOfWork _uow;
    private TView _view;

    public Presenter(IUnitOfWork uow, TView view)
    {
        _uow = uow;
        _view = view;
    }

    /* ... */
}


Repository and Unit-of-Work patterns usually go hand-in-hand; here is an example of an implementation I've written in VBA recently. It's another language and an earlier data access technology, but you'll find many similarities - look at the pattern, not the code itself.

And/or read up on repository+UoW design patterns; adding these to your arsenal should greatly simplify your life.