Дмитрий Леонтьев #26

Dary0302 · 2024-12-19T20:35:27Z

shiyois · 2024-12-20T09:09:50Z

di.sln

@@ -1,16 +0,0 @@
-


Кажись это было лишним

MSBUILD : error MSB1009: файл проекта не существует.

shiyois · 2024-12-20T12:26:59Z

TagsCloudVisualization/Settings/SpiralGeneratorSettings.cs

+
+public class SpiralGeneratorSettings(double angleOffset, double spiralStep, Point center)
+{
+    public double AngleOffset { get; private set; } = angleOffset;


Если нужна неизменяемость для этих целей существует record, предлагаю переделать

public record SpiralGeneratorSettings(double AngleOffset, double SpiralStep, Point Center);

shiyois · 2024-12-20T12:30:22Z

TagCloudConsoleApp/Program.cs

+    containerBuilder.RegisterType<TxtTextReader>().As<ITextReader>().SingleInstance();
+    containerBuilder.RegisterType<ImageSaver>().As<IImageSaver>().SingleInstance();
+    containerBuilder.RegisterType<TagCloudImageGenerator>();
+    //containerBuilder.RegisterType<BoringWordsTextFilter>().As<ITextFilter>().SingleInstance();


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

shiyois · 2024-12-21T07:04:49Z

TagsCloudVisualization/Client/Client.cs

+
+namespace TagsCloudVisualization.Client;
+
+public class Client(string[] args) : IClient


Относится к настройкам консольного приложения?
Предлагаю перенести в TagCloudConsoleApp.csproj
И переименовать в SettingsProvider, сейчас не отражает суть и путает с существующим термином

shiyois · 2024-12-21T07:07:42Z

TagsCloudVisualization/Filter/BoringWordsTextFilter.cs

+{
+    public List<string> ApplyFilter(IEnumerable<string> text)
+    {
+        //TODO: implement boring words filter


Это же у нас обязательный пункт? https://github.com/kontur-courses/di-updated/blob/master/HomeExercise.md

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

shiyois · 2024-12-21T07:14:15Z

TagsCloudVisualization/Generator/SpiralPositionGenerator.cs

+
+namespace TagsCloudVisualization.Generator;
+
+public class SpiralPositionGenerator(SpiralGeneratorSettings settings) : IPositionGenerator


Предлагаю в названии отразить, что это именно Архимедова спираль

shiyois · 2024-12-21T07:35:05Z

TagsCloudVisualization/CloudLayouter/CircularCloudLayouter.cs

@@ -0,0 +1,77 @@
+using System.Drawing;


Предлагаю еще немного поработать над структурой проекта.
Парочку распространённых в проектах вещей:

Интерфейсы создавать в отдельной директории Interfaces

Модельки в Models

Использовать в названии множественное число, например не Filter, а Filters

В названии директорий использовать существительные. Например Draw - глагол. Я бы назвал Visualizatiuon и сложил бы туда связанную с отображением логику общую для всех клиентов

shiyois · 2024-12-21T07:36:59Z

TagsCloudVisualization/Extension/RectangleExtension.cs

+    public static bool IntersectsWithAnyOf(this Rectangle rectangle, IEnumerable<Rectangle> rectangles) =>
+        rectangles.Any(rectangle.IntersectsWith);
+
+    public static void ShiftAlongXAxis(this Rectangle rectangle, int shift)


Не нашел использования в коде. Для чего они нужны?

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

А не нужно было сделать это в предыдущей задаче?) На сколько помню, это было одно из требований к решению

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

shiyois · 2024-12-21T07:41:36Z

TagsCloudVisualization.Tests/ImageSaverTest.cs

+{
+    private CircularCloudLayouter cloudLayouter;
+    private RectangleDraftsman drawer;
+    private ImageSaver imageSaver;


"Non-nullable field 'imageSaver' is uninitialized. Consider declaring the field as nullable"

Предлагаю вынести получение настроек непосредственно в метод сохранения, а не на весь инстанс класса. Инициализировать SetUp

shiyois · 2024-12-21T07:43:24Z

TagsCloudVisualization.Tests/RectangleDraftsmanTest.cs

+    [TestCase(0, 1)]
+    public void Constructor_OnInvalidArguments_ThrowsArgumentException(int width, int height)
+    {
+        Action action = () => new RectangleDraftsman(width, height);


Можно просто var

shiyois · 2024-12-21T07:45:30Z

TagsCloudVisualization.Tests/RectangleDraftsmanTest.cs

+    [Test]
+    public void CreateImage_WhenListOfRectanglesIsNull_ThrowsArgumentException()
+    {
+        var action = () => drawer.CreateImage(null);


null!

CreateImage никак не обрабатывает параметры входящие, сейчас так получилось что он бросит NullReference по ходу работы...

…t different image formats, and made it possible to influence the list of boring words that won't get into the cloud

shiyois · 2024-12-26T18:27:51Z

TagCloudConsoleApp/TagCloudConsoleApp.csproj

+    </ItemGroup>
+
+    <ItemGroup>
+      <Folder Include="Models\" />


Она используется?

shiyois · 2024-12-26T18:31:08Z

TagCloudConsoleApp/Program.cs

+using TagsCloudVisualization.Settings;
+
+var options = Parser.Default.ParseArguments<CommandLineOptions>(args).Value;
+var client = new SettingsProvider(options);


shiyois · 2024-12-26T18:55:25Z

TagsCloudVisualization/Models/Generators/TagCloudImageGenerator.cs

+            .OrderByDescending(words => words.Count())
+            .ToDictionary(words => words.Key, words => words.Count());
+
+        minWordCount = wordsFrequency.Values.Min();


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

shiyois · 2024-12-26T18:56:34Z

TagsCloudVisualization/Models/Generators/TagCloudImageGenerator.cs

+    IBitmapGenerator bitmapGenerator,
+    IEnumerable<ITextFilter> filters)
+{
+    private readonly int maxFontSize = textSettings.MaxFontSize;


Почему нельзя просто обратиться к textSettings.MaxFontSize, для чего создавать readonly поле?

shiyois · 2024-12-26T18:59:15Z

TagCloudConsoleApp/SettingsProvider/SettingsProvider.cs

+    public SettingsManager GetSettings()
+    {
+        return new(new BitmapGeneratorSettings(new(options.ImageWidth, options.ImageHeight),
+                GetColor(options.BackgroundColor),


Не обрабатываются null значенияPossible null reference argument for parameter 'color' in 'TagCloudConsoleApp. SettingsProvider. SettingsProvider. GetColor'

shiyois · 2024-12-26T19:09:30Z

TagsCloudVisualization/Models/Generators/TagCloudImageGenerator.cs

+    {
+        var text = fileReadersSelector.SelectFileReader().ReadText();
+
+        var wordsFrequency = filters


Не хватает тестов на работы фильтров

shiyois · 2024-12-26T19:13:00Z

TagsCloudVisualization/Models/Generators/TagCloudImageGenerator.cs

+            .Select(w => new TagWord(w.Key, GetFontSize(w.Value)));
+
+        var bitmap = bitmapGenerator.GenerateBitmap(words);
+        saver.SaveImageToFile(bitmap, saveSettings);


Не хватает тестов на основной сценарий.
Как вариант можно задуматься о скриншотом тестировании. Оно широко распространено для тестирования интерфейсов на фронте. Сравнивать эталонную картинку выкладки при прогоне тестов после внесения правок. Можно поискать такие библиотеки для .net

Сделал сравнение по пикселям

shiyois · 2024-12-26T19:14:35Z

TagCloudConsoleApp/Program.cs

+void RegisterFileReaders(ContainerBuilder containerBuilder)
+{
+    containerBuilder.RegisterType<TxtTextReader>().Keyed<ITextReader>(".txt");
+    containerBuilder.RegisterType<DocTextReader>().Keyed<ITextReader>(".doc");


Не хватает тестов на новую логику

shiyois · 2024-12-26T19:16:25Z

TagsCloudVisualization/Models/Generators/ArchimedeanSpiralPositionGenerator.cs

+using TagsCloudVisualization.Interfaces;
+using TagsCloudVisualization.Settings;
+
+namespace TagsCloudVisualization.Models.Generators;


Почему в папке models все генераторы, фильтры и т.д.

shiyois · 2024-12-26T19:23:10Z

TagsCloudVisualization/Models/Filters/BoringWordsTextFilter.cs

+        boringWords.Remove(word);
+    }
+
+    public void RemoveBoringWords(IEnumerable<string> words)


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

Я их добавил как возможности влиять на список скучных слов, просто не использовал в своём коде, что нужно делать в таком случае? Использовать их в тестах?

Dary0302 added 2 commits December 20, 2024 01:18

add solution and any interfaces

34bcc27

updated the tests, updated the architecture, added the first solution

b676d6f

shiyois reviewed Dec 20, 2024

View reviewed changes

added input.txt

009091f

shiyois reviewed Dec 20, 2024

View reviewed changes

shiyois reviewed Dec 21, 2024

View reviewed changes

Dary0302 added 3 commits December 22, 2024 21:53

corrected the comments, added the text of the tasks

d0849db

complete task, added the ability to support different formats, suppor…

7272c53

…t different image formats, and made it possible to influence the list of boring words that won't get into the cloud

unnecessary Usings have been deleted

8a1a935

shiyois reviewed Dec 26, 2024

View reviewed changes

Dary0302 added 2 commits December 27, 2024 15:52

correct comments

6c71c4a

add more tests

c7a0ba9

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Дмитрий Леонтьев #26

Дмитрий Леонтьев #26

Dary0302 commented Dec 19, 2024

shiyois Dec 20, 2024

shiyois Dec 20, 2024

shiyois Dec 20, 2024 •

edited

Loading

shiyois Dec 21, 2024 •

edited

Loading

shiyois Dec 21, 2024

Dary0302 Dec 21, 2024

shiyois Dec 21, 2024

shiyois Dec 21, 2024

shiyois Dec 21, 2024

Dary0302 Dec 21, 2024

shiyois Dec 26, 2024 •

edited

Loading

Dary0302 Dec 26, 2024

shiyois Dec 21, 2024

shiyois Dec 21, 2024

shiyois Dec 21, 2024

shiyois Dec 26, 2024

shiyois Dec 26, 2024

shiyois Dec 26, 2024

shiyois Dec 26, 2024

shiyois Dec 26, 2024

shiyois Dec 26, 2024

shiyois Dec 26, 2024

Dary0302 Dec 27, 2024

shiyois Dec 26, 2024

shiyois Dec 26, 2024 •

edited

Loading

shiyois Dec 26, 2024 •

edited

Loading

Dary0302 Dec 26, 2024


		namespace TagsCloudVisualization.Client;

		public class Client(string[] args) : IClient


		namespace TagsCloudVisualization.Generator;

		public class SpiralPositionGenerator(SpiralGeneratorSettings settings) : IPositionGenerator

Дмитрий Леонтьев #26

Are you sure you want to change the base?

Дмитрий Леонтьев #26

Conversation

Dary0302 commented Dec 19, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

shiyois Dec 20, 2024 • edited Loading

Choose a reason for hiding this comment

shiyois Dec 21, 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

shiyois Dec 26, 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

shiyois Dec 26, 2024 • edited Loading

Choose a reason for hiding this comment

shiyois Dec 26, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

shiyois Dec 20, 2024 •

edited

Loading

shiyois Dec 21, 2024 •

edited

Loading

shiyois Dec 26, 2024 •

edited

Loading

shiyois Dec 26, 2024 •

edited

Loading

shiyois Dec 26, 2024 •

edited

Loading