Оптимизировать объемный код вывода в TextBlock

Question

Прошу Вас помочь оптимизировать код вывода в textblock. Показать как правильно. Спасибо.

public void PrintCurrentlyPlaying(object sender, EventArgs e)
    {
        var sessionManager = GlobalSystemMediaTransportControlsSessionManager.RequestAsync().GetAwaiter().GetResult(); // получаем результат из Control Media Windows 10
        if (sessionManager != null) // если результат получен, то 
        {
            var currentSession = sessionManager.GetCurrentSession(); // возращаем сессию
            if (currentSession != null) // если сессия не пустая, то
            {
                var mediaProperties = currentSession.TryGetMediaPropertiesAsync().GetAwaiter().GetResult(); // в mediaProperties мы отправляем название воспроизведения и имя автора
                if (unknown_artist.IsChecked == true && dash_instead_text.IsChecked == false && mediaProperties.Artist == "Неизвестный исполнитель") // проверяем настройки приложения и выводим со словами
                    compocition_output.Text = $"Проигрывается: {mediaProperties.Title}";
                else
                {
                    if (unknown_artist.IsChecked == true && dash_instead_text.IsChecked == true && mediaProperties.Artist == "Неизвестный исполнитель") // иначе выводим без слов
                        compocition_output.Text = $"{mediaProperties.Title}";
                    else
                    {
                        if (dash_instead_text.IsChecked == true && mediaProperties.Artist != "")  
                            compocition_output.Text = $"{mediaProperties.Title} - {mediaProperties.Artist}";
                        else 
                        {
                            if (dash_instead_text.IsChecked == true)
                                compocition_output.Text = $"{mediaProperties.Title}";
                            else
                            {
                                if (mediaProperties.Artist != "")
                                {
                                    if (author_checkbox.IsChecked == true && compocition_checkbox.IsChecked == true)
                                        compocition_output.Text = $"Проигрывается: {mediaProperties.Title}, от автора: {mediaProperties.Artist}";
                                    else
                                    {
                                        if (author_checkbox.IsChecked == false && compocition_checkbox.IsChecked == true)
                                            compocition_output.Text = $"Проигрывается: {mediaProperties.Title}";
                                        else
                                        {
                                            if (author_checkbox.IsChecked == true && compocition_checkbox.IsChecked == false)
                                                compocition_output.Text = $"Автор воспроизведения: {mediaProperties.Artist}";
                                        }
                                    }
                                }
                                else
                                    compocition_output.Text = $"Проигрывается: {mediaProperties.Title}";
                            }
                        }
                    }
                }
            }
            else
                compocition_output.Text = "Упс... А музыка у нас закончилась!";
        }
        else
            compocition_output.Text = "Упс... А музыка у нас закончилась!";
        if (output_file_checkbox.IsChecked == true)
        {
            file_save(null, null); // сохраняем название в .txt файл
        }
}

Если будет возможно с комментариями.

Answer 1

1. Добро пожаловать в Асинхронное программирование. Если вы используете .GetAwaiter().GetResult(), либо вы абсолютно точно уверены, что вы знаете, что делаете, либо что-то пошло не так, например вы не в курсе о существовании async/await. Нужно использовать await.
2. Есть такая проверка на null - NULL-условный оператор ?.. Познакомьтесь с ним, очень сильно упрощает код в некоторых местах.
3. У вас повторяется код, вы одно и то же условие перепроверяете по многу раз. Чтобы одно и то же перезаписать в текстовое поле. Вам по сути нужно построить строчку из кусочков, исходя из разных условий. Отлично! Для этого есть замечательный класс StringBuilder, к тому же он работает намного быстрее, чем интерполяция или конкатенация строк.
4. if (b == true) можно заменить на if (b), а if (b == false) можно заменить на if (!b), потому что логическое выражение и так само по себе true или false и преобразовывать например true в true с помощью проверки на true - излишество.
5. text != "" можно записать как text.Length > 0, ну потому что это технически работает быстрее. Так как числовые операции всегда быстрее строковых.

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

public async void PrintCurrentlyPlaying(object sender, EventArgs e)
{
    // в async void методах обязательно нужно обрабатывать все возможные исключения, иначе вы их просто не увидите, а работать не будет
    try
    {
        var sessionManager = await GlobalSystemMediaTransportControlsSessionManager.RequestAsync();
        var currentSession = sessionManager?.GetCurrentSession();
        if (currentSession != null)
        {
            var mediaProperties = await currentSession.TryGetMediaPropertiesAsync();
            StringBuilder sb = new StringBuilder();
            if (unknown_artist.IsChecked && mediaProperties.Artist == "Неизвестный исполнитель")
            {
                if (!dash_instead_text.IsChecked)
                    sb.Append("Проигрывается: ");
                sb.Append(mediaProperties.Title);
            }
            else
            {
                if (dash_instead_text.IsChecked)
                {
                    sb.Append(mediaProperties.Title);
                    if (mediaProperties.Artist.Length > 0)
                        sb.Append(" - ").Append(mediaProperties.Artist);
                }
                else
                {
                    if (mediaProperties.Artist.Length > 0)
                    {
                        if (compocition_checkbox.IsChecked)
                        {
                            sb.Append("Проигрывается: ").Append(mediaProperties.Title);
                            if (author_checkbox.IsChecked)
                                sb.Append(", от автора: ").Append(mediaProperties.Artist);
                        }
                        else
                            sb.Append("Автор воспроизведения: ").Append(mediaProperties.Artist);
                    }
                    else
                        sb.Append("Проигрывается: ").Append(mediaProperties.Title); ;
                }
            }
            compocition_output.Text = sb.ToString();
        }
        else
            compocition_output.Text = "Упс... А музыка у нас закончилась!";
        if (output_file_checkbox.IsChecked)
        {
            file_save(null, null);
        }
    }
    catch (Exception ex)
    {
        MessageBox.Show(ex.Message);
    }
}

Answer 2

Просто оставлю это здесь)

public async void PrintCurrentlyPlaying(object sender, EventArgs e)
{
    var currentSession = (await GlobalSystemMediaTransportControlsSessionManager.RequestAsync())?.GetCurrentSession(); // возращаем сессию
    if (currentSession != null) // если сессия не пустая, то
    {
        var mediaProperties = await currentSession.TryGetMediaPropertiesAsync(); // в mediaProperties мы отправляем название воспроизведения и имя автора
        var isUnknownArtist = mediaProperties.Artist is "" or "Неизвестный исполнитель";
    compocition_output.Text =
        (IsUnknownArtist: isUnknownArtist,
        UnknownArtistChecked: unknown_artist.IsChecked,
        DashInsteadText: dash_instead_text.IsChecked,
        IsAuthor: author_checkbox.IsChecked,
        IsCompocition: compocition_checkbox.IsChecked) switch
        {
            { UnknownArtistChecked: true, DashInsteadText: true, IsUnknownArtist: true } => $"{mediaProperties.Title}",
            { DashInsteadText: true, IsUnknownArtist: false } => $"{mediaProperties.Title} - {mediaProperties.Artist}",
            { DashInsteadText: true, IsUnknownArtist: true } => $"{mediaProperties.Title}",
            { IsUnknownArtist: false, IsAuthor: true, IsCompocition: true } => $"Проигрывается: {mediaProperties.Title}, от автора: {mediaProperties.Artist}",
            { IsUnknownArtist: false, IsAuthor: true, IsCompocition: false } => $"Автор воспроизведения: {mediaProperties.Artist}",
            _ => $"Проигрывается: {mediaProperties.Title}"
        };
}
else
    compocition_output.Text = "Упс... А музыка у нас закончилась!";

if (output_file_checkbox.IsChecked == true)
    file_save(null, null); // сохраняем название в .txt файл

}

^{Не проверял, да и уверен, что накосячил с некоторыми проверками, там уже если нужно будет, подправите под себя.}

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

Из кода выше можно заметить, что некоторые проверки весьма странные и лишние, да и вообще не пойму зачем такая "динамика" в название, обычно это четко сформированный стандарт (аля: автор - трек), а остальное это "рюшечки" UI, может вам стоит пойти именно по этому пути?

Также из замечаний, почему метод PrintCurrentlyPlaying вдруг отвечает за сохранение? Print это вывести, а не сохранить.

Ну и про всякие async и прочее вам уже сказали другим ответом.

---

В моем примере используется последняя версия языка (C#9), можно без проблем опустить до C#8, заменив

var isUnknownArtist = mediaProperties.Artist is "" or "Неизвестный исполнитель";

на

var isUnknownArtist = string.IsNullOrWhiteSpace(mediaProperties.Artist) || mediaProperties.Artist == "Неизвестный исполнитель";

Ниже C#8 работать не будет, ибо все эти навороты с switch появились в нем (хотя кто знает, может умельцы сделали стороннюю библиотеку).