0

Пытаюсь создать карандаш как в Paint, а пользователь может выбрать размер карандаша. Если диаметр карандаша 1 всё идеально, а вот с 2 начинается непонятная ситуация, как на картинке.

Проблема в том что принцип работы карандаша должен быть вот таким. Допустим у меня радиус рисования 1. То есть карандаш должен рисовать на тех пикселях у которых центр находится внутри круга с радиусом 1px. А центр круга там где нажал пользователь. Более понятное объяснение на картинке.

введите сюда описание изображения

Это мой код

int diameter = (int)PencilSize.value;
for (int radius = 0; radius * 2 <= diameter; radius++)
{
    for (int angle = 0; angle < 360; angle++)
    {
        float x = OnPixel().x + radius * Mathf.Sin(angle * Mathf.Deg2Rad);
        float y = OnPixel().y + radius * Mathf.Cos(angle * Mathf.Deg2Rad);

        int drawX = (int)Mathf.Round(x);
        int drawY = (int)Mathf.Round(y);

        if (IsPixel(drawX, drawY))
        {
            WorkshopScript.SelectedLayer.SetPixel(drawX, drawY, ColorsScript.GetColor());
        }
    }
}

Метод OnPixel() показывает на каких координатах мышка (тип Vector2). Он работает правильно, проверено 100 раз. Я понимаю что у меня ошибка в округлении.
Как исправить эту ошибку?


П.Н.
Если понадобится вот ещё метод OnPixel.

public Vector2 OnPixel()
{
    Vector2 canvasMousePosition = new Vector2
    (
        Input.mousePosition.x - WorkshopScript.Canvas.transform.position.x + WorkshopScript.Canvas.GetComponent<RectTransform>().rect.width / 2,
        Input.mousePosition.y - WorkshopScript.Canvas.transform.position.y + WorkshopScript.Canvas.GetComponent<RectTransform>().rect.height / 2
    );
    Vector2 onPixel = new Vector2(canvasMousePosition.x / WorkshopScript.pixelLength, canvasMousePosition.y / WorkshopScript.pixelLength);
    return onPixel;
}

PencilSize это slider который определяет диаметр карандаша.
IsPixel это метод который проверяет есть ли в текстуре такой пиксель.
WorkshopScript.SelectedLayer это та текстура на котором можно рисовать.
ColorsScript.GetColor() это метод который возвращает выбранный пользователем цвет.

8
  • Попробуйте для округления вместо Round использовать Ceil 2 авг 2019 в 11:29
  • @ЮрийКозлов Тоже самое
    – eccs0103
    2 авг 2019 в 11:31
  • Unity3D я не знаю, но вот такой цикл for (float radius = 0; radius <= diameter / 2; radius++) выглядит очень странно для меня. Цикл с инкрементом по float? 2 авг 2019 в 11:36
  • @ЮрийКозлов я бы поставил его int но диаметр может быть 1 а 1/2=0,5 поэтому чтобы ошибок не было сделал float
    – eccs0103
    2 авг 2019 в 11:38
  • 1
    @ЮрийКозлов изменил на int
    – eccs0103
    2 авг 2019 в 11:42

1 ответ 1

1
for (int radius = 0; radius * 2 <= diameter; radius++)
{
    for (int angle = 0; angle < 360; angle++)
    {

Вот в этом-то и проблема. Вы перебираете не сами пиксели, а отвязанные от них точки, которые еще и распределены неравномерно (тем плотнее чем ближе к центру). У вас будет из-за и еще один баг - при очень большом радиусе некоторые пиксели не закрасятся.

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

Вот работающий код

int diameter = (int)PencilSize.value;
float radius = (float)diameter / 2;

for (float y = OnPixelInt().y - radius + 0.5f; y <= OnPixelInt().y + radius + 0.5f; y++)
{
    for (float x = OnPixelInt().x - radius + 0.5f; x <= OnPixelInt().x + radius + 0.5f; x++)
    {
        float distance = Mathf.Sqrt
        (
            Mathf.Pow(OnPixel().x - x, 2) +
            Mathf.Pow(OnPixel().y - y, 2)
        );

        if (distance <= radius)
        {
            if (IsPixel((int)x, (int)y))
            {
                WorkshopScript.SelectedLayer.SetPixel((int)x, (int)y, ColorsScript.GetColor());
            }
        }
    }
}
11
  • А это не будет лагать если например будет 10000 пикселей? Ну или у меня максимальный размер 1000 x 1000 выдержит ли это процессор?
    – eccs0103
    2 авг 2019 в 11:52
  • 10000 пикселей это квадрат 100x100, ваш метод для него сделает 100*360=36000 итераций, так что мой оптимальнее. Если надо еще быстрее, смотрите в сторону Quadtree. 2 авг 2019 в 11:56
  • Хорошо, спасибо. Я сейчас попробую ваш совет и если проблем не будет поставлю галочку.
    – eccs0103
    2 авг 2019 в 11:58
  • Проблемы могут быть на большой толщине карандаша, но там понадобится оптимизация уже другого рода 2 авг 2019 в 11:59
  • Идеально работает. Спасибо!
    – eccs0103
    2 авг 2019 в 12:20

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.