Сахабутдинов Ильфир #10

ILFirV-V · 2024-12-18T13:34:25Z

dmitgaranin · 2024-12-18T16:20:09Z

TagsCloudContainer.ConsoleUi/Handlers/VisualizationHandler.cs

+        var wordSettings = wordSettingsProvider.GetWordSettings();
+        var text = fileReader.ReadText(options.InputPath);
+        var analyzeWords = textPreprocessor.GetWordFrequencies(text, wordSettings);
+        using var image = wordsCloudVisualizer.CreateImage(imageSettings, analyzeWords);


Здесь смешались ответственности (SRP): простановка опций и непосредственная генерация. Учитывая, что все остальные хендлеры исключительно про работу с опциями, то и здесь второй домен нужно вынести

Разбил на несколько опций

dmitgaranin · 2024-12-18T16:21:16Z

TagsCloudContainer.ConsoleUi/Handlers/ImageSettingHandler.cs

+
+    public string Execute(ImageSettingsOptions options)
+    {
+        ChangeSettings(options);


Я так понимаю, это не change, а скорее set, т.к. у пользователя лишь единственная возможность проставить настройки

Пользователь может менять настройки как до генерации, так и после. Настройки после запишутся для следующей генерации. Переименовал

dmitgaranin · 2024-12-18T16:21:45Z

TagsCloudContainer.ConsoleUi/Handlers/ImageSettingHandler.cs

+        {
+            imageSettingsProvider.SetWordColor(options.WordColor);
+        }
+        if (!options.FontFamily.Equals(currentSettings.FontFamily))


Почему-то у меня есть вот такая странная ошибка

dmitgaranin · 2024-12-18T16:22:26Z

TagsCloudContainer.ConsoleUi/Runner/TagsCloudContainerUi.cs

+{
+    private readonly IReadOnlyCollection<IHandler> handlers;
+
+    private readonly IReadOnlySet<Type> optionsTypes = new HashSet<Type>()


Давай их тоже получать через DI

Сделал

dmitgaranin · 2024-12-18T16:23:51Z

TagsCloudContainer.ConsoleUi/Runner/TagsCloudContainerUi.cs

+
+    private void Handle(IOptions options)
+    {
+        var message = string.Empty;


А при каком раскладе возможна ситуация, что после запуска программа не отработает? И правда ли такая возможность должна быть? Если правда, то может ошибку кидать?

Переделал. Сейчас вывожу сообщение о том что нет нужной команды

dmitgaranin · 2024-12-18T16:54:14Z

TagsCloudContainer.Tests/TagsCloudVisualization/WeigherWordSizerTests.cs

+using TagsCloudContainer.TagsCloudVisualization.Logic.SizeCalculators;
+using TagsCloudContainer.TagsCloudVisualization.Models.Settings;
+using TagsCloudContainer.TagsCloudVisualization.Providers.Interfaces;
+


Не забудь написать тесты на всю логику, которую требуется сделать по заданию)

Добавил тестов

dmitgaranin · 2024-12-18T16:59:43Z

TagsCloudContainer.ConsoleUi/Handlers/ExitHandler.cs

+{
+    public bool TryExecute(IOptions options, out string result)
+    {
+        if (options is ExitOptions exitOptions)


Предлагаю подумать, можно ли переделать архитектуру или подход на полиморфизме?

Запрещено использовать операторы if, switch, ?: и прочие условные операторы, если их можно заменить полиморфизмом.

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

Если переделывать, то можно это все убрать и сделать сильно проще. Достаточно не требовать от пользователя ввода данных во время работы приложения, а требовать конфигурировать запуск exe'шника параметрами запуска. Это сразу упростит код, уберет if'ы, появится примеры запуска программы в ридми с разными параметрами, а также для генерации облака потребуется только одно нажатие enter)

dmitgaranin · 2024-12-18T17:06:46Z

TagsCloudContainer.TagsCloudVisualization/Logic/SizeCalculators/WeigherWordSizer.cs

+    public IReadOnlyCollection<ViewWord> CalculateWordSizes(IReadOnlyDictionary<string, int> wordFrequencies,
+        int minSize = 8, int maxSize = 24)
+    {
+        if (wordFrequencies is null || wordFrequencies.Count == 0)


В каких случаях wordFrequencies может быть null?

Изменил. Таких случаев сейчас нет

dmitgaranin · 2024-12-18T17:09:06Z

TagsCloudContainer.TextAnalyzer/TextAnalyzerModule.cs

+    protected override void Load(ContainerBuilder builder)
+    {
+        var configuration = new ConfigurationBuilder()
+            .AddJsonFile("appsettings.json")


Предлагаю попробовать не завязываться на внешние файлы. Хочется уметь конфигурировать запуск программы только из единой точки

Переделал. Получаю путь программно

dmitgaranin · 2024-12-18T17:10:20Z

TagsCloudContainer.ConsoleUi/Program.cs

+var app = builder.Build();
+using var scope = app.BeginLifetimeScope();
+var appRunner = scope.Resolve<ITagsCloudContainerUi>();
+appRunner.Run();


Можешь добавить в реп сгенерированные примеры? И какой-нибудь мануал, как запускать твою программу? Мб с примером, чтобы я его просто скопировал, вставил и все заработало? Любые варианты удобнее принимаются)

Добавил Readme.md и примеры в папке Examples

…ency Injection

…oft Dependency Injection

…и построении круга

…кации размещения и логики расстановки прямоугольников

- Изменены интерфейсы - Выделен отдельный обработчик для настроек путей файлов - Добавлены проверки перед изменением настроек

… обработчиков

…indows

…кода

dmitgaranin · 2024-12-23T12:19:33Z

TagsCloudContainer.ConsoleUi/Readme.md

@@ -0,0 +1,146 @@
+## Мануал по использованию консольного приложения Tags Cloud Container
+
+Приложение позволяет генерировать облака тегов из текстовых файлов с возможностью настройки различных параметров.


Мануал просто супер) очень подробно и детально! 🔥🔥🔥

dmitgaranin · 2024-12-23T12:21:03Z

TagsCloudContainer.TagsCloudVisualization/Extensions/ConfigureServicesExtensions.cs

+{
+    public static IServiceCollection AddTagsCloudVisualization(this IServiceCollection services)
+    {
+        services.AddTransient<ILayouter, Layouter>();


А в чем разница этих трех lifecycle? Почему решил использовать именно в таком сочетании?

dmitgaranin · 2024-12-23T12:22:47Z

TagsCloudContainer.TagsCloudVisualization/Logic/Visualizers/WordCloudVisualizer.cs

+
+    private Bitmap VisualizeWords(Bitmap bitmap, IReadOnlyCollection<ViewWord> viewWords, ImageSettings imageSettings)
+    {
+        var container = containerCreator.GetOrNull() ?? throw new ArgumentNullException(nameof(containerCreator));


Может он все-таки не null?

Если мы не зарегистрировали сервис, который нужен, то получим null. Я решил что лучше пусть падает там где получаем, чем там где используем

А можно ли научить DI при старте проверять, зарегестрированы ли все зависимости?)

dmitgaranin · 2024-12-23T12:24:48Z

TagsCloudContainer.Tests/TagsCloudVisualizationTests/CircularCloudLayouterTests.cs

+        var center = imageSettings.Size.Center();
+        var circularCloudLayouter = new Layouter(imageSettingsProvider, rectanglePlacementStrategy);
+
+        var rectangles = sizes.Select(size => circularCloudLayouter.PutNextRectangle(size)).ToList();


А нам правда нужно делать ToList?

dmitgaranin · 2024-12-23T12:37:18Z

TagsCloudContainer.TextAnalyzer/TagsCloudContainer.TextAnalyzer.csproj

+        </AssemblyAttribute>
+    </ItemGroup>
+    <ItemGroup>
+      <Folder Include="Logic\" />


Лишнее

dmitgaranin · 2024-12-23T12:39:39Z

TagsCloudContainer.ConsoleUi/ConsoleClientModule.cs

@@ -0,0 +1,27 @@
+using Autofac;


Еще не хватает теста, проверяющего работу всей программы

Добавь несколько более крупных тестов, проверяющих работу всей программы в сборе.

Добавил

Ilfir Sakhabutdinov added 14 commits December 18, 2024 17:54

Add. Созданы проекты для работы

fcfef16

Add. Тесты на анализатор размеров слов

abccc22

Add. Тесты на CircularCloudLayouter

6949841

Add. Модели для работы с визуализацией слов

01469e5

Add. Провайдер для доставки и изменения настроек визуализации

7506f8c

Add. Логика визуализации слов

11a771c

Add. Модели для анализа текста

875b3e7

Add. Провайдер для изменения настроек анализа слов

9799764

Add. Логика анализа текста

374e8b8

Add. Options консольного клиента

cc2d9ec

Add. Обработчики команд для консольного клиента

b280405

Add. Реализация консольного цикла UI

3cf4786

Add. DI консольного клиента

b036085

Fix. Убрал локальный путь к MyStem

3d0f01e

dmitgaranin reviewed Dec 18, 2024

View reviewed changes

Ilfir Sakhabutdinov added 15 commits December 22, 2024 23:56

Add. Тесты для проекта "TextAnalyzer"

6555faf

Update. Изменены тесты для класса "CircularCloudLayouter"

b7d1f84

Update. Изменены тесты для класса "WeigherWordSizer"

dafbd1f

Add. Добавлены тесты для класса "LayoutCreator"

12d3867

Add. Добавлены тесты для класса "SpiralPlacementStrategy"

576342e

Update. Проект "TextAnalyzer" переведен с Autofac на Microsoft Depend…

ed52137

…ency Injection

Add. Добавлены внешние ресурсы

89a8163

Update. Изменен уровень доступа модели для тестирования

0c30b05

Update. В проекте "TextAnalyzer" убраны неиспользуемые методы

a518a02

Fix. Исправлена ошибка с сохранением только уникальных слов

811186a

Update. Проект "TagsCloudVisualization" переведен с Autofac на Micros…

168c4bb

…oft Dependency Injection

Add. Добавлены настройки путей к исходному и конечному файлам

5bf1148

Add. Добавлен создатель "Layouter" для получения нового экземпляра пр…

0218dcb

…и построении круга

Update. Разделение ответственности. Выделены функции для получения ло…

f90b190

…кации размещения и логики расстановки прямоугольников

Fix. Убрана лишняя проверка на null

17995b9

Ilfir Sakhabutdinov added 11 commits December 23, 2024 00:34

Update. Переделана регистрация в di сервисов из библиотек

3dbbf15

Update. Изменены опции для отдельной работы с настройками и генерацией

33ff049

Update. Изменены обработчики.

73fa95a

- Изменены интерфейсы - Выделен отдельный обработчик для настроек путей файлов - Добавлены проверки перед изменением настроек

Update. Обработка команд заменена на использование специализированных…

beb0653

… обработчиков

Update. Регистрация обработчиков и опций в di

f8138dd

Update. Переход на net8.0-windows для зависимостей, специфичных для W…

a956e96

…indows

Fix. Правки проверок изменений опций для настроек в консольном клиенте

2afad1f

Fix. Небольшие исправления после инспекции кода связанные с чистотой …

0ca0484

…кода

Fix. Правки связанные с пробелами и отступами

3bf28fa

Add. Более подробное описание опции в HelpText

fcacd8e

Add. Примеры и мануал

3e9a479

dmitgaranin reviewed Dec 23, 2024

View reviewed changes

Ilfir Sakhabutdinov added 6 commits December 24, 2024 23:55

Update. Конфигурирование через параметры запуска

2cdc617

Fix. Поправка жизненного цикла сервиса

95ed38d

Fix. Удаление ненужного вызова .ToList()

bee0e72

Fix. Форматирование стиля кода (добавление пробелов)

c63a0bb

Fix. Удаление папки Logic из ItemGroup

ce3021e

Add. Добавлены тесты, проверяющие работу всей программы в сборке

99843fc

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Сахабутдинов Ильфир #10

Сахабутдинов Ильфир #10

ILFirV-V commented Dec 18, 2024

dmitgaranin Dec 18, 2024

ILFirV-V Dec 22, 2024

dmitgaranin Dec 18, 2024

ILFirV-V Dec 22, 2024

dmitgaranin Dec 18, 2024

dmitgaranin Dec 18, 2024

ILFirV-V Dec 22, 2024

dmitgaranin Dec 18, 2024

ILFirV-V Dec 22, 2024

dmitgaranin Dec 18, 2024

ILFirV-V Dec 22, 2024

dmitgaranin Dec 18, 2024

dmitgaranin Dec 23, 2024

dmitgaranin Dec 18, 2024

ILFirV-V Dec 22, 2024

dmitgaranin Dec 18, 2024

ILFirV-V Dec 22, 2024

dmitgaranin Dec 18, 2024

ILFirV-V Dec 22, 2024

dmitgaranin Dec 23, 2024

dmitgaranin Dec 23, 2024

dmitgaranin Dec 23, 2024

ILFirV-V Dec 24, 2024

dmitgaranin Dec 25, 2024

dmitgaranin Dec 23, 2024

dmitgaranin Dec 23, 2024

ILFirV-V Dec 24, 2024

dmitgaranin Dec 23, 2024

ILFirV-V Dec 24, 2024

		@@ -0,0 +1,146 @@
		## Мануал по использованию консольного приложения Tags Cloud Container

		Приложение позволяет генерировать облака тегов из текстовых файлов с возможностью настройки различных параметров.

Сахабутдинов Ильфир #10

Are you sure you want to change the base?

Сахабутдинов Ильфир #10

Conversation

ILFirV-V commented Dec 18, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment