1

Мне необходимо заполнить DataGrid данными из БД. Для этого использую

DataGrid.ItemsSource = Group.GetGroups();

Правильно ли использовать статичный метод класса, который возвращает список с типом самого класса? Будет ли очищаться память, если для заполнения грида несколько раз вызвать Group.GetGroups()?

public class Group
    {
        public int IdGrp { get; set; }
        public string GroupName { get; set; }

        public static List<Group> GetGroups()
        {
            List<Group> Groups = new List<Group>();

            try
            {
                using (OracleConnection connection = new OracleConnection(Properties.Settings.Default.ConnectionPROD))
                {
                    connection.Open();

                    using (OracleCommand command = new OracleCommand("select a.idgrp, a.name from groups a", connection))
                    {
                        OracleDataReader reader = command.ExecuteReader();
                        while (reader.Read())
                        {
                            Groups.Add(new Group()
                            {
                                IdGrp = Int32.Parse(reader["idgrp"].ToString()),
                                GroupName = reader["name"].ToString()//,
                            });
                        }
                        reader.Close();
                    }
                }
            }
            catch (Exception e)
            {                
            }

            return Groups;
        }
    }

Спасибо за советы!

3
  • А теперь попробуйте покрыть тестами свой проект. Именно те места, где используется этот метод. 1 авг 2019 в 18:48
  • я не силен в тестах, но создал тестовый метод [TestMethod] public void GetGroupsTest() { var groups = Group.GetGroups(); }. Все ок!
    – Илья
    2 авг 2019 в 7:20
  • Нет, протестируйте не этот метод, а другие методы, в которых вызывается этот. Юнит-тест должен быть быстрым как молния. То есть вместо реального получения данных из БД (которой может вовсе не быть на билд-сервере), следует использовать мок (фейковые данные). А это становится невозможно при использовании статического метода. 2 авг 2019 в 19:51

1 ответ 1

1

Память будет "очищаться", когда на возвращенный список не останется активных ссылок.

Правильно ли иметь в объекте данных метод, который читает такие объекты из базы данных, - вопрос мнений.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.