WPF

Ranger49 wrote:

tried my code in a sample program that only had an i-loop that did nothing and there my progressBar worked fine...

BackgroundWorker bw = new BackgroundWorker();

public Window1()
{
    InitializeComponent();

    bw.WorkerReportsProgress = true;
    bw.DoWork += new DoWorkEventHandler(bw_DoWork);
    bw.ProgressChanged += new ProgressChangedEventHandler(bw_ProgressChanged);
}

private void button1_Click(object sender, RoutedEventArgs e)
{
    if (!bw.IsBusy)
        bw.RunWorkerAsync();
}

void bw_DoWork(object sender, DoWorkEventArgs e)
{
    for (int i = 0; i < 100; i++)
    {
        bw.ReportProgress(i);

        Thread.Sleep(50);
    }
}

void bw_ProgressChanged(object sender, ProgressChangedEventArgs e)
{
    Random rnd = new Random();
    Pen p = new Pen();
    byte red, green, blue;

    SolidColorBrush brush = new SolidColorBrush();

    Line line = new Line();
    red = (byte)rnd.Next(0, 255);
    green = (byte)rnd.Next(0, 255);
    blue = (byte)rnd.Next(0, 255);
    Color col = new Color();
    col.R = red;
    col.G = green;
    col.B = blue;
    col.A = 255;
    brush.Color = col;

    line.X2 = rnd.Next(0, (int)canvas1.ActualWidth);
    line.Y2 = rnd.Next(0, (int)canvas1.ActualHeight);
    line.X1 = rnd.Next(0, (int)canvas1.ActualWidth);
    line.Y1 = rnd.Next(0, (int)canvas1.ActualHeight);
    line.Stroke = brush;
    line.StrokeThickness = 1.0;

    canvas1.Children.Add(line);

    progressbar1.Value = e.ProgressPercentage;
}

*edit*  Funny side note:  Taking the Sleep() call out of my example makes it<br />
run too fast to get a newly seeded Random object so only a few lines appear<br />
(because there's so many duplicate lines :)).  I suppose without the Sleep() call,<br />
the Random object should be created only once outside the loop.  That change could <br />
look something like this:

void bw_DoWork(object sender, DoWorkEventArgs e)
 {
     Random rnd = new Random();

     for (int i = 0; i < 100; i++)
     {
         bw.ReportProgress(i, rnd);

         //Thread.Sleep(25);
     }
 }

 void bw_ProgressChanged(object sender, ProgressChangedEventArgs e)
 {
     Random rnd = e.UserState as Random;
  ...

Mark Salsbery
Microsoft MVP - Visual C++

public void bw_DoWork(object sender, DoWorkEventArgs e)
 {
     Random rnd = new Random();
     Pen p = new Pen();
     byte red, green, blue;
     Line line = new Line();

     for (int i = 0; i < 100; i++)
     {
         SolidColorBrush brush = new SolidColorBrush();

         red = (byte)rnd.Next(0, 255);
         green = (byte)rnd.Next(0, 255);
         blue = (byte)rnd.Next(0, 255);
         Color col = new Color();
         col.R = red;
         col.G = green;
         col.B = blue;
         col.A = 255;
         brush.Color = col;

         line.X2 = rnd.Next(0, (int)canvas1.ActualWidth);
         line.Y2 = rnd.Next(0, (int)canvas1.ActualHeight);
         line.X1 = rnd.Next(0, (int)canvas1.ActualWidth);
         line.Y1 = rnd.Next(0, (int)canvas1.ActualHeight);
         line.Stroke = brush;
         line.StrokeThickness = 1.0;
         canvas1.Children.Add(line);

         bw.ReportProgress(i);
     }
 }

 public void bw_ProgressChanged(object sender, ProgressChangedEventArgs e)
 {
     progressbar1.Value = e.ProgressPercentage;
 }

Ranger49 wrote:

should the code to compute the fractal be put in the DoWork method, where the progressbar is in the ProgressChanged method?

Mark Salsbery
Microsoft MVP - Visual C++

public partial class Window1 : Window
{
    BackgroundWorker bw = new BackgroundWorker();

    public Window1()
    {
        InitializeComponent();

        bw.WorkerReportsProgress = true;
        bw.DoWork += new DoWorkEventHandler(bw_DoWork);
        bw.ProgressChanged += new ProgressChangedEventHandler(bw_ProgressChanged);
    }

    private void button1_Click(object sender, RoutedEventArgs e)
    {
        if (!bw.IsBusy)
            bw.RunWorkerAsync();
    }

    void bw_DoWork(object sender, DoWorkEventArgs e)
    {
        <code>// Background thread - no UI element access allowed here!</code>

        LineDescriptor ld = new LineDescriptor();

        for (int i = 0; i < 100; i++)
        {
            Random rnd = new Random();
            //Pen p = new Pen();
            byte red, green, blue;

            red = (byte)rnd.Next(0, 255);
            green = (byte)rnd.Next(0, 255);
            blue = (byte)rnd.Next(0, 255);
            Color col = new Color();
            col.R = red;
            col.G = green;
            col.B = blue;
            col.A = 255;
            ld.Color = col;

            ld.X2 = rnd.Next(0, (int)canvas1.ActualWidth);
            ld.Y2 = rnd.Next(0, (int)canvas1.ActualHeight);
            ld.X1 = rnd.Next(0, (int)canvas1.ActualWidth);
            ld.Y1 = rnd.Next(0, (int)canvas1.ActualHeight);

            bw.ReportProgress(i + 1, ld); <code>//<-- bug fix - added 1 to i</code> :)

            Thread.Sleep(25);
        }
    }

    void bw_ProgressChanged(object sender, ProgressChangedEventArgs e)
    {
        <code>// UI thread - UI element access allowed here!</code>

        LineDescriptor ld = e.UserState as LineDescriptor;

        Line line = new Line();
        line.X1 = ld.X1;
        line.Y1 = ld.Y1;
        line.X2 = ld.X2;
        line.Y2 = ld.Y2;
        line.Stroke = new SolidColorBrush(ld.Color);
        line.StrokeThickness = 1.0;

        canvas1.Children.Add(line);

        progressbar1.Value = e.ProgressPercentage;
    }
}


class LineDescriptor
{
    public Color Color { get; set; }
    public double X1 { get; set; }
    public double Y1 { get; set; }
    public double X2 { get; set; }
    public double Y2 { get; set; }
}

Mark Salsbery
Microsoft MVP - Visual C++

DahrkDaiz wrote:

Any advice?

ravimori wrote:

for anything you do using keyboard or mouse input at that time make the value of 'xyz' to 0..