Зайцев ДА #28

Emi1337-ops · 2024-12-23T17:08:00Z

dmitgaranin · 2024-12-24T05:30:42Z

TagsCloudContainer/Visualizers/ImageCreater.cs

+
+        public void Generate(IEnumerable<Rectangle> items, string directory)
+        {
+            var image = new Bitmap(Width, Height);


Странная ошибка

dmitgaranin · 2024-12-24T05:31:05Z

TagsCloudContainer/Visualizers/ImageCreater.cs

+
+namespace TagsCloudContainer.Visualizers
+{
+    public class ImageCreater


Скорее Creator

dmitgaranin · 2024-12-24T05:31:31Z

TagsCloudContainer/Visualizers/ImageCreater.cs

+using System.Drawing.Imaging;
+
+namespace TagsCloudContainer.Visualizers
+{


Для чистоты кода можно использовать file-scoped namespace

dmitgaranin · 2024-12-24T05:32:17Z

TagsCloudContainer/Parsers/SimpleParser.cs

+        public IDictionary<string, int> Parse(string text)
+        {
+            var dict = new Dictionary<string, int>();
+            string pattern = @"\b[а-яА-ЯёЁa-zA-Z]+\b";


Если заранее компилировать regex, то он будет работать сильно быстрее)

dmitgaranin · 2024-12-24T05:32:56Z

TagsCloudContainer/Parsers/SimpleParser.cs

+            var words = Regex.Matches(text, pattern);
+
+            var wordArray = new List<string>();
+            for (int i = 0; i < words.Count; i++)


Где-то используются явные типы, а где-то var. Предлагаю писать единообразно

dmitgaranin · 2024-12-24T05:42:33Z

TagCloudContainerTests/FileReadTest.cs

@@ -0,0 +1,50 @@
+using FluentAssertions;


Также не нашел всех точек расширения:

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

dmitgaranin · 2024-12-24T05:44:52Z

TagsCloudContainer/Clients/ConsoleClient.cs

+using System.Linq;
+using System.Runtime.InteropServices;
+
+namespace TagsCloudContainer.Clients


Консольный клиент стоит разместить в отдельном проекте, т.к. у него домен ответственности - взаимодействие с пользователем, тогда как у логики - быть логикой) И дальше логику можно переиспользовать в разные клиенты (например, GUI), а к ним подключать еще и консольного - точно не хорошо

dmitgaranin · 2024-12-24T05:47:00Z

TagsCloudContainer/Filters/WordsFilter.cs

+        {
+            words = new HashSet<string>();
+
+            var projectDirectory =


Давай эти настройки вынесем в отдельный класс/рекорд, и будем получать через DI
Потому что сам класс фильтрации не должен ничего знать про захардкоженные места нахождения файлов. Например, с ними ты не сможешь написать тесты корректно

Даже возможно стоит сделать отдельный класс, который будет читать файлы, а конкретно сюда через DI получать уже прочитанный контент

dmitgaranin · 2024-12-24T05:48:30Z

TagsCloudContainer/Visualizers/ImageCreater.cs

+        public void Generate(IEnumerable<Rectangle> items, string directory)
+        {
+            var image = new Bitmap(Width, Height);
+            var g = Graphics.FromImage(image);


Это disposable объект

dmitgaranin · 2024-12-26T07:53:15Z

ConsoleClient/ConsoleClient.cs

+            PictureColors is null ? Constants.PictureColors : PictureColors
+           );
+
+        TagsCloudContainer.Program.Main(config);


Исполняемым проектом должен быть только проект консольного клиента. Также он должен быть точкой сборки всего DI, при этом какие-то заготовки по тому, как наполнять DI могут быть в самих либах (проектов с только логикой)

CloudContainer все еще <OutputType>Exe</OutputType>

dmitgaranin · 2024-12-26T07:54:09Z

ConsoleClient/ConsoleClient.cs

+
+    public static int Main(string[] args)
+    {
+        return CommandLineApplication.Execute<ConsoleClient>(args);


Стоит разделить ответственности: ответственность запуска и ответственность логики работы консольного клиента

Тут нужно было разделить класс на два:

Метод Main - про запуск приложения, отдельный класс, что-то типо EntryPoint.cs

ConsoleClient - про настройки и свойства (InputDirectory и остальные), а также метод запуски консольного клиента

В целом, можно было вынести только настройки и свойства в отдельный класс и уже было бы сильно хорошо

Зачем это делать? Две ответственности: про запуск приложения и про логику непосредственно приложения. Например, на консольный клиент может захотеться написать тест. В текущем виде он неразрывен от метода Main, что будет неправильно затаскивать в тесты

dmitgaranin · 2024-12-26T07:54:39Z

ConsoleClient/ConsoleClient.cs

@@ -0,0 +1,53 @@
+using McMaster.Extensions.CommandLineUtils;
+using System.Drawing;


Не используется

dmitgaranin · 2024-12-26T07:55:21Z

ConsoleClient/ConsoleClient.cs

+    private void OnExecute()
+    {
+        var config = new Config(
+            InputDirectory is null ? Constants.InputDirectory : InputDirectory,


Как объект может быть null, если на проекте включено <Nullable>enable</Nullable>?

dmitgaranin · 2024-12-26T07:56:21Z

+              using System.Data;
+              using TagsCloudContainer.FileReaders;
+              namespace TagCloudContainerTests

		@@ -0,0 +1,53 @@
		using McMaster.Extensions.CommandLineUtils;
		using System.Drawing;

+                  {
+                      Reader = new TxtFileReader();
+                      var filePath = Path.Combine(FolderPath, "wordsTXT.txt");
+                      var actual = Reader.Read(filePath);

+              public class WordsFilter : IFilter
+              {
+                  private HashSet<string> words;
+                  private Config config;

+                  public void AddStopWord(string word)
+                  {
+                          words.Add(word);

+                      return new Point(center.X - size.Width / 2, center.Y - size.Height / 2);
+                  }
+                  public static Point GetCenterPoint(Rectangle rect)

+                              existingRectangle.Bounds.IntersectsWith(rectangle));
+                  }
+                  public static Point GetCornerPoint(Point center, Size size)

+              using System.Drawing;
+              namespace TagsCloudContainer.WordClasses;
+              public record SizeWord(string Value, Size Size, Font font);

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

+              using TagsCloudContainer.Visualizers;
+              using TagsCloudContainer.WordSizer;
+              public class ApplicationRunner : IApplicationRunner

+                      );
+                  }
+                  public static IContainer BuildContainer(Config config)

+                  {
+                      var sizeList = new List<SizeWord>();
+                      for (int i = 10; i < 100; i += 10)

+                          for (int j = 5; j < 50; j += 5)
+                              sizeList.Add(GetDefaultSizeWord(new Size(i, j)));
+                      var rectangles = layouter.GetLayout(sizeList);

+                  public IEnumerable<SizeWord> GetSizes(IDictionary<string, int> words)
+                  {
+                      var screeenSize = config.PictureHeight * config.PictureWidth;

+                      var filter = new WordsFilter(config);
+                      var reader = new TxtFileReader();
+                      var filterWords =

+              namespace TagCloudContainerTests;
+              [TestFixture]
+              public class FilterTests

+                      {
+                          var heavierWord = GetSizeSquare(sized[i].Size);
+                          var lighterWord = GetSizeSquare(sized[i + 1].Size);
+                          heavierWord.Should().BeGreaterThan(lighterWord);

+                      sizeList.Add(GetDefaultSizeWord(new (20, 30)));
+                      var rectangles = layouter.GetLayout(sizeList);
+                      var firstRectangle = rectangles.First();

+                      {
+                          config = new Config();
+                          config.InputDirectory = @"C:\Users\dima0\source\repos\di-updated\TagsCloudContainer\Files\General.txt";
+                          config.OutputDirectory = @"C:\Users\dima0\source\repos\di-updated\TagsCloudContainer\Pictures\general.jpg";

+                          stopWords.All(x => filter.Contains(x)).Should().BeTrue();
+                          filter.Contains("я").Should().BeFalse();
+                          wordSizes.All(x => filter.Contains(x.Value)).Should().BeFalse();

Зайцев ДА #28

Are you sure you want to change the base?

Зайцев ДА #28

Conversation

Emi1337-ops commented Dec 23, 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

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

Emi1337-ops commented Dec 23, 2024 •

edited

Loading

+                          layout.All(x => x.font.Name == "Impact").Should().BeTrue();
+                          layout.Any(x => x.Value == "я").Should().BeTrue();
+                          using Image image = Image.FromFile(config.OutputDirectory);

		@@ -0,0 +1,7 @@
		namespace TagsCloudContainer;
		public static class Program