Гость
Форумы / WPF, Silverlight [игнор отключен] [закрыт для гостей] / [SL] Одна переменная для нескольких обработчиков / 25 сообщений из 40, страница 1 из 2
21.05.2012, 11:56
    #37803556
enigmatic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
Могут ли возникнуть ошибки одновременного доступа к объекту в следующем коде?
Код выполняется в Silverlight.

Есть некий сервис, запрашивающий данные:
Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
public static class DataService{
	public static void GetSomeData(long id, Action<SomeData> callback){
		var client = new WebClient();
		client.DownloadStringAsync(uri);
		client.DownloadStringCompleted+=(s,ea)=>{
			if (ea.error != null)
				callback(ea.Result)
			else
				MessageBox.show(ea.error);
		}
	}
}

И есть код:
Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
pulbic void Test(){
	var asyncCounter = 0;
	foreach (var id in idList){
		asyncCounter++;
		DataService.GetSomeData(id, data=>{
			asyncCounter--;
			if (asyncCounter == 0){
				DoSmthElse(...);
			}
		});
	}
}

Здесь происходит несколько одновременных запросов, по завершении которых происходит вызов DoSmthElse();
Достигается это за счет использования переменной asyncCounter.

Будут ли проблемы при использовании этой одной переменной в обработчиках разных запросов в Silverlight?
...
Рейтинг: 0 / 0
21.05.2012, 12:12
    #37803589
AlexeiK
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
enigmatic,

предлагаешь всем подебажить твои отрывки кода в своих головах? тогда тебе в тему "Экстрасенсы говорят".
...
Рейтинг: 0 / 0
21.05.2012, 12:38
    #37803638
Алексей К
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
enigmaticБудут ли проблемы при использовании этой одной переменной в обработчиках разных запросов в Silverlight?Да.

зы: Если DownloadStringCompleted выполняется в гуйном потоке - тогда нет. Надо смотреть мануал по этому событию.
...
Рейтинг: 0 / 0
21.05.2012, 12:41
    #37803642
Алексей К
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
зы: Зря ты не послушал совета, и не стал использовать Task-и. Вместо этой байды применил бы Task.ContinueWhenAll . Дело твоё конечно, просто мысль...
...
Рейтинг: 0 / 0
21.05.2012, 14:30
    #37803920
enigmatic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
Алексей КenigmaticБудут ли проблемы при использовании этой одной переменной в обработчиках разных запросов в Silverlight?Да.

зы: Если DownloadStringCompleted выполняется в гуйном потоке - тогда нет. Надо смотреть мануал по этому событию.
Хорошо. Значит, можно (пока) не беспокоиться.
http://msdn.microsoft.com/en-us/library/cc197953(v=vs.95).aspx The WebClient callback, which is raised when the HTTP response is returned, is invoked on the User Interface (UI) thread, and can be used to update the properties of UI elements.
Алексей Кзы: Зря ты не послушал совета, и не стал использовать Task-и. Вместо этой байды применил бы Task.ContinueWhenAll . Дело твоё конечно, просто мысль...Пока просто не могу этого сделать, т.к. пока не умею с тасками и пока в приоритете работа вообще не связанная с этим топиком.
Целью этого вопроса было убедиться, что вышеописанное не таит в себе скрытых граблей, на которые можно нарваться по неопытности.
В перспективе, нужно будет реализовать то же самое на Task (или Rx).

AlexeiKпредлагаешь всем подебажить твои отрывки кода в своих головах? тогда тебе в тему "Экстрасенсы говорят".Если хочется подебажить, то можно сделать тестовый солюшен с сервисом и обращением. Сделать?
...
Рейтинг: 0 / 0
13.06.2012, 12:53
    #37836200
enigmatic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
Попробовал через Task.ContinueWhenAll. Все отработало замечательно.
Прилагаю тестовый проект под 5-й SL.

Как бы сделать подобное под четвертым, где нет Task'ов?
Обновиться до пятого возможности пока нет.

Попробовал Rx под четвертым. Не получилось. Думаю, Rx предназначен скорее для работы с массивами данных генерируемыми мышкой, нежели с "действиями". Возможно не прав.

P.S. В тестовом проекте есть интересный момент - обработчики WebClient.DownloadStringCompleted отрабатывают после параметра Action<Tasks[]> continuationAction метода Task.Factory.ContinueWhenAll, хотя сначала выполняется код методов Get. Поэтому сначала выводится сообщение "Complete", а затем пара "Got 1-й/2-й".
...
Рейтинг: 0 / 0
14.06.2012, 06:35
    #37837209
enigmatic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
enigmaticВ тестовом проекте есть интересный момент ...
Все правильно. Сначала выполняется код тасков, затем завершающий метод (continuationAction) и потом обработчики DownloadStringCompleted. Потому как continuationAction не будет ожидать обработчиков без соответствующей обработки.
...
Рейтинг: 0 / 0
14.06.2012, 07:25
    #37837223
Алексей К
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
enigmaticВсе правильно.Всё не правильно. :-)

Вместо
Код: c#
1.
2.
3.
4.
5.
6.
        private void Get(Uri uri)
        {
            var client = new System.Net.WebClient();
            client.DownloadStringAsync(uri);
            client.DownloadStringCompleted += (s, ea) => Function(uri.ToString());
        }


должно быть что-то вроде
Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
var t1 = Get(uri1);
var t2 = Get(uri2);

Task.Factory.ContinueWhenAll(new[] { t1, t2 }, tt => Dispatcher.BeginInvoke(() => MessageBox.Show("Complete")));

...

private Task Get(Uri uri)
{
    return Task.Factory.FromAsync(........);
}



зы: для синхронизации вместо явного вызова Dispatcher.BeginInvoke можно использовать контекст синхронизации.
...
Рейтинг: 0 / 0
14.06.2012, 08:17
    #37837256
enigmatic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
Алексей КВсё не правильно. :-)Тот пример был написан на скорую руку и необдуманно. :)

Попробовал пример обертки EAP (Event-based Async Pattern) с MSDN ( линк (раздел Exposing Complex EAP Operations As Tasks) и линк ). Под нормальным дотнетом работает, под SL - виснет. Видимо, под SL опять тонкости, связанные с UI-потоком.
Прилагаю тестовые проекты под раром, они оба используют один и тот же "AsyncService"-класс для обертки EAP в таск:
ConsoleApplication1 - нормальный дотнет,
testTplTask - сильверлайт.
Если пробовать дебажить, то лучше начать с SL-проекта, чтобы поднять локальный вебсервер, на который будет потом коннектиться нормальный дотнет.
Алексей КВместо
Код: c#
1.
2.
3.
4.
5.
6.
        private void Get(Uri uri)
        {
            var client = new System.Net.WebClient();
            client.DownloadStringAsync(uri);
            client.DownloadStringCompleted += (s, ea) => Function(uri.ToString());
        }

должно быть что-то вроде
Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
var t1 = Get(uri1);
var t2 = Get(uri2);

Task.Factory.ContinueWhenAll(new[] { t1, t2 }, tt => Dispatcher.BeginInvoke(() => MessageBox.Show("Complete")));

...

private Task Get(Uri uri)
{
    return Task.Factory.FromAsync(........);
}

Спасибо, попробую.Алексей Кзы: для синхронизации вместо явного вызова Dispatcher.BeginInvoke можно использовать контекст синхронизации.Тоже попробую.
...
Рейтинг: 0 / 0
14.06.2012, 09:42
    #37837361
enigmatic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
enigmatic,

авторПрилагаю тестовые проекты под раром,
...
Рейтинг: 0 / 0
14.06.2012, 13:06
    #37837750
enigmatic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
Алексей Кдолжно быть что-то вроде
Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
var t1 = Get(uri1);
var t2 = Get(uri2);

Task.Factory.ContinueWhenAll(new[] { t1, t2 }, tt => Dispatcher.BeginInvoke(() => MessageBox.Show("Complete")));

...

private Task Get(Uri uri)
{
    return Task.Factory.FromAsync(........);
}


Похоже, Task.Factory.FromAsync(...) применяется для оборачивания таких методов как HttpWebRequest.BeginResponse/EndResponse (которые Async Programming Model), но не WebClient.DownloadStringAsync/DownloadStringCompleted (которые Event-based Async Pattern). По крайней мере, применить FromAsync к DownloadStringCompleted не получилось.
Для обертки EAP, судя по всему, предназначен TaskCompletionSource.
Но его использование под SL, похоже, вешает UI поток. Причем, похоже, TaskCompletionSource вешает его только под SL, но не в обычном .NET.
Гуглинг по использованию FromAsync для WebClient.DownloadStringCompleted не помог найти ничего нужного, но вместо этого дал отсылку к TaskCompletionSource ( секция Converting an Event-Based Pattern ).
...
Рейтинг: 0 / 0
14.06.2012, 13:09
    #37837755
enigmatic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
Алексей К,

Не приходилось ли оборачивать WebClient с его DownloadStringAsync/DownloadStringCompleted в Task используя Task.Factory.FromAsync(...)?
...
Рейтинг: 0 / 0
14.06.2012, 13:29
    #37837786
Алексей К
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
enigmaticПохоже, Task.Factory.FromAsync(...) применяется для оборачивания таких методов как HttpWebRequest.BeginResponse/EndResponse (которые Async Programming Model), но не WebClient.DownloadStringAsync/DownloadStringCompleted (которые Event-based Async Pattern). По крайней мере, применить FromAsync к DownloadStringCompleted не получилось.Согласен. Может как-то и можно, но у меня тоже не получилось. :-)

С другой стороны через TaskCompletionSource всё просто. Не запускал, но вроде должно работать:
Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
    public static class WebClientHelper
    {
        public static Task<string> DownloadStringAsyncEx(this WebClient wc, Uri uri)
        {
            var tcs = new TaskCompletionSource<string>();
            wc.DownloadStringCompleted += DownloadStringCompleted;
            wc.DownloadStringAsync(uri, tcs);
            return tcs.Task;    
        }

        static void DownloadStringCompleted(object sender, DownloadStringCompletedEventArgs e)
        {
            var wc = (WebClient)sender;
            wc.DownloadStringCompleted -= DownloadStringCompleted;

            var tcs = (TaskCompletionSource<string>)e.UserState;

            if (e.Cancelled)
            {
                tcs.SetCanceled();
                return;
            }

            if (e.Error != null)
            {
                tcs.SetException(e.Error);
                return;
            }

            tcs.SetResult(e.Result);
        }
    }


В случае с Силверлайтом - не знаю, надо думать, у нас тут жара аж капец... :-)
...
Рейтинг: 0 / 0
14.06.2012, 13:46
    #37837832
Алексей К
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
enigmaticПочему бы не использовать HttpWebRequest.BeginGetResponse/EndGetResponse?
...
Рейтинг: 0 / 0
14.06.2012, 14:14
    #37837900
enigmatic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
Алексей КС другой стороны через TaskCompletionSource всё просто. Не запускал, но вроде должно работать:
Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
    public static class WebClientHelper
    {
        public static Task<string> DownloadStringAsyncEx(this WebClient wc, Uri uri)
        {
            var tcs = new TaskCompletionSource<string>();
            wc.DownloadStringCompleted += DownloadStringCompleted;
            wc.DownloadStringAsync(uri, tcs);
            return tcs.Task;    
        }

        static void DownloadStringCompleted(object sender, DownloadStringCompletedEventArgs e)
        {
            var wc = (WebClient)sender;
            wc.DownloadStringCompleted -= DownloadStringCompleted;

            var tcs = (TaskCompletionSource<string>)e.UserState;

            if (e.Cancelled)
            {
                tcs.SetCanceled();
                return;
            }

            if (e.Error != null)
            {
                tcs.SetException(e.Error);
                return;
            }

            tcs.SetResult(e.Result);
        }
    }


Хм ... :) Использовать TaskCompletionSource в методе возврата Task. Хорошая мысль. Даже как-то не подумалось о такой возможности.

Алексей КВ случае с Силверлайтом - не знаю, надо думать, у нас тут жара аж капец... :-)Попробовал, благополучно виснет, причем только под SL.
То ли TaskCompleteSource вешает UI, то ли я что-то совсем не так делаю :)
Мы тоже кондиционеры на полную выкручиваем.

Алексей КПочему бы не использовать HttpWebRequest.BeginGetResponse/EndGetResponse?Надо будет попробовать. Но подгонять задачу (расправиться с WebClient'ом) под средства (Task'и) не наш метод же!
...
Рейтинг: 0 / 0
14.06.2012, 14:18
    #37837909
enigmatic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
enigmaticПопробовал, благополучно виснет, причем только под SL...Тестовый код сейчас выглядит примерно так:
Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
public partial class MainPage
    {
        public MainPage()
        {
            InitializeComponent();

            var task = DownloadStringAsyncEx(new Uri("http://localhost:1732/data1.txt"));

            MessageBox.Show(string.Format("Got {0}", task.Result));
        }

        public Task<string> DownloadStringAsyncEx(Uri uri) {
            var tcs = new TaskCompletionSource<string>();
            var client = new WebClient();
            client.DownloadStringCompleted += (s, ea) => tcs.SetResult(ea.Result);
            client.DownloadStringAsync(uri, tcs);
            return tcs.Task;
        }
    }
...
Рейтинг: 0 / 0
14.06.2012, 15:01
    #37837985
Алексей К
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
enigmaticНо подгонять задачу (расправиться с WebClient'ом) под средства (Task'и) не наш метод же!Боюсь, ничего хорошего из этого не выйдет. Мне кажется, WebClient с его встроенной синхронизацией и Task-и вещи взаимоисключающие.
...
Рейтинг: 0 / 0
14.06.2012, 15:25
    #37838038
Алексей К
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
enigmaticТестовый код сейчас выглядит примерно так:
Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
    public partial class MainPage
    {
        public MainPage()
        {
            InitializeComponent();
            var task = DownloadStringAsyncEx(new Uri("http://localhost:1732/data1.txt"));
            MessageBox.Show(string.Format("Got {0}", task.Result));
        }


Так конечно, правильно что виснет. :-)
Надо как-то так:
Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
    public partial class MainPage
    {
        public MainPage()
        {
            InitializeComponent();
            
            DownloadStringAsyncEx(new Uri("http://localhost:1732/data1.txt"))
                .ContinueWith(t => Dispatcher.BeginInvoke(() =>
                       MessageBox.Show(string.Format("Got {0}", t.Result)
                )); // + обработку ошибок добавить
        }
...
Рейтинг: 0 / 0
14.06.2012, 15:35
    #37838068
enigmatic
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
Алексей К,

Но ведь консольное приложение отрабатывает нормально. Уверен, проблемы из-за отличия CLR SL от CLR обычного дотнета.
...
Рейтинг: 0 / 0
15.06.2012, 05:52
    #37838816
Алексей К
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
enigmaticНо ведь консольное приложение отрабатывает нормально. Уверен, проблемы из-за отличия CLR SL от CLR обычного дотнета.Нет. Отличие в том, что приложение консольное. А в силверлайтном приложении возникает мёртвая блокировка. Ожидание завершения при обращении к свойству task.Result в конструкторе в гуйном потоке + событие завершения идёт в гуйном потоке == мёртвая блокировка.
...
Рейтинг: 0 / 0
15.06.2012, 05:54
    #37838817
Алексей К
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
Все отличия из-за того, что в Силверлайте DownloadStringCompleted синхронизирован с гуйным потоком. Но эти отличия были бы видны при сравнении с например WPF-ным приложением.
...
Рейтинг: 0 / 0
16.06.2012, 11:06
    #37840697
SeVa
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
enigmaticМогут ли возникнуть ошибки одновременного доступа к объекту в следующем коде?
Код выполняется в Silverlight.

Есть некий сервис, запрашивающий данные:
Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
public static class DataService{
	public static void GetSomeData(long id, Action<SomeData> callback){
		var client = new WebClient();
		client.DownloadStringAsync(uri);
		client.DownloadStringCompleted+=(s,ea)=>{
			if (ea.error != null)
				callback(ea.Result)
			else
				MessageBox.show(ea.error);
		}
	}
}

И есть код:
Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
pulbic void Test(){
	var asyncCounter = 0;
	foreach (var id in idList){
		asyncCounter++;
		DataService.GetSomeData(id, data=>{
			asyncCounter--;
			if (asyncCounter == 0){
				DoSmthElse(...);
			}
		});
	}
}

Здесь происходит несколько одновременных запросов, по завершении которых происходит вызов DoSmthElse();
Достигается это за счет использования переменной asyncCounter.

Будут ли проблемы при использовании этой одной переменной в обработчиках разных запросов в Silverlight?

Могут и будут. Асинхронные вызовы могут выполняться в разной последовательности, вызовы нужно делать с дополнительным параметром state, который нужно анализировать в обработчике. Помимо этого есть утечки памяти. При множественных вызовах webrequest - более предпочтительных вариант, тк нет лишней диспетчеризации в ui поток.
...
Рейтинг: 0 / 0
16.06.2012, 11:10
    #37840698
SeVa
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
Алексей КВсе отличия из-за того, что в Силверлайте DownloadStringCompleted синхронизирован с гуйным потоком. Но эти отличия были бы видны при сравнении с например WPF-ным приложением.

Привильно здесь deadlock, а task'и - унылый гумос. rx в разы удобней и лучше.В особенности для клиентских приложений
...
Рейтинг: 0 / 0
16.06.2012, 12:07
    #37840750
Алексей К
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
SeVaа task'и - унылый гумос. rx в разы удобней и лучше.В особенности для клиентских приложенийДля построения цепочки асинхронных операций таски удобны. Всё просто и ничего лишнего.

Покажи как будет выглядеть аналогичный код на Rx.
Код: c#
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
Task<int> F1()
{ return ... }

Task<int> F2(int v1)
{ return ... }

Task<int> F3(int v2)
{ return ... }

...


Task<int> Execute()
{
    return F1()
        .ContinueWith(t => F2(t.Result))
        .Unwrap()
        .ContinueWith(t => F3(t.Result))
        .Unwrap();
}
...
Рейтинг: 0 / 0
16.06.2012, 12:44
    #37840798
SeVa
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
[SL] Одна переменная для нескольких обработчиков
Элементарно
Код: c#
1.
2.
3.
4.
5.
var res = from t1 in Observable.Return(Task1())
              from t2 in Observable.Return(Task2())
              select t1+t2;
or
    Observable.Return(Task1()).And(Observable.Return(Task2()).Then((t1,t2) = > t1+t2)      



Пример и вопрос примитивный, тк task'и только под это заточены, а у rx возможностей больше
...
Рейтинг: 0 / 0
Форумы / WPF, Silverlight [игнор отключен] [закрыт для гостей] / [SL] Одна переменная для нескольких обработчиков / 25 сообщений из 40, страница 1 из 2
Целевая тема:
Создать новую тему:
Автор:
Найденые пользователи ...
Разблокировать пользователей ...
Читали форум (0):
Пользователи онлайн (0):
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]