Как оптимизировать код?

Question

[Tarasovych]

Нужно передать свойства в метод, который удалит запись из БД:

string taskId = (lstTasks.SelectedItem as DataRowView)["id"].ToString();
    string taskName = (lstTasks.SelectedItem as DataRowView)["name"].ToString();
    DeleteTask(taskId, taskName);

Думаю, это плохой код.
Вот мои варианты:
1) инициализировать класс, потом в нужной функции получать его свойства

public class Task {
    public string Id {get; set;}
    public string Name {get; set;}
}

Task task = new Task {Id=(lstTasks.SelectedItem as DataRowView)["id"].ToString(), Name = (lstTasks.SelectedItem as DataRowView)["name"].ToString()}

2)

var task = (lstTasks.SelectedItem as DataRowView);
    string taskId = task["id"].ToString();
    string taskName = task["name"].ToString();

Как поступить? Рассмотрю ваши варианты. Спасибо!

Answer 1

[eRKa]

Не стоит городить огород из ничего или иначе overhead. У Вас все нормально, только нужно учесть в паре мест возможность эксепшина. Это при приведении as может вернуть null и при чтении ключа, его может не быть.

var item = lstTasks.SelectedItem as DataRowView;
if(item == null)
    return throw new NullReferenceExeption(nameof(item));
var resultId = item.TryGetValue("id", out int id);
var resultName = item.TryGetValue("name", out string name);
if(resultId && resultName)
    DeleteTask(id.ToString(), name);

Comments

[mpnj]

Кажется,что можно ещё заоптимизировать)

var item = lstTasks.SelectedItem as DataRowView;
if(item == null)
    return throw new NullReferenceExeption(nameof(item));
if(item.TryGetValue("id", out int id) && item.TryGetValue("name", out string name))
    DeleteTask(id.ToString(), name);

Не создали лишних переменных, +(что важней) если первый TryGetValue закончился неудачно, то второй мы делать не станем)

Мелочи, но приятно)

Answer 2

[lam0x86]

Почему бы не задать гриду коллекцию Task-ов в качестве ItemsSource?

Comments

[Tarasovych]

Не совсем понял: если задать коллекцию, я просто свяжу значения из БД и, в моем случае, listbox, где они находятся? Как это поможет в процессе удаления?

[lam0x86]

Tarasovych, без кода сложно что-то рекомендовать, но кажется, у вас нет разделения Model и ViewModel. См. MVVM.

[Tarasovych]

lam0x86, буду благодарен за хорошую литературу/полезные ссылки по паттерну, если не затруднит