C#

Mark Salsbery wrote:

and AsyncCompletedEventArgs.Error == null there?

Mark Salsbery wrote:

Is it a debugger issue? I can't reproduce it here.

John Simmons / outlaw programmer wrote:

If I try to run it under the debugger, all hell breaks loose.

		// in some class

		private System.ComponentModel.BackgroundWorker backgroundWorker1;

		private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
		{
			for (int i = 0; i < 5; i++)
			{
				Thread.Sleep(1000);
				this.backgroundWorker1.ReportProgress(i * 20);
			}
			e.Result = string.Format("****       Result String!");
			this.backgroundWorker1.ReportProgress(100);
			System.Diagnostics.Trace.WriteLine("****   End of DoWork");
		}

		private void backgroundWorker1_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
		{
			System.Diagnostics.Trace.WriteLine("****   RunWorkerCompleted");

			if (e.Error != null)
			{
				System.Diagnostics.Trace.WriteLine(e.Error.Message);
			}
			else if (e.Cancelled)
			{
				System.Diagnostics.Trace.WriteLine("Canceled");
			}
			else
			{
				System.Diagnostics.Trace.WriteLine(e.Result.ToString());
			}
		}

		private void backgroundWorker1_ProgressChanged(object sender, ProgressChangedEventArgs e)
		{
			System.Diagnostics.Trace.WriteLine(string.Format("****       ProgressChanged {0}%", e.ProgressPercentage));
		}

...

this.backgroundWorker1 = new System.ComponentModel.BackgroundWorker();
this.backgroundWorker1.WorkerReportsProgress = true;
this.backgroundWorker1.WorkerSupportsCancellation = false;
this.backgroundWorker1.DoWork += new DoWorkEventHandler(backgroundWorker1_DoWork);
this.backgroundWorker1.RunWorkerCompleted += new RunWorkerCompletedEventHandler(backgroundWorker1_RunWorkerCompleted);
this.backgroundWorker1.ProgressChanged += new ProgressChangedEventHandler(backgroundWorker1_ProgressChanged);
this.backgroundWorker1.RunWorkerAsync();

Mark Salsbery
Microsoft MVP - Visual C++

Creamboy wrote:

then, use some randomizing technique

private string[] PickFrom(string[] values, int count) {
	Random r = new Random();
	string[] result = new string[count];
	for (int i = 0, index = 0; index < count; i++) {
		if (r.Next(values.Length - i) < count - index) {
			result[index++] = values[i];
		}
	}
	return result;
}

Luc Pattyn wrote:

Shouldn't all candidate values get the same probability?
IMO the test should read if ( r.Next(values.Length) < count )
so each value[i] has probability count/value.Length

Luc Pattyn wrote:

So you are cheating a bit
to make it work all the time, at the expense of the first elements in the array,
since your probabilities are increasing at first.