Авторизация в отдельном потоке

@Zwelenewskiy · Регистрация: 09.01.2017

Author24 — интернет-сервис помощи студентам

Доброго времени суток. Авторизируюсь Вк через VKNET:

Кликните здесь для просмотра всего текста

C#

CancellationTokenSource mToken = new CancellationTokenSource();
 
var Authorization = Task.Factory.StartNew(o => Authorize((CancellationToken)o), mToken.Token, 
    TaskCreationOptions.LongRunning);
 
try
{
    mToken.Cancel();
    Task.WaitAll(Authorization);
}
catch
{
 
}
 
.......
 
private void Authorize(CancellationToken token)
{
    while (!token.IsCancellationRequested)
    {
        token.WaitHandle.WaitOne(1000);
 
        Program.Account.Authorize(new VkNet.Model.ApiAuthParams
        {
            ApplicationId = Application_id,
            Login = TB_Login.Text,
            Password = TB_Password.Text,
            Settings = VkNet.Enums.Filters.Settings.All
        });
    }            
}

В итоге, поток не запускается вообще. Почему?

@Usaga · 27.09.2018, 04:31

Zwelenewskiy, вы запускаете задачу и передаёте ей токен, который тут же отменяете. Гораздо эффективнее было бы вообще не запускать задачу, было бы ровно тоже самое, но код был бы проще.

@Zwelenewskiy · 28.09.2018, 01:10 **[ТС]**

Usaga, использую отдельный поток из-за того,что при авторизации GUI виснет. А это не есть хорошо)
Добавлено через 37 минут

Кликните здесь для просмотра всего текста

C#

Program.Account = new VkApi();
 
    try
    {
        CancellationTokenSource mToken = new CancellationTokenSource();
 
        var Authorization = Task.Factory.StartNew(o => Authorize(TB_Login.Text, TB_Password.Text), mToken.Token, 
            TaskCreationOptions.LongRunning); 
 
        Task.WaitAll(Authorization);
 
        if (Program.Account.Token != null)
        {
            Hide();
            new Form1().ShowDialog();
        }
    }
    catch
    {
        MessageBox.Show("Данные введены неверно");
        return;
    }
 
.......
 
private void Authorize(CancellationToken token)
{
    private void Authorize(string Login, string Password)
    {
        Program.Account.Authorize(new VkNet.Model.ApiAuthParams
        {
            ApplicationId = Application_id,
            Login = Login,                
            Password = Password,
            Settings = VkNet.Enums.Filters.Settings.All
        });
    }     
}

Собственно, рабочий вариант. Убрал управляющий токен

Добавлено через 3 минуты
Теперь GUI "не виснет", спокойно ждет завершения потока авторизации. Но тем не менее, пока он ждет, GUI, все-таки, не отвечает на другие действия, т.е., все-таки, виснет... Можно ли от этого избавиться?

@Usaga · 28.09.2018, 04:19

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

Сообщение от Zwelenewskiy

Можно ли от этого избавиться?

Не ждать завершения потока?

@kenny69 · 29.09.2018, 07:55

Используйте метод AuthorizeAsync
А по сути это I/O-bound операция, так что отдельный поток здесь не нужен.

Добавлено через 3 минуты
Я глянул исходники VkNet, у них кстати асинхронность через выделение потока, т.е. через жопу

C#

public static Task<T> TryInvokeMethodAsync<T>(Func<T> func)
        {
            var tcs = new TaskCompletionSource<T>();
 
            Task.Factory.StartNew(() =>
                {
                    try
                    {
                        var result = func.Invoke();
                        tcs.SetResult(result);
                    }
                    catch (VkApiException ex)
                    {
                        tcs.SetException(ex);
                    }
                })
                .ConfigureAwait(false);
 
            return tcs.Task;
        }

C#

    public Task AuthorizeAsync(IApiAuthParams @params)
        {
            return TypeHelper.TryInvokeMethodAsync(func: () => Authorize(@params: @params));
        }

По хорошему надо было сделать через async методы HttpClient, как и сделано в https://github.com/worldbeater/VkLibrary , которую я и использую.

@Zwelenewskiy · 07.10.2018, 01:22 **[ТС]**

Сообщение от Usaga

Не ждать завершения потока?

Да, была сделана большая глупость)

@Zwelenewskiy · 07.11.2018, 02:00 **[ТС]**

Сообщение от kenny69

т.е. через жопу

Как тогда сделать не через жопу?)

@Usaga 12089 / 8367 / 1280 Регистрация: 21.01.2016 Сообщений: 31,543
	27.09.2018, 04:31	2
	Zwelenewskiy, вы запускаете задачу и передаёте ей токен, который тут же отменяете. Гораздо эффективнее было бы вообще не запускать задачу, было бы ровно тоже самое, но код был бы проще. 1

@Usaga 12089 / 8367 / 1280 Регистрация: 21.01.2016 Сообщений: 31,543
	28.09.2018, 04:19	4
	Вы запустили отдельный поток, а в основном стали ожидать его завершения) Это ровно тоже самое, что сделать всё в одном потоке, только кода больше. Сообщение от Zwelenewskiy Можно ли от этого избавиться? Не ждать завершения потока? 0

@Zwelenewskiy 129 / 127 / 107 Регистрация: 09.01.2017 Сообщений: 1,651
	07.10.2018, 01:22 [ТС]	6
	Сообщение от Usaga Не ждать завершения потока? Да, была сделана большая глупость) 0

@Zwelenewskiy 129 / 127 / 107 Регистрация: 09.01.2017 Сообщений: 1,651
	07.11.2018, 02:00 [ТС]	7
	Сообщение от kenny69 т.е. через жопу Как тогда сделать не через жопу?) 0

Опции темы