Бабин Георгий #168

tripples25 · 2024-02-04T12:47:03Z

Внесённые изменения

Классы приложения переведены на использование паттерна Result.
Обработаны основные исключительные ситуации.
Добавлены дополнительные тесты.
Файл README.MD обновлён в соответствии с текущим состоянием проекта.

the-homeless-god · 2024-02-05T11:06:13Z

TagsCloudResult/README.md

+# Примеры раскладки с разными параметрами
+## Rectangles = 500; deltaAngle = 5°; distance = 2
+![example1](https://github.com/tripples25/tdd/blob/8b58d8f0592e3592d28af7ce6bfe3c817a716d16/TagsCloudVisualization/layoutImages/new%20500%20rects%20angle%205%20distance%202.Png?raw=true)
+## Rectangles = 100; deltaAngle = 5°; distance = 2
+![example1](https://github.com/tripples25/tdd/blob/8b58d8f0592e3592d28af7ce6bfe3c817a716d16/TagsCloudVisualization/layoutImages/new%20angle%205%20distance%202.Png?raw=true)
+## Rectangles = 100; deltaAngle = 10°; distance = 2
+![example2](https://github.com/tripples25/tdd/blob/8b58d8f0592e3592d28af7ce6bfe3c817a716d16/TagsCloudVisualization/layoutImages/new%20angle%2010%20distance%202.Png?raw=true)
+## Rectangles = 100; deltaAngle = 5°; distance = 0.5
+![example3](https://github.com/tripples25/tdd/blob/8b58d8f0592e3592d28af7ce6bfe3c817a716d16/TagsCloudVisualization/layoutImages/new%20angle%205%20distance%200.5.Png?raw=true)
+## Rectangles = 100; deltaAngle = 1°; distance = 0.5
+![example4](https://github.com/tripples25/tdd/blob/8b58d8f0592e3592d28af7ce6bfe3c817a716d16/TagsCloudVisualization/layoutImages/new%20angle%201%20distance%200.5.Png?raw=true)


Спасибо за README, рекомендую отформатировать только его и перенести из заголовков второго уровня информацию про параметры, можно использовать таблички

the-homeless-god · 2024-02-05T11:06:39Z

TagsCloudResult/TagsCloudVisualization.WFApp/Actions/ImageSettingsAction.cs

+        if (dialog == DialogResult.OK)
+            imageHolder.RecreateImage(imageSettings);
+    }
+}


В конце файлов нужны пустые строки, можно настроить EditorConfig

the-homeless-god · 2024-02-05T11:06:54Z

TagsCloudResult/TagsCloudVisualization.WFApp/Actions/ImageSettingsAction.cs

+    public void Perform()
+    {
+        var dialog = SettingsForm.For(imageSettings).ShowDialog();
+        if (dialog == DialogResult.OK)


Старайся использовать { } и избегать однострочных выражений

the-homeless-god · 2024-02-05T11:07:41Z

TagsCloudResult/TagsCloudVisualization.WFApp/Actions/SourceSettingsAction.cs

+    }
+
+    public MenuCategory Category => MenuCategory.Settings;
+    public string Name => "Источник...";


Я бы утащил все тексты в Resources Dictionary

the-homeless-god · 2024-02-05T11:08:19Z

TagsCloudResult/TagsCloudVisualization.WFApp/Program.cs

+        // container.RegisterType<TxtTextReader>().Keyed<TextReader>(FileExtension.Txt);
+        // container.RegisterType<DocTextReader>().Keyed<TextReader>(FileExtension.Doc).Keyed<TextReader>(FileExtension.Docx);


Избегаем закомменченого кода

the-homeless-god · 2024-02-05T11:10:09Z

TagsCloudResult/TagsCloudVisualization/Common/Result.cs

+    public class None
+    {
+        private None()
+        {
+        }
+    }


Не понимаю логику

Также классы лучше разнести под namespace в разных файлах

Result я брал из примера, что предоставили нам в Контуре. None в данном случае используется для представления возвращаемого значения void-методов. Какой-то дополнительной логики этот класс более не несёт.

the-homeless-god · 2024-02-05T11:11:22Z

TagsCloudResult/layoutImages/tags.Png

в названии файлов не следует использовать пробелы, также .png вместо .Png

the-homeless-god · 2024-02-06T12:48:26Z

TagsCloudResult/README.md

+
+Приложение для создания облак тегов на основе текста.
+
+![example1](https://github.com/tripples25/fp/blob/1a4585180188efc9c6a35736650295a4e470f916/TagsCloudResult/layoutImages/sample.png?raw=true)


В квадратных скобках переименовать название, у самого файла сделать адекватный путь и название без camelCase к примеру

the-homeless-god · 2024-02-06T12:50:00Z

TagsCloudResult/TagsCloudVisualization.WFApp/Common/PictureBoxImageHolder.cs

+        Refresh();
+        Application.DoEvents();


Почему так решил делать?

Думал запускать цикл при отрисовке тегов, чтобы отрисовывать в real-time. В целом, он теперь не нужен, да и не рекомендуем к использованию, если верить документации и форумам, поэтому убрал.

the-homeless-god · 2024-02-06T12:50:44Z

TagsCloudResult/TagsCloudVisualization.WFApp/Infrastructure/UIActionExtensions.cs

+        return items;
+    }
+
+    private static ToolStripMenuItem CreateTopLevelMenuItem(MenuCategory category, IList<IUiAction> items)


Почему не фабричный метод, а статические методы?

Ну, я не рассчитывал на создание других клиентов или других типов/видов кнопок, поэтому пренебрёг пластичностью и просто создал удобные для себя статические расширения.
Создал фабрику, правда, в качестве результата там пока ToolStripItem. Это позволит создавать другие типы кнопок, унаследованные от ToolStripItem, но не подменять их другими классами не связанными с ним.

the-homeless-god · 2024-02-06T12:51:07Z

TagsCloudResult/TagsCloudVisualization.WFApp/Program.cs

+
+        container.RegisterAssemblyTypes(typeof(IUiAction).Assembly).AsImplementedInterfaces();
+
+        container.RegisterType<PictureBoxImageHolder>().As<PictureBoxImageHolder, IImageHolder>().SingleInstance();


Было бы круто добавить здесь комментариев

the-homeless-god · 2024-02-06T12:51:36Z

TagsCloudResult/TagsCloudVisualization/CloudLayouters/CircularCloudLayouter.cs

+    private Point MovePointAlongXAxis(Point point, Size rectangleSize)
+    {
+        var offsetToCenter = new Size(-rectangleSize.Width / 2, -rectangleSize.Height / 2);
+        var offsetX = Math.Sign(Center.X + offsetToCenter.Width - point.X);
+        while (Center.X + offsetToCenter.Width - point.X != 0 && offsetX != 0)
+        {
+            var newPoint = point.WithOffset(new Size(offsetX, 0));
+            if (!IsPlaceSuitableForRectangle(new Rectangle(newPoint, rectangleSize)))
+                break;
+
+            point = newPoint;
+        }
+
+        return point;
+    }
+
+    private Point MovePointAlongYAxis(Point point, Size rectangleSize)
+    {
+        var offsetToCenter = new Size(-rectangleSize.Width / 2, -rectangleSize.Height / 2);
+        var offsetY = Math.Sign(Center.Y + offsetToCenter.Height - point.Y);
+        while (Center.Y + offsetToCenter.Height - point.Y != 0 && offsetY != 0)
+        {
+            var newPoint = point.WithOffset(new Size(0, offsetY));
+            if (!IsPlaceSuitableForRectangle(new Rectangle(newPoint, rectangleSize)))
+                break;
+
+            point = newPoint;
+        }
+
+        return point;


Дублирование кода, магические цифры, не хватает SRP

the-homeless-god · 2024-02-06T12:53:12Z

TagsCloudResult/TagsCloudVisualization/TagsCloudVisualizator.cs

+                if (!tagDraw.IsSuccess)
+                    return tagDraw;


Почему так?

Потому что по задумке метод должен вернуть Result с ошибкой на первом теге, который не сумел отрисоваться. Переписал на LINQ, дополнительно выделив методы, убрал часть вложенности.

the-homeless-god · 2024-02-06T12:53:34Z

TagsCloudResult/TagsCloudVisualization/TagsCloudVisualizator.cs

+
+    private Color GetTagColor(Tag tag)
+    {
+        return tag.Coeff > 0.75 ? tagsSettings.PrimaryColor : tag.Coeff > 0.35 ? tagsSettings.SecondaryColor 


Избавиться от однострочности, нечитаемо, возможно даже стоит добавить комментов

Переписал, использовав константы и оператор switch.

the-homeless-god · 2024-02-06T12:53:49Z

TagsCloudResult/TagsCloudVisualization/TextReaders/DocTextReader.cs

+    public DocTextReader(SourceSettings settings) : base(settings)
+    {
+    }


А зачем пустой?

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

the-homeless-god · 2024-02-06T12:54:09Z

TagsCloudResult/TagsCloudVisualization/TextReaders/TxtTextReader.cs

+    public TxtTextReader(SourceSettings settings) : base(settings)
+    {
+    }


Тот же вопрос почему появляются пустые конструкторы?

the-homeless-god · 2024-02-06T12:54:54Z

TagsCloudResult/layoutImages/sample.png

Красивое 🚀

…awtags

… simplified methods

the-homeless-god

LGTM 🚀 само приложение не потестил, вопросов нет

the-homeless-god · 2024-02-07T13:18:07Z

TagsCloudResult/README.md

+
+Приложение для создания облак тегов на основе текста.
+
+![tagcloudsample](layoutImages/sample.png)


исправить tagcloudsample на tag_cloud_sample
исправить layoutImages/ на layout/images/

the-homeless-god · 2024-02-07T13:18:32Z

TagsCloudResult/TagsCloudVisualization.WFApp/Actions/SaveImageAction.cs

+        var res = dialog.ShowDialog();
+


Suggested change

var res = dialog.ShowDialog();

var res = dialog.ShowDialog();

the-homeless-god suggested changes Feb 5, 2024

View reviewed changes

tripples25 added 7 commits February 5, 2024 18:17

feat: transfered project from previous repository.

3cf612f

feat: changed classes on using Result<T> pattern.

be6ef3f

feat: added TextReaderFactory, refactor TagProvider

485e393

test: added tests on TagProvider

9d8a697

test: added StringExtensions tests

cf7cdb4

refactor: simplified TagsCloudVisualizator class

e0cea7d

refactor: removed commented out code

2dece1f

tripples25 force-pushed the master branch from 3b76842 to 2dece1f Compare February 5, 2024 13:38

tripples25 added 11 commits February 5, 2024 19:02

style: added blank spaces in the end of files

7fb7abc

chore: replaced outdated layout samples with the new one

1a45851

chore(readme): inserted new layout image and updated info

ff89cfb

refactor(result): split class in three separate files

d69990e

feat: localized texts in resources of WF app

67e5006

refactor: removed outdated line

57786f4

refactor(sourcesettings): moved validate methods to class

d79adc8

feat: added validate method to all settings, improved validating

2297cc6

refactor: removed unused usings

dac6341

test: added tests on visualizator

8135e85

refactor: removed unused usings and mainform name resource

a78cd63

the-homeless-god suggested changes Feb 6, 2024

View reviewed changes

chore(readme): fixed image link

2178244

tripples25 force-pushed the master branch from bf8ff30 to 2178244 Compare February 6, 2024 14:36

tripples25 added 7 commits February 6, 2024 20:04

refactor: added comments to program class, removed doitems call

2012eb4

refactor: replace abstract text reader class with interface

08ead61

refactor(visualizator): improved gettagbrush readability, improved dr…

f3ae8d3

…awtags

feat: created factory for menuitems

a8d5846

refactor: changed abstract factory name

e12b0de

refactor: moved strings to resources in both projects

b457dd5

refactor(layouters): removed duplications, create consts for numbers,…

3df2e35

… simplified methods

test: add few tests on settings classes

1ec244f

the-homeless-god approved these changes Feb 7, 2024

View reviewed changes

chore: minor style changes and readme edit

34d4ab7

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Бабин Георгий #168

Бабин Георгий #168

tripples25 commented Feb 4, 2024 •

edited

Loading

the-homeless-god Feb 5, 2024

the-homeless-god Feb 5, 2024

the-homeless-god Feb 5, 2024

the-homeless-god Feb 5, 2024

the-homeless-god Feb 5, 2024

the-homeless-god Feb 5, 2024

tripples25 Feb 5, 2024

the-homeless-god Feb 5, 2024

the-homeless-god Feb 6, 2024

the-homeless-god Feb 6, 2024

tripples25 Feb 6, 2024

the-homeless-god Feb 6, 2024

tripples25 Feb 6, 2024

the-homeless-god Feb 6, 2024

the-homeless-god Feb 6, 2024

the-homeless-god Feb 6, 2024

tripples25 Feb 6, 2024

the-homeless-god Feb 6, 2024

tripples25 Feb 6, 2024

the-homeless-god Feb 6, 2024

tripples25 Feb 6, 2024

the-homeless-god Feb 6, 2024

the-homeless-god Feb 6, 2024

the-homeless-god left a comment

the-homeless-god Feb 7, 2024

the-homeless-god Feb 7, 2024

		// container.RegisterType<TxtTextReader>().Keyed<TextReader>(FileExtension.Txt);
		// container.RegisterType<DocTextReader>().Keyed<TextReader>(FileExtension.Doc).Keyed<TextReader>(FileExtension.Docx);


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

		![example1](https://github.com/tripples25/fp/blob/1a4585180188efc9c6a35736650295a4e470f916/TagsCloudResult/layoutImages/sample.png?raw=true)


		container.RegisterAssemblyTypes(typeof(IUiAction).Assembly).AsImplementedInterfaces();

		container.RegisterType<PictureBoxImageHolder>().As<PictureBoxImageHolder, IImageHolder>().SingleInstance();


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

		![tagcloudsample](layoutImages/sample.png)

	var res = dialog.ShowDialog();

	var res = dialog.ShowDialog();

Бабин Георгий #168

Are you sure you want to change the base?

Бабин Георгий #168

Conversation

tripples25 commented Feb 4, 2024 • edited Loading

Внесённые изменения

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

the-homeless-god left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tripples25 commented Feb 4, 2024 •

edited

Loading