リファクタリングの最も良い方法は配列要素へのアクセス

Question

aは提供されたitemIDsに応じてユーザー情報の配列を返す外部dllを持っています。そのための単純なコード、次のようになります。

private string userName; 
private int userAge; 

const string NAME = "NAME"; 
const string AGE = "AGE"; 

string[] arrID = { NAME, AGE }; //IDs to get name and age, to get surname please add "SURNAME" to the array 

User[] users; 
string[] results; //OK - item found, Unknown - item id not found 

////////read from external dll//////// 
databaseAccessor.GetUsers(arrID, out users, out results); 
////////////////////////////////////// 

int itemNameIndex = Array.IndexOf(arrID, NAME); 
int itemAgeIndex = Array.IndexOf(arrID, AGE); 

if(results[itemNameIndex] == "OK") 
{ 
    userName = users[itemNameIndex].Name; 
} 

if(results[itemAgeIndex] == "OK") 
{ 
    userAge = users[itemAgeIndex].Age; 
} 

私は新しいユーザー「機能」などを取得しようとしたとき、もし新しい句が追加されなければならないので、このコードは好きではありません高さ。変更できないことの1つは、外部DLLへの呼び出しですdatabaseAccessor.GetUsers(arrID, out users, out results);

新しいifsを追加しないようにリファクタリングするにはどうすればよいでしょうか？

Answer 1

外部呼び出しのラッパークラスを作成して、そこに結果を解釈できると思います。次に、有効なユーザーエンティティのみが返されます。ユーザーエンティティは、別のユーザーエンティティクラスのインスタンスでも、名前や年齢などのプロパティでも返されます。それは私の意見でそれをよりきれいにするでしょう（そして、もし必要ならば、外部dllを簡単に交換すること）。ような何か：

IEnumerable<UserEntity> users = UserManager.Get(); 

は、それからちょうどあなたがUserManager.Getでユーザーをアクセスするために必要なすべてのコードを実装します。もちろん、配列から情報にアクセスするためのプライベートメソッドを作成することもできます。 TryGetValueのようにresults[itemNameIndex] == "OK"のチェックを行い、値を返します。

だから、値を取得するためのコードを作成して、より一般的なものを私のアドバイスにしてください。