Ослина Анастасия #15

malinaboky · 2024-12-18T17:16:09Z

Folleach

Резюмируя, вышел прикольный код!
Мне нравится, что в нём не забывается уничножение IDisposable переменных. А классы, которые делают лишь одну вещь, удобно читать, это здорово! Есть конечно местами забытые ключевые слова но это совсем уж мелочи. Тем не менее, напомнил о них на всякий случай в комментариях ниже.

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

Сейчас интересно увидеть, как будут встроены дополнительные реализации интерфейсов

TagsCloudVisualization/TagsCloudVisualization.csproj

TagsCloudVisualization/ConsoleCommands/Options.cs

Folleach · 2024-12-20T15:29:33Z

TagsCloudVisualization/Program.cs

+
+public class Program
+{
+    static void Main(string[] args)


Странно, что при неправильном вызове приложения выводится Exception в конце подсказки

~~resolved~~

upd: нет, не совсем.

Мне не понятно, почему команда сообщает о двух ошибках одновременно:

Required option 'i, inputFilePath' is missing. Required option 'o, outputDirectory' is missing.

Value cannot be null. (Parameter 'instance')

Кажется, в данном случае правильно показать только требуемые параметры, а не null reference exception.
В то же время важно сохранить настоящие исключения, чтобы в случае ошибок было понятно, как исправлять

TagsCloudVisualization/ConsoleCommands/Options.cs

TagsCloudVisualization/WordPreprocessors/WordValidators/DefaultWordValidator.cs

Folleach · 2024-12-20T16:53:06Z

TagsCloudVisualization/WordPreprocessors/DefaultWordPreprocessor.cs

+    public Dictionary<string, int> ProcessWords(IEnumerable<string> words)
+    {
+        return words.SelectMany(word => word.Split())
+            .Where(word => !string.IsNullOrEmpty(word))
+            .GroupBy(word => word.ToLower())
+            .Where(group => wordValidator.IsValid(group.Key))
+            .OrderByDescending(group => group.Count())
+            .ToDictionary(group => group.Key, group => group.Count());
+    }


У Dictionary разве определён порядок?

TagsCloudVisualization/WordPreprocessors/FontCreators/IFontCreator.cs

TagsCloudVisualization/Layouters/CircularCloudLayouter.cs

Folleach · 2024-12-20T17:03:27Z

TagsCloudVisualization/ContainerConfig.cs

+        var builder = new ContainerBuilder();
+        builder.RegisterInstance(options).AsSelf().SingleInstance();


Меня несколько смущает, что объект Options это синглтон, который позволяет себя модифицировать.
Какой-то из компонентов может воспользоваться этим и поменять некоторые значения, ожидая, что у него личный экземпляр настроек, словно локальная переменная.

Несмотря на то, что по умолчанию Autofac регистрирует как per dependency, тут зарегистрирован конкретный объект options. Autofac не будет его копировать, поэтому шанс случайной модификации остаётся возможен

malinaboky · 2024-12-23T18:54:53Z

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

Добавила конфиг для минимального и максимального размера шрифта

malinaboky · 2024-12-23T19:05:35Z

В итоге реализовала два пункта в перспективу: чтение разных входных форматов (txt, doc, docx) и разные генераторы цветов (рандом и градиент), пример градиента подгрузила

Folleach · 2024-12-27T08:42:20Z

TagsCloudVisualization/WordPreprocessors/DefaultWordPreprocessor.cs

+
+    public IEnumerable<Tuple<string, int>> ProcessTextToWords(string text)
+    {
+        var analysis = myStem.Analysis(text).Split("\r\n", StringSplitOptions.RemoveEmptyEntries);


Если создать файл в unix-like операционной системе, то переносом строк будет только символ \n.

Folleach · 2024-12-27T09:25:33Z

TagsCloudVisualizationTests/WordValidatorTests.cs

+    [TestCaseSource(nameof(WordValidatorSourceTestCases))]
+    public void IsValid_ShouldValidateCorrectly(WordInfo wordInfo, bool isValid)
+    {
+        new DefaultWordValidator().IsValid(wordInfo).Should().Be(isValid);
+    }
+
+    private static IEnumerable<TestCaseData> WordValidatorSourceTestCases()
+    {
+        yield return new TestCaseData(new WordInfo { Grammeme = "CONJ", Lemma = "text" }, false);
+        yield return new TestCaseData(new WordInfo { Grammeme = "INTJ", Lemma = "text" }, false);
+        yield return new TestCaseData(new WordInfo { Grammeme = "PART", Lemma = "text" }, false);
+        yield return new TestCaseData(new WordInfo { Grammeme = "PR", Lemma = "text" }, false);
+        yield return new TestCaseData(new WordInfo { Grammeme = "SPRO", Lemma = "text" }, false);
+        yield return new TestCaseData(new WordInfo { Grammeme = "V", Lemma = "text" }, true);
+        yield return new TestCaseData(new WordInfo { Grammeme = "NUM", Lemma = "text" }, true);
+        yield return new TestCaseData(new WordInfo { Grammeme = "A", Lemma = "text" }, true);
+    }


В IDE тесты отображаются, будто их 2.
Хотя на деле больше

Folleach · 2024-12-27T11:38:01Z

TagsCloudVisualizationTests/WordPreprocessorTests.cs

+    [Test]
+    public void ProcessTextToWords_ShouldContainOnlyNotBoringWordsReducedToInitialFormWithLowerCase()
+    {
+        var words = ReadTestFile();
+        var result = new List<Tuple<string, int>>
+        {
+            new("кошка", 2),
+            new("собака", 3)
+        };
+
+        var wordProcessor = Scope.Resolve<IWordPreprocessor>();
+        wordProcessor.ProcessTextToWords(words).Should().BeEquivalentTo(result);
+
+    }
+
+    [Test]
+    public void ProcessTextToWords_ShouldContainCorrectCountOfWords()
+    {
+        var words = ReadTestFile();
+
+        var wordProcessor = Scope.Resolve<IWordPreprocessor>();
+        wordProcessor.ProcessTextToWords(words).Should().HaveCount(2);
+    }


Кажется, тест ProcessTextToWords_ShouldContainCorrectCountOfWords дублирует тест ProcessTextToWords_ShouldContainOnlyNotBoringWordsReducedToInitialFormWithLowerCase

Folleach · 2024-12-27T11:45:47Z

TagsCloudVisualizationTests/ConsoleAppTests.cs

+    private ConsoleApp ConsoleApp { get; set; }
+    private ILifetimeScope Scope { get; set; }
+    private IWordPreprocessor WordPreprocessor { get; set; }
+    private ICloudLayouter Layouter { get; set; }
+    private Options Options { get; set; }


А в проекте с тестами nullable context не выключен )

Folleach · 2024-12-27T12:22:26Z

TagsCloudVisualization/WordPreprocessors/FontCreators/DefaultFontCreator.cs

+    public Font CreateFont(int fontSizeFactor)
+    {
+        var fontSize = Math.Min(Math.Max(minFontSize, fontSizeFactor), maxFontSize);
+
+        return new Font(fontName, fontSize);
+    }


Было бы здорово интерполировать размер шрифта.

Например, для { ("попугай", 3), ("кошка", 2), ("собака", 1) }
при maxFontSize = 10, minFontSize = 5
размер шрифта был таким:

"попугай" -> 10 "кошка" -> 7.5 "собака" -> 5

Сейчас просто на маленьких количествах разница не особо заметна

Homework first step

cf66fc4

Folleach reviewed Dec 20, 2024

View reviewed changes

Fix problems

a8cea67

malinaboky added 3 commits December 24, 2024 17:35

Add bitmap processor

82f5655

Add rectangle size calculator

584a23e

Add tests

7c68d9d

Folleach reviewed Dec 27, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ослина Анастасия #15

Ослина Анастасия #15

malinaboky commented Dec 18, 2024

Folleach left a comment •

edited

Loading

Folleach Dec 20, 2024

Folleach Dec 27, 2024 •

edited

Loading

Folleach Dec 20, 2024

Folleach Dec 27, 2024

Folleach Dec 20, 2024 •

edited

Loading

Folleach Dec 27, 2024

malinaboky commented Dec 23, 2024 •

edited

Loading

malinaboky commented Dec 23, 2024

Folleach Dec 27, 2024

Folleach Dec 27, 2024

Folleach Dec 27, 2024

Folleach Dec 27, 2024

Folleach Dec 27, 2024

		var builder = new ContainerBuilder();
		builder.RegisterInstance(options).AsSelf().SingleInstance();

Ослина Анастасия #15

Are you sure you want to change the base?

Ослина Анастасия #15

Conversation

malinaboky commented Dec 18, 2024

Folleach left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Folleach Dec 27, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Folleach Dec 20, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

malinaboky commented Dec 23, 2024 • edited Loading

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

Folleach left a comment •

edited

Loading

Folleach Dec 27, 2024 •

edited

Loading

Folleach Dec 20, 2024 •

edited

Loading

malinaboky commented Dec 23, 2024 •

edited

Loading