views:

136

answers:

3

Hi normally I would do this with a Background Worker, but I would like to do it with C# Task instead, just to understand Task better.

The thing is that I have a class with the following properties

    private int _number1;
    public int Number1
    {
        get { return _number1; }
        set { _number1 = value; OnPropertyChanged("Number1");}
    }

    private int _number2;
    public int Number2
    {
        get { return _number2; }
        set { _number2 = value; OnPropertyChanged("Number2");}
    }

Please note that I use the INotifyPropertyChanged.

Number1 = Task<int>.Factory.StartNew(() => GenerateResult()).Result;
Number2 = Task<int>.Factory.StartNew(() => GenerateResult2()).Result;

The GenerateResult and GenerateResult2 are just dumme methods, who sleeps and then return a number.

How would I make this work Async? Since right now, GenerateResult2() is first called when GenerateResult() is finished.

I need it to be able work Async, since I have no idea of when each task is going to finish or even if its going to finish.

A: 
Task<int>.Factory.StartNew(() => GenerateResult2()).ContinueWith(() => GenerateResult());
gandjustas
Unless I'm mistaken, this is exactly the same semantics as the OP's code - the second task isn't started until the first has completed.
Isak Savo
+3  A: 

When you get the Result property, you are effectively waiting for the task to complete. It will execute on a background thread but you are waiting in your main thread for the completion before you start the next thread.

See the MSDN doc for details.

You should be able to just assign your properties from the background thread:

Task<int>.Factory.StartNew(() => Number1 = GenerateResult());

WPF databinding will take care of marshalling the PropertyChanged event to the correct dispatcher thread.

Isak Savo
Thanks alot Isak Savo workes perfect and is simple to read :-) Alot easier then Backgroundworkers
gulbaek
A: 

I checked this: Task Parallelism (Task Parallel Library) and it it states that when using System.Threading.Tasks.Task<TResult> the Tasks run asynchronously and may complete in any order. If Result is accessed before the computation completes, the property will block the thread until the value is available.

I think that means if you are accessing .Result before it has a value, as you are doing in your sample code, you will have to wait for it to complete first.

It does make sense as the Result property would not be populated until the Task is completed.

François
Didn't see Isak Savo's answer which already points out that accessing Result will wait for task to complete.
François