проблемы производительности параллельной работы c #

Question

Этот код предназначен для преобразования из цветного в черно-белый.

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

но производительность вообще не улучшилась.

Я попытался получить изображение 800 * 600, изменив значение делителя (его здесь 800 в коде) в 2 для циклов и приращение значения счетчика.

Это изменение должно увеличить или уменьшить количество потоков

результаты:

value |  num of thread |   time
 800       1              1.17 sec
 400       2              1.17 sec
 200       4              1.17 sec
and so on ...

Помогите мне увеличить производительность, потому что для больших изображений требуется 8 секунд, и я хочу сделать это параллельным

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
using System.Diagnostics;
using System.Threading;

namespace IMG
{

public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();
    }
    string path = "";
    public void openimage()
    {
        if (openFileDialog1.ShowDialog() == DialogResult.OK)
        {
            path = openFileDialog1.FileName;
        }
    }
    Bitmap bm;
    Bitmap gs;
    private void button1_Click(object sender, EventArgs e)
    {
        if (path == "")
        {
            openimage();
        }
        Graphics g = this.CreateGraphics();
        bm = new Bitmap(path);
        // Draw image with no effects.....
        g.DrawImage(bm, 100, 100, 200, 200);
        gs = new Bitmap(bm.Width, bm.Height);
        worker soso = new worker(gs, bm);
        worker.counter = 0;
        soso.tograyscale();
        // Draw gray image.......
        g.DrawImage(gs, 305, 100, 200, 200);
        DateTime x = DateTime.Now;
        for (int koko = 0; koko < gs.Width/400; koko++)
        {        
                Thread t1 = new Thread(new ThreadStart(soso.trasform));
                t1.Start();
                t1.Join();
                soso.increment();
        }
        g.DrawImage(bm, 510, 100, 200, 200);
        gs.Dispose();
        g.Dispose();
        textBox19.Text=(DateTime.Now - x).ToString();
    }
}
public class worker
{
    public static int counter = 0;
    public Bitmap gs;
    public Bitmap bm;
    public int xxxx;
    public worker(Bitmap a, Bitmap b)
    {
        gs = a;
        bm = b;
        xxxx = a.Height;
    }
    public void increment()
    {
        counter += 400;
    }
    public void trasform()
    {
        argtransform(counter);

    }
    public void tograyscale()
    {
        for (int i = 0; i < bm.Width; i++)
        {
            for (int j = 0; j < bm.Height; j++)
            {
                Color c = bm.GetPixel(i, j);
                int y = (int)(0.3 * c.R + 0.59 * c.G + 0.11 * c.B);
                gs.SetPixel(i, j, Color.FromArgb(y, y, y));
            }
        }
    }
    public void argtransform(int cc)
    {
        for (int i = cc; i < cc + 400; i++)
        {
            for (int j = 0; j < xxxx; j++)
            {
                Color c = gs.GetPixel(i, j);
                int y1 = 0;
                if (c.R >= 128)
                    y1 = 255;
                bm.SetPixel(i, j, Color.FromArgb(y1, y1, y1));
            }
        }
    }
  }
}

спасибо, пожалуйста, как только сможете

---

спасибо всем вопрос к Крейгу Стунцу: как я могу отредактировать цвет пикселя, не вызывая setpixel? спасибо за вашу помощь

Answer 1

Не распараллеливайте, прежде чем оптимизировать. Запустите ваш код под хорошим профилировщиком и сначала исправьте проблемы с производительностью. Да, это может оказаться хорошим кандидатом на распараллеливание в конце, но я держу пари, что 1 миллион обращений к SetPixel - гораздо более серьезная проблема.

Answer 2

Позвонив по номеру Thread.Start, а затем немедленно позвонив по номеру Thread.Join, вы фактически работаете в последовательном, а не параллельном режиме. Поместите второй цикл после цикла Thread.Start, который вызывает Thread.Join.

List<Thread> threads = new List<Thread>();
for (int koko = 0; koko < gs.Width/400; koko++)
{        
    Thread t1 = new Thread(new ThreadStart(soso.trasform));
    t1.Start();
    threads.Add(t1);
    soso.increment();
}

foreach (Thread thread in threads) thread.Join();

Answer 3

GetPixel () / SetPixel () способ замедлить это.Вместо этого вы должны использовать BitmapData с «небезопасными» и указателями для редактирования изображения.

Обновление: И быстрый поиск в Google по BitmapData и оттенкам серого оказался в списке этого примера .

Answer 4

Прежде чем даже думать о распараллеливании, Вы должны профилировать свой код, чтобы узнать, где время фактически тратится.

Профилировщик Eqatec - это бесплатный и простой в использовании инструмент профилирования .NET.