Ватлин Алексей #242

Alex-Vay · 2024-11-13T00:50:23Z

@SlavikGh0st
Решенные 3 задания с примерами работы программы (вид количество фигур/шаг/угол). Также при запуске и провале теста создаются соответствующие изображения в bin\Debug\net8.0+нужная директория внутри папок самих проектов

cs/TagsCloudVisualizationTests/CircularCloudLayouterTest.cs

+        var layouter = new CircularCloudLayouter(new(0, 0), 0.1, 0.1);
+        var size = new Size(width, height);
+
+        Action action = () => layouter.PutNextRectangle(size);


cs/TagsCloudVisualization/PointsGenerators/SpiralPointsGenerator.cs

cs/TagsCloudVisualization/CloudLayouter/CircularCloudLayouter.cs

+
+public class CircularCloudLayouter : ICircularCloudLayouter
+{
+    private Point center;


cs/TagsCloudVisualization/CloudLayouter/CircularCloudLayouter.cs

+        if (rectangleSize.Width <= 0 || rectangleSize.Height <= 0)
+            throw new ArgumentException($"{nameof(rectangleSize)} height and width must be greater than zero");
+        var rectangle = GetNextRectangle(rectangleSize);
+        while (rectangles.Any(rectangle.IntersectsWith)) //����� ������������ do while


cs/TagsCloudVisualization/CloudLayouter/CircularCloudLayouter.cs

+public class CircularCloudLayouter : ICircularCloudLayouter
+{
+    private Point center;
+    private List<Rectangle> rectangles = [];


cs/TagsCloudVisualization/CloudLayouter/CircularCloudLayouter.cs

+        return centerOfRectangle;
+    }
+
+    private Rectangle CreateRectangleWithCenter(Point center, Size rectangleSize)


cs/TagsCloudVisualization/CloudLayouter/CircularCloudLayouter.cs

+    private Rectangle GetNextRectangle(Size rectagleSize)
+    {
+        var rectanglePosition = spiral.GetNextPointPosition();
+        var centerOfRectangle = CreateRectangleWithCenter(rectanglePosition, rectagleSize);


cs/TagsCloudVisualization/CloudLayouter/CircularCloudLayouter.cs

+        return rectangle;
+    }
+
+    private Rectangle GetNextRectangle(Size rectagleSize)


cs/TagsCloudVisualizationTests/CircularCloudLayouterTest.cs

+    [TestCase(1, 0, TestName = "WhenHeightIsZero")]
+    [TestCase(-1, 1, TestName = "WhenWidthIsNegative")]
+    [TestCase(1, -1, TestName = "WhenHeightIsNegative")]
+    public void LayouterPutNextRectangle_ShouldThrowArgumentException(int width, int height)


cs/TagsCloudVisualizationTests/CircularCloudLayouterTest.cs

+    }
+
+    [Test]
+    public void FirstRectang_ShouldBeInCenter()


cs/TagsCloudVisualizationTests/CircularCloudLayouterTest.cs

+        var rectangleSize = new Size(10, 10);
+
+        var actualRectangle = layouter.PutNextRectangle(rectangleSize);
+        var expectedRectangle = new Rectangle(


cs/TagsCloudVisualizationTests/CircularCloudLayouterTest.cs

+
+    [Test]
+    [Repeat(10)]
+    public void AllRectanglesCenter_ShoulBeLikeInitCenter()


SlavikGh0st · 2024-11-14T10:35:42Z

cs/TagsCloudVisualizationTests/CircularCloudLayouterTest.cs

+
+    [Test]
+    [Repeat(10)]
+    public void Rectangles_ShouldNotHaveIntersects()


Тест-метод должен говорить, какой метод мы тестируем

cs/TagsCloudVisualizationTests/CircularCloudLayouterTest.cs

+
+    [Test]
+    [Repeat(10)]
+    public void RectanglesDensity_ShouldBeMax()


cs/TagsCloudVisualizationTests/CircularCloudLayouterTest.cs

+        var layouter = CloudGenerator.GenerateCloud(100);
+        rectanglesInTest = layouter.GetRectangles();
+
+        var actualCenter = GetCenterOfAllRectangles(rectanglesInTest);


cs/TagsCloudVisualizationTests/CircularCloudLayouterTest.cs

+        var layouter = CloudGenerator.GenerateCloud(100);
+        rectanglesInTest = layouter.GetRectangles();
+
+        var rectanglesArea = rectanglesInTest.Sum(rect => rect.Width * rect.Height);


cs/TagsCloudVisualizationTests/CircularCloudLayouterTest.cs

+
+    private double GetDistanceBetweenPoints(Point point1, Point point2)
+        => Math.Sqrt(Math.Pow(point1.X - point2.X, 2) + Math.Pow(point1.Y - point2.Y, 2));
+}


cs/TagsCloudVisualizationTests/CircularCloudLayouterTest.cs

cs/TagsCloudVisualization/Constans.cs

+    public const double LayoutStep = 0.1;
+    public const double LayoutAngleOffset = 0.1;
+    public const string ImagesDirectory = "images";
+}


cs/TagsCloudVisualization/CloudLayouter/CloudGenerator.cs

+
+public static class CloudGenerator
+{
+    public static CircularCloudLayouter GenerateCloud(int rectanglesNumber = Constans.RectanglesNumber)


cs/TagsCloudVisualization/CloudLayouter/CloudGenerator.cs

+{
+    public static CircularCloudLayouter GenerateCloud(int rectanglesNumber = Constans.RectanglesNumber)
+    {
+        var center = new Point(Constans.ImageWidth / 2, Constans.ImageHeight / 2);


cs/TagsCloudVisualization/CloudLayouter/CloudGenerator.cs

+        var cloudLayouter = new CircularCloudLayouter(center);
+        var random = new Random();
+        var rectangles = new Rectangle[rectanglesNumber];
+        rectangles = rectangles


cs/TagsCloudVisualization/CloudLayouter/CloudGenerator.cs

+                random.Next(Constans.MinRectangleSize, Constans.MaxRectangleSize))))
+            .ToArray();
+        return cloudLayouter;
+    }


cs/TagsCloudVisualization/Visualizers/SimpleCloudVisualizer.cs

+        IEnumerable<Rectangle> rectangles, 
+        Size bitmapSize,
+        string directory = Constans.ImagesDirectory,
+        string currentPath = null


cs/TagsCloudVisualization/Visualizers/SimpleCloudVisualizer.cs

+        bitmap.Save(path, ImageFormat.Jpeg);
+    }
+
+    private Color GetRandomColor()


SlavikGh0st · 2024-11-14T11:55:18Z

cs/TagsCloudVisualization/Visualizers/SimpleCloudVisualizer.cs

+        )
+    {
+        var bitmap = new Bitmap(bitmapSize.Width, bitmapSize.Height);
+        var graphics = Graphics.FromImage(bitmap);


Не диспоузим этот объект - потеря памяти

using graphics = ...

…енератор точек, создал readme, перенес тесты из отдельного проекта в основной

SlavikGh0st · 2024-11-17T05:55:24Z

cs/TagsCloudVisualization/CloudLayouter/CircularCloudLayouter.cs

+{
+    public Point Center { get; }
+    public List<Rectangle> GeneratedRectangles { get; }
+    private IPointsGenerator spiral { get; }


Вижу тебе понравились свойства)

Для spiral логичнее использовать поле с пометкой readonly - таким образом зафиксируем, что оно инициализируется в конструкторе, и больше в логике этого класса не меняется.

А сейчас это private свойство до которого никто не может достучаться)

SlavikGh0st · 2024-11-17T05:57:12Z

cs/TagsCloudVisualization/CloudLayouter/ICircularCloudLayouterExtensions.cs

+
+public static class ICircularCloudLayouterExtensions 
+{
+    public static void GenerateCloud(this ICircularCloudLayouter cloudLayouter, 


По код-стайлу, если в одну строку сигнатура метода не вмещается, то лучше все параметры с новой строки.

SlavikGh0st · 2024-11-17T05:59:27Z

cs/TagsCloudVisualization/PointsGenerators/SpiralPointsGenerator.cs

+    private readonly double step = 0.1;
+    private readonly double angleOffset = 0.1;
+    private readonly Point center;
+    private double angle = 0;


Лучше переименовать на currentAngle

SlavikGh0st · 2024-11-17T06:03:10Z

cs/TagsCloudVisualization/PointsGenerators/SpiralPointsGenerator.cs

+    private readonly Point center;
+    private double angle = 0;
+
+    public SpiralPointsGenerator(Point center) => this.center = center;


В этом случае как-то теряются "дефолтные значения", давай сделаем один конструктор, но с опциональными параметрами:

public SpiralPointsGenerator(Point center, double step = 0.1, double angleOffset = 0.1) { ...

соответственно инициализацию у полей можно будет убрать

private readonly double step; private readonly double angleOffset; private readonly Point center; private double angle;

SlavikGh0st · 2024-11-17T06:26:08Z

cs/TagsCloudVisualization/Tests/CircularCloudLayouterTests.cs

+
+namespace TagsCloudVisualization.Tests.CircularCloudLayouterTests;
+
+[TestFixture, Parallelizable]


Для Parallelizable нужно указать, что "не параллельно"

SlavikGh0st · 2024-11-17T16:34:33Z

cs/TagsCloudVisualization/CloudLayouter/CircularCloudLayouter.cs

+
+    public CircularCloudLayouter(Point center, int step, int angleOffset)
+    {
+        GeneratedRectangles = new List<Rectangle>();


центр забыл проинициализировать

можно наследоваться от других конструкторов и писать только различающуюся логику

public CircularCloudLayouter(Point center, int step, int angleOffset) : this(center) { spiral = new SpiralPointsGenerator(center, step, angleOffset); }

Решенные 3 задания с примерами работы программы

4480f67

SlavikGh0st reviewed Nov 13, 2024

View reviewed changes

cs/TagsCloudVisualizationTests/CircularCloudLayouterTest.cs Outdated

var layouter = new CircularCloudLayouter(new(0, 0), 0.1, 0.1);

var size = new Size(width, height);

Action action = () => layouter.PutNextRectangle(size);

This comment was marked as resolved.

Sign in to view

Alex-Vay added 2 commits November 13, 2024 16:36

Доработал код под соответсвие интерфейсу

9a47b32

Поправил namespace и тест

8aa0286