0

Есть класс Querys с таким кодом -

MainWindow mainWindow;       
public static DataTable readEmployers()
{
    MainWindow mainWindow = new MainWindow()
    try
    {
        SqlDataAdapter da = new SqlDataAdapter("SELECT * FROM [employment]", ConnectionAdres);
        DataTable dt = new DataTable("Call Reciept");
        da.Fill(dt);   
        mainWindow.successfulConnectuon();
        return dt;
    }
    catch (Exception ex)
    {
        mainWindow.errorConnection(ex);
        return null;
    }
}

И такие методы в MainWindow -

public void successfulConnectuon()
    {
        Status.Content = "Запрос выполнен успешно";
        MessageBox.Show("test");
    }
public void errorConnection(Exception ex)
    {
        Status.Content = "Ошибка выполнения запроса";
    }

Сам Querys вызывается из другой страницы который в свою очередь вызывается из MainWindow. Проблема в том, что ошибок нет, MessageBox выводится но Label не меняется.

9
  • 2
    Кхм, так лучше не писать... Этот винегрет из работы с БД и с UI -- адский кошмар для любого более-менее вменяемого разработчика. Для WPF используйте MVVM, или на худой конец создайте хотя бы отдельный класс Repository, в котором проводите всю работу с БД (извлечение данных, обновление, добавление данных), а в кодбихайнде обрабатывайте события кнопок и осуществ. взаимодействие с контролами.
    – Bulson
    5 янв 2018 в 18:08
  • @Bulson мне нужно как то вывести в главное окно - успешно ли прошёл запрос или нет. А этот код пишется далеко не для вменяемого разработчика)
    – xomem
    5 янв 2018 в 18:10
  • У вас должен быть метод, отдельный метод, который обращается к БД и извлекает данные, которые он возвращает в виде либо коллекции элементов (объектов соотв. таблице в БД), либо в виде экземпляра объекта, либо возвращает null. Допустим, по нажатию на кнопку в окне, вы вызываете это метод, получаете данные, проверяете на null и если не нул показываете их пользователю, или сообщаете ему, что что-то пошло не так.
    – Bulson
    5 янв 2018 в 18:15
  • @Bulson то есть лучше не try/catch а if/else?
    – xomem
    5 янв 2018 в 18:17
  • Нееет... Причем здесь try/catch ? Создайте отдельный метод, в котором идет работа с БД, и запускайте его на выполнение по клику на кнопке в окне.
    – Bulson
    5 янв 2018 в 18:19

1 ответ 1

1

попробуй так

var win = Application.Current.Windows.OfType<MainWindow>().FirstOrDefault();
win.successfulConnectuon();

static public MainWindow mainWindow = Application.Current.Windows.OfType<MainWindow>().FirstOrDefault();
mainWindow.successfulConnectuon();
11
  • Спасибо. Помогло. А можно его написать мне метода? Выше, под классом? Что-то типа private static var mainWindow = Application.Current.Windows.OfType<MainWindow>().FirstOrDefault(); ?
    – xomem
    5 янв 2018 в 17:34
  • да можно только не var это локальная переменая
    – Саске
    5 янв 2018 в 17:36
  • напиши так static Window mainWindow = Application.Current.Windows.OfType<MainWindow>().FirstOrDefa‌​ult();
    – Саске
    5 янв 2018 в 17:37
  • 1
    @xomem вы совершенно неверно используете WPF. Научитесь делать Binding и не надо будет делать такие дурацкие манипуляции...
    – EvgeniyZ
    5 янв 2018 в 18:08
  • 2
    @xomem Вы не понимаете основную суть привязок, MVVM и других аспектов. Ваш код не как не должен быть связан напрямую с дизайном. Вы создаете коллекцию (скажем List<string> Logs), она у вас отдельно, в неком классе находится. В коде, вы работаете только с этой коллекцией, код не знает что в главном окне у вас есть скажем ListBox c именем "вася", он знает только то, что у него есть коллекция и только с ней работает. А вот дизайн привязывается при помощи биндинка к этой самой коллекции, скажем {binding Logs}. Почитайте про привязки и MVVM. То, что вы сейчас делаете - это изврат!.
    – EvgeniyZ
    5 янв 2018 в 18:51

Ваш ответ

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

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