Что не так с синусом?

Question

[Popou]

Я хотел сделать подобный эффект волны

Для этого я использовал

следующий алгоритм:

for (int y = 0; y < img.Height; ++y)
{
    Span<Rgba32> pixelRowSpan = img.GetPixelRowSpan(y);
    for (int x = 0; x < img.Width; ++x)
    {

        int y1 = Convert.ToInt32(y + 20.0 * Math.Sin(x / 32.0));

        if (y1 >= img.Height)
            y1 = img.Height - 1;
        if (y1 < 0)
            y1 = 0;

        var sourcePixel = img.GetPixelRowSpan(y1)[x];

        pixelRowSpan[x] = sourcePixel;
    }
}

Но вместо желаемого результата, у меня выходит

это

Исходное изображение

Спасибо Wataru!

Исходное изображение

Результат

Comments

[Илья]

Не только при побеге, при наступлении на врага тоже хорошо ульта помогает

[Василий Банников]

Почини картинку с результатом. Не видно

[Popou]

Василий Банников, видно все, в этом и беда, что появляется такие линии, но ответ уже нашёл

[QWERTYUIOPas]

Конечный скрин с блэкберри-девайса сделан? соотнош. сторон необычное для андроида

Answer 1

[Wataru]

Вы меняете изображение на месте, проходя по нему сверху вниз. Когда ваш синус сдвигает пиксели вниз - вы перетираете все пиксели в столбце самым верхним.

Надо или копировать с изменениями в новое изображение, или менять направление прохода в зависимости от знака синуса.

Comments

[Popou]

Блин точно, спасибо

[QWERTYUIOPas]

Либо просто обрезать результат

[Wataru]

QWERTYUIOPas, чем обрезанный результат поможет от того, что в этом результате половина столбцов неправильные?

[QWERTYUIOPas]

Там, где синус отрицателен, перевернуть картинку, исказить по модулю синуса, а потом опять перевернуть. А обрезать, чтобы край был ровный

[Wataru]

QWERTYUIOPas, Проще уж циклы развернуть. Обрезка к вопросу не относится - это отдельная проблема: По краям появляются пустые области.