Плохая практика использования в подписи метода DbCommand?

Question

Я использую DbCommand из EnterpriseLibrary. У меня есть два метода, которые получают тот же форматированный выбор из базы данных. Таким образом, DataReader выглядит так же, но я выполняю другие процедуры.

Мой вопрос: является ли это хорошей идеей является перейти к одному методу исполняющего DbCommand и чтения данных из DataReader, как это:

public Dictionary<Guid,List<string>> GetCurrentLoginData() 
{ 
    // here is maked command and private method is called 
} 
public Dictionary<Guid,List<string>> GetSpecificLoginData(string login) 
{ 
    // here is maked command and private method is called 
} 

// method which is called from both public methods 
private Dictionary<Guid,List<string>> GetLoginData(DbCommand command) 
{ 
    // here is code to executeCommand and data reading 
} 

Или, может быть, есть лучший способ? Другим способом является перемещение только данных в другой метод.

Спасибо за ответы, надеюсь, это поможет мне.

Answer 1

Я не вижу ничего плохого при таком подходе. Вы перемещаете общую функциональность в метод GetLoginData, что является хорошей практикой, а общедоступные методы имеют значимые имена и правильные подписи.