Почему при запросе объекта стиля, получается null?

Question

[Untiwe]

Есть стиль, прописанный в ресурсах.

<Window.Resources>
        
        <Style x:Name="NubmersStyle" TargetType="TextBlock" BasedOn="{StaticResource BaseFontFamily}">
            <Setter Property="Control.FontSize" Value="50"/>
            <Setter Property="Control.Background" Value="Transparent"/>
            <Setter Property="Control.HorizontalContentAlignment" Value="Center"/>
            <Setter Property="Control.VerticalContentAlignment" Value="Center"/>
            <Setter Property="Control.HorizontalAlignment" Value="Stretch"/>
            <Setter Property="Control.VerticalAlignment" Value="Stretch"/>
            <Setter Property="Control.Foreground" Value="white"/>
        </Style>


 </Window.Resources>

Я пытаюсь поменять цвет текста в коде. Для этого в гугле быстро находится решение, которое надо чуть поменять и получается это

public void SetStyles()
        {
            Setter setter = new Setter(Control.ForegroundProperty, Color.FromArgb(0, 0, 0, 0));
            Style = Resources["NubmersStyle"] as Style;
            Style.Setters.Add(setter);
        }

Из него видно, что я общаюсь к словарю ресурсов по ключу, получаю объект и привожу его у типу Style.
Вот только после приведения типов, я получаю null и при добавлении нового setter получаю ошибку. Из дебагера видно, что словарь Resources есть и у него даже есть ключ с именем NubmersStyle Я так догадываюсь, что тут ошибка с приведением типов, но это только догадка.

Comments

[Борис Животное]

Несколько советов/замечаний

В сеттерах свойств не нужно писать имена свойств текстом, так как такой код подвержен ошибкам, после его редактирования, а именно переименований свойств. Лучше использовать nameof(PropertyName)

public short PlayerNomber
{
    get => _PlayerNomber;
    set
    {
        _PlayerNomber = value;
        OnPropertyChanged("PlayerNomber");
    }
}

Можно лучше записывать так:

public short PlayerNomber
{
    get => _PlayerNomber;
    set
    {
        _PlayerNomber = value;
        OnPropertyChanged(nameof(PlayerNomber));
    }
}

Но в данном случае вообще не нужно ничего указывать, а нужно писать так:
OnPropertyChanged();

Так как на место prop будет подставлено имя свойства автоматически

public void OnPropertyChanged([CallerMemberName] string prop = "")
{ 
    if (PropertyChanged != null)
        PropertyChanged(this, new PropertyChangedEventArgs(prop));
}

[Борис Животное]

А ещё лучше подключить библиотеку MvvmLight и не изобретать ненужные велосипеды (см. MvvmLightLibsStd10):

<ItemGroup>
    <PackageReference Include="Extended.Wpf.Toolkit" Version="4.1.0" />
    <PackageReference Include="MvvmLightLibsStd10" Version="5.4.1.1" />
    <PackageReference Include="WpfScreenHelper" Version="1.0.0" />
  </ItemGroup>

Вот так будет выглядеть твой класс:

using GalaSoft.MvvmLight;

namespace Monitor
{
    public class CentralInfoViewModel : ViewModelBase
    {
        private short _period;
        private string _name;

        /// <summary>
        /// Номер текущего периода
        /// </summary>
        /// <value>short</value>
        public short Period
        {
            get => _period;
            set
            {
                if (value < 0)
                    value = 0;
                // Это тоже самое, что и OnPropertyChanged(nameof(Period));, только ещё и обновляет поле
                Set(ref _period, value);
            }
        }

        public string Name
        {
            get => _name;
            set
            {
                if (Set(ref _name, value))
                {
                    // Свойство изменилось, что-то делаем
                }
            }
        }
    }
}

[Untiwe]

Спасибо большое. К сожалению я уже не у компьютера и могу посмотреть только поверхностоно но обязательно все исправлю как будет возможность.

[Борис Животное]

Здесь не происходит никакого ожидания в 1 секунду, у тебя этот код работает в бесконечном цикле без задержек.

/// <summary>
/// Запускаем обновление временни на малом табло
/// </summary>
private void StartRefrashDateTime()
{
    Task.Factory.StartNew(() =>
    {
        while (true)
        {
            Dispatcher.Invoke(() =>
            {
                ClockBox.Text = DateTime.Now.ToString("HH:mm");
            });

            Task.Delay(1000);
        }
    });
}

Достаточно таймера здесь. Для примера удалил лишнее всё. Метод выше точно нужно удалить совсем. Код без остановки таймера, так как окно у тебя открыто всё время, пока работает приложение, но может стать невидимым. Можешь отключать таймер на это время, в принципе.

public partial class SmallWindow : Window
{
    private readonly DispatcherTimer _timer;

    /// <summary>
    /// Инициализация класса, малоко окна
    /// </summary>
    public SmallWindow()
    {
        InitializeComponent();

        _timer = new DispatcherTimer();
        _timer.Interval = TimeSpan.FromSeconds(1);
        _timer.Tick += OnTimerTick;
        // Запускаем обновление временни на малом табло
        _timer.Start();
    }

    private void OnTimerTick(object sender, EventArgs e)
    {
        ClockBox.Text = DateTime.Now.ToString("HH:mm");
    }
}

[Untiwe]

Борис Животное,
На счет OnPropertyChanged(); это привычка, которая осталась у меня с python и его философии

Explicit is better than implicit.

На счет MvvmLight не надо каждый раз объявлять OnPropertyChanged? Выглядит очень интересно, обязательно почитаю про него.

На счет StartRefrashDateTime и OnTimerTick
Я не вижу установки интервала для DispatcherTimer, я догадываюсь что он срабатывает каждую мс не является ли это бессмысленной нагрузкой на компьютер?
Подскажите, почему Task.Delay(1000); не останавливает поток?

[Борис Животное]

Untiwe, ой точно, я просто забыл написать интервал срабатывания :). Скоро поправлю код. Я просто сначала интервал прописал в конструктор, а потом решил его оттуда убрать и присваивать свойству.

На счет MvvmLight не надо каждый раз объявлять OnPropertyChanged?

Нет, не надо. Его и у тебя в коде не надо каждый раз объявлять. Ты можешь создать абстрактный класс ViewModelBase и там реализовать интерфейс. И потом от него наследоваться вью моделями. Только зачем? Например, я уже на второй работе использую MvvmLight. На первой я сам его заюзал, на вторую уже пришёл, он там был в проекте. Правда сейчас у меня UWP, а не WPF. После WPF та ещё боль, там урезан функционал биндингов и вообще много мелочей отличаются.

[Борис Животное]

Подскажите, почему Task.Delay(1000); не останавливает поток?

Потому что Delay возвращает экземпляр Task, который никто не ожидает. Вот так будет работать правильно, но я бы таймер использовал.

public partial class SmallWindow : Window
{
    public SmallWindow()
    {
        InitializeComponent();
        // Запускаем обновление временни на малом табло
        StartRefreshDateTime();
    }

    /// <summary>
    /// Запускаем обновление временни на малом табло
    /// </summary>
    private async void StartRefreshDateTime()
    {
        await Task.Factory.StartNew(async () =>
        {
            while (true)
            {
                Dispatcher.Invoke(() =>
                {
                    ClockBox.Text = DateTime.Now.ToString("HH:mm");
                });

                await Task.Delay(1000);
            }
        }, TaskCreationOptions.LongRunning);
    }
}

[Борис Животное]

Обновил код выше (и таймер тоже)

[Untiwe]

Борис Животное, на сколько я понимаю, UWP это улучшенная замена WPF, как в 2006 WPF был улучшенной заменой WimForms. Только она работает на десятке и чуть-чуть на восьмёрке.
Не уж то ее сделали хуже и менее удобной? Ведь для майков есть хороший пример в виде HTML + CSS (мне так кажется)

[Борис Животное]

Untiwe, не то что бы хуже. Там есть анимации, которые считаются не в UI потоке и много других удобных штук. Но там основной подход это одно окно и переход между страницами. Там приложения публикуются через MS Store и у них ограниченный доступ к системе. Там в манифесте можно задавать разрешения для приложения разные. По умолчанию стоит одно - Internet Client. Приложение по умолчанию может писать и читать только из его собственных папок, к другим у него доступа нет. Так же у него немного другой жизненный цикл приложения, как на мобильниках примерно и планшетах. Ты свернул приложение, оно как бы засыпает. Можно запросить время на дальнейшую работу в фоне, не факт что ОС его даст, например, если винда стоит на ноуте, батарея садится или у пользователя запрет в настройках на работу приложений в фоне. Там много всего, так не расскажешь, нужно читать документацию.

[Борис Животное]

Untiwe, там пошло дальше развитие. Сейчас уже MAUI на подходе и WinUI 3.0 (Project Reunion)
статья на хабре - https://habr.com/ru/post/546576/

[Untiwe]

Борис Животное, ааа... они больше делают что то универсальное, что может работать на компьютере и телефоне одновременно

Answer 1

[Борис Животное]

Как вариант, можно создать окно, в котором задать заранее определённые стили, задать им TargetType и это надо сделать. Задать каждому стилю уникальный ключ. Создать в Code Behind окна свойство с текущим стилем и его менять. К этому свойству можно прибиндиться TextBlock(ом). Можно всё это сделать красивее и т.д., но суть из примера будет ясна, думаю.

Вот можно посмотреть как здесь предлагают Changing the styles at runtime in WPF

<Window
    x:Class="Monitor.SomeWindow"
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
    xmlns:local="clr-namespace:Monitor"
    xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
    Title="SomeWindow"
    Width="800"
    Height="450"
    mc:Ignorable="d">
    <Window.Resources>
        <Style
            x:Key="BaseFontFamily"
            TargetType="TextBlock">
            <Setter Property="FontSize" Value="90" />
        </Style>
        <Style
            x:Key="Numbers1Style"
            BasedOn="{StaticResource BaseFontFamily}"
            TargetType="TextBlock">
            <Setter Property="Foreground" Value="LightCoral" />
        </Style>
        <Style
            x:Key="Numbers2Style"
            BasedOn="{StaticResource BaseFontFamily}"
            TargetType="TextBlock">
            <Setter Property="Foreground" Value="Bisque" />
        </Style>
    </Window.Resources>
    <Grid>
        <Grid.RowDefinitions>
            <RowDefinition Height="*" />
            <RowDefinition Height="*" />
        </Grid.RowDefinitions>
        <TextBlock
            Grid.Row="0"
            HorizontalAlignment="Center"
            VerticalAlignment="Center"
            Style="{Binding TimeBlockStyle, RelativeSource={RelativeSource AncestorType=local:SomeWindow}}"
            Text="{Binding Path=RightTeam.TeamCounter, Mode=OneWay, UpdateSourceTrigger=PropertyChanged}" />
        <Button
            Grid.Row="1"
            Click="OnButtonClick" />
    </Grid>
</Window>

public partial class SomeWindow : Window
{
    public static readonly DependencyProperty TimeBlockStyleProperty = DependencyProperty.Register(
        nameof(TimeBlockStyle), typeof(Style), typeof(SomeWindow), new PropertyMetadata(default(Style)));

    public Style TimeBlockStyle
    {
        get { return (Style)GetValue(TimeBlockStyleProperty); }
        set { SetValue(TimeBlockStyleProperty, value); }
    }

    public SomeWindow()
    {
        InitializeComponent();
        TimeBlockStyle = (Style)Resources["Numbers1Style"];
    }

    private void OnButtonClick(object sender, RoutedEventArgs e)
    {
        TimeBlockStyle = (Style)Resources["Numbers2Style"];
    }
}

Comments

[Untiwe]

Ого! Это действительно очень интересный ход!

[Борис Животное]

Untiwe, ещё рекомендую установить XamlStyler и всегда им форматировать XAML. Удобная штука, а Xaml код удобно читать, так как он всегда и везде выглядит одинаково, свойства отсоритрованы.

[Untiwe]

Борис Животное, Вау! Стало гораздо красивее

[Борис Животное]

Untiwe, рекомендую настроить XamlStyler так:

[Untiwe]

Борис Животное, Поставил, спасибо)

Answer 2

[AndromedaStar]

Resources["NubmersStyle"] - опечатка. А опечатка случилась из-за магических слов. В таком стиле на C# стараются не писать.

Comments

[Untiwe]

Спасибо, а то я совсем не видел.

Еще пришлось x:Name="NubmersStyle" поменять на x:Key="NubmersStyle" стиль наконец стало видно, но теперь он ругается System.InvalidOperationException: "TargetType "TextBlock" не соответствует типу элемента "BigWindow"." Наверно из-за параметра TargetType="TextBlock"

P.S. действие происходит не в MainWinow а в дочернем окне BigWindow
P.S.S На сколько я знаю в стиле C# писать CamelCase

[Борис Животное]

Untiwe, неважно, где ты получаешь стиль. Важно, к чему ты его применяешь. Если ты создал стиль для TextBlock, а присваиваешь его окну, то получишь ошибку по понятным причинам.

Еще пришлось x:Name="NubmersStyle" поменять на x:Key="NubmersStyle"

Так ты опять написал одно и тоже. В комментарии точно.

Наверное, имелось ввиду Numbers, а ты написал Nubmers

[Untiwe]

Борис Животное, да, написал только в комментарии. В коде же исправил

<Style x:Key="NumbersStyle" TargetType="TextBlock" BasedOn="{StaticResource BaseFontFamily}">

Отладчик, как и вы, тоже говорит мне про разные типы. Похоже тут что то с решением в одну строку, но я понимаю что именно.

public void SetStyles()
        {
            Setter setter = new Setter(ForegroundProperty, Color.FromArgb(0, 0, 0, 0));
            Style = Resources["NumbersStyle"] as Style;
            Style.Setters.Add(setter);
        }

[Борис Животное]

Untiwe, для ответа на вопрос мало информации. Но включу свои догадки. SetStyle это метод класса BigWindow из чего следует, что свойство Style это свойство класса BigWindow. Так как стиль присваивается BigWindow, то и TargetType стиля должен быть BigWindow, а не TextBlock. Думаю, это ошибка не получения стиля, а присвоения его свойству. Так это или нет проверить легко, но судя по сообщению это так.

И чуть не в тему, но я бы сначала добавлял новый Setter, а потом свойству присваивал стиль. Приводить к типу нужно с использованием as только если ожидается, что не удастся привести к типу Style ресурс с именем "NumbersStyle".

[Untiwe]

Борис Животное, да, ваши догадки верны.
1. Если убрать TargetType ошибка будет уже другая.
Стиль

<Style x:Key="NumbersStyle" BasedOn="{StaticResource BaseFontFamily}">

Ошибка


2. TargetType позволяет мне применять стиль ко всем TextBlock в окне и не прописывать его в ручную к каждому. По этому хотелось бы менять цвет у стиля, имеющим TargetType

[Борис Животное]

Untiwe, написано, что нельзя менять, значит уже нельзя ничего добавить, если не создать стиль в коде с нуля. Можешь положить стиль без ключа в App.xaml и он применится ко всем TextBlock в приложении, но, насколько помню, ещё и к тем, которые находятся в стилях контролов, то есть вообще ко всем, даже тем, которые на кнопках пишут текст, например, если я не путаю ничего.

<Style TargetType="TextBlock" BasedOn="{StaticResource BaseFontFamily}">

Лучше опиши чётко задачу, что ты пытаешься сделать. Просто применить ко всем текстовым блокам стиль? А зачем тогда что-то добавлять в сеттеры?

Нельзя присвоить стиль для текстблока окну. Если присвоить стиль окну без таргет тайп или с ним, но для окна, то текстовые блоки это не затронет, это не так работает.

[Борис Животное]

Обновил коммент выше

[Untiwe]

Изначально я сделал стиль, который применяется ко всем TextBlock благодаря параметру TargetType Стили пишутся в Window.Resources по этому я его туда заснул.

<Window.Resources>
        <Style x:Key="BaseFontFamily">
         <!--  Базовый стиль с цветом-->
            <Setter Property="Control.Foreground" Value="#FF4500"/>
        </Style>

       <Style x:Key="NumbersStyle" TargetType="TextBlock" BasedOn="{StaticResource BaseFontFamily}">
            <Setter Property="Control.FontSize" Value="50"/>
            <Setter Property="Control.Background" Value="Transparent"/>
            <Setter Property="Control.HorizontalContentAlignment" Value="Center"/>
            <Setter Property="Control.VerticalContentAlignment" Value="Center"/>
            <Setter Property="Control.HorizontalAlignment" Value="Stretch"/>
            <Setter Property="Control.VerticalAlignment" Value="Stretch"/>
        </Style>

        <!--  Еще стили с наследованием BaseFontFamily-->

</Window.Resources>

Потом меня попросили, что бы пользователь мог менять цвет сам. Я стал искать способ программно менять Setter В гугле нашел похожий пример и адаптировал под свои нужды. Для этого у стиля появился x:Key="NumbersStyle" что бы его можно было найти в словаре ресурсов.

И появился такой метод в классе BigWindow

public void SetStyles()
        {
            Setter setter = new Setter(ForegroundProperty, Color.FromArgb(0, 0, 0, 0));
            Style = Resources["NumbersStyle"] as Style;
            Style.Setters.Add(setter);
        }

[Борис Животное]

Untiwe, я погуглил и понял, что нужно создавать программно копию стиля и её менять как угодно. Можно попробовать прочитать готовый XAML как текст и уже этому объекту добавлять сеттеры. Что-то такое я много лет назад писал, нужно поискать. Не обещаю, что найду.

[Untiwe]

Борис Животное,
1. TargetType="TextBlock" почему то не работает, когда и почему это случилось, мне уже не интересно, замучался я с ним.
2. Я могу поменять цвет через явно определенный стиль Style="{StaticResource DeleteNumbersStyle}" Только тут же знакомая ошибка

Как я понимаю, единственный выход, это создавать стиль программного и переназначать его каждому элементу. Более простого пути нет?

[Untiwe]

Борис Животное,
В итоге сделал это через контекст данных. Теперь класс выглядит примерно так.

<Style x:Key="NumbersStyle">
    <Setter Property="Control.Foreground" Value="{Binding Path=BigWindowColors.ColorNumbers, TargetNullValue=white}"/>
</Style>

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

[Борис Животное]

Untiwe, а можно увидеть весь проект? Или проект с воспроизведением проблемы? Я просто видел обрывки кода и лишь примерно представляю, что происходит. Возможно, это как-то легко решается, но проект я не видел и мне сложно что-то советовать. Я бы хотел увидеть проект, где эта проблема решена вышепоказанным способом.

[Untiwe]

В классе MainWindow есть событие OpenBigWindow которое срабатывает при нажатии на кнопку открытия большого экрана.

Стиль должен менять цвет у цифр на этом экране, сейчас он стоит белый. Привязка данных идет через WindowsStyles и его объект BigWindowColors в классе View
через который в свою очередь, идет привязка данных. Цвет читается из файла настроек app.config на данный момент, там есть цвет только для одного стиля.

public partial class MainWindow : Window
    {
       BigWindow bigWindow = new BigWindow();
       SmallWindow smallWindow = new SmallWindow();
       public View _view = new View();

public MainWindow()
{
            //some code
            DataContext = _view;
            bigWindow.DataContext = DataContext;
            //some code

В классе BigWindow есть метод SetStyles сейчас он включен, по этому ошибка будет сразу при запуске программы

public partial class BigWindow : Window
    {
        
        public BigWindow()
        {
            InitializeComponent();
            SetStyles(); //пробуем поменать стили
        }

Можете скачать проект и посмотреть https://github.com/untiwe/Monitor

P.S. Я понимаю, что далеко не сеньор и если будут советы/адекватная критика я готов выслушать.