Некрасивый код

@WTF-_- · Регистрация: 29.05.2016

Студворк — интернет-сервис помощи студентам

Всем привет, как избавиться от некрасивого кода?Смысл программы: мы выбираем любой из доступных методов сортировки, и хотим посмотреть его реализацию на разных языках.
Пример метода

Кликните здесь для просмотра всего текста

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
       private void treeViewProgLanguage_AfterSelect(object sender, TreeViewEventArgs e)
        {
            switch(treeViewProgLanguage.SelectedNode.Text)
            {
                case "C":
                    {
                        if(comboBox1.SelectedIndex==0)
                            richTextBoxProgLanguage.LoadFile(@"Information\buble sort\code\c.txt", RichTextBoxStreamType.UnicodePlainText);
 
                        if (comboBox1.SelectedIndex == 1)
                            richTextBoxProgLanguage.LoadFile(@"Information\shake sort\code\c.txt", RichTextBoxStreamType.UnicodePlainText);
                    }
                    break;
            }   
        }

Очень уж режет глаза case и многочисленные if, может кто подскажет как сделать лучше?

@Теня · 22.04.2018, 20:10

Как вариант можно сделать главный абстрактный класс, который будет дисплеить код. Пусть его звут Printer.
В нем метод Display(). В Display() подгружать нужный файл и выводить на екран.
Потом пронаследовался от класа для всех возможный вариантов(или внутри установил поля, но тогда останутся if и case).
Каждому comboBox`у сделать свой метод обработки клика, в котором будет подменятся обєкт Printer'а, и заново вызыватся метод Display().

Добавлено через 48 секунд
Если сделать полями, то не пересоздавать, а изменить значение полей и перевызвать Display()

Добавлено через 14 минут
Сойдет, или еще один способ набросать?

@WTF-_- · 22.04.2018, 20:37 **[ТС]**

Сообщение от Теня

Как вариант можно сделать главный абстрактный класс, который будет дисплеить код. Пусть его звут Printer.
В нем метод Display(). В Display() подгружать нужный файл и выводить на екран.
Потом пронаследовался от класа для всех возможный вариантов(или внутри установил поля, но тогда останутся if и case).
Каждому comboBox`у сделать свой метод обработки клика, в котором будет подменятся обєкт Printer'а, и заново вызыватся метод Display().
Добавлено через 48 секунд
Если сделать полями, то не пересоздавать, а изменить значение полей и перевызвать Display()
Добавлено через 14 минут
Сойдет, или еще один способ набросать?

Вы так нагрузили меня, что покуда хватит одного , спасибо

@diadiavova · 22.04.2018, 20:53

WTF-_-, проще всего будет формировать пути по данным узлов дерева и выбранного иднекса. Насколько я понимаю у тебя каждый тип сортировки в отдельно папке, а имя файла зависит от языка. Ну тогда свойство Tag узла дерева должно иметь значение c.txt.
Чтобы поставить в соответствие индексы комбобокса и имена папок, можно составить коллекцию, где эти имена будут находиться под соответствующими индексами.

C#
1
2
3
var comboIndex = new string[]{"buble sort", "shake sort"};
var fileName = System.IO.Path.Combine(pathToRoot, comboInd[comboBox1.SelectedIndex], treeViewProgLanguage.SelectedNode.Tag );
richTextBoxProgLanguage.LoadFile(fileName);

@WTF-_- · 23.04.2018, 14:59 **[ТС]**

Сообщение от diadiavova

WTF-_-, проще всего будет формировать пути по данным узлов дерева и выбранного иднекса. Насколько я понимаю у тебя каждый тип сортировки в отдельно папке, а имя файла зависит от языка. Ну тогда свойство Tag узла дерева должно иметь значение c.txt.
Чтобы поставить в соответствие индексы комбобокса и имена папок, можно составить коллекцию, где эти имена будут находиться под соответствующими индексами.

Очень интересный способ, спасибо, буду сейчас пробовать)

Добавлено через 17 часов 55 минут
diadiavova, спасибо за простой и крутой метод)

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11680&d=1772460536 Одним из. . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .

@Теня 1 / 1 / 0 Регистрация: 02.09.2017 Сообщений: 14
	22.04.2018, 20:10
	Как вариант можно сделать главный абстрактный класс, который будет дисплеить код. Пусть его звут Printer. В нем метод Display(). В Display() подгружать нужный файл и выводить на екран. Потом пронаследовался от класа для всех возможный вариантов(или внутри установил поля, но тогда останутся if и case). Каждому comboBox`у сделать свой метод обработки клика, в котором будет подменятся обєкт Printer'а, и заново вызыватся метод Display(). Добавлено через 48 секунд Если сделать полями, то не пересоздавать, а изменить значение полей и перевызвать Display() Добавлено через 14 минут Сойдет, или еще один способ набросать? 1

Некрасивый код

Решение