Упрощение кода с использованием типа Func вместо Action в Silverlight

Question

возможно ли рефакторинг этого кода с использованием типа Func вместо Action? я хотел бы избавиться от дополнительного класса (CalcParams), используемого для возврата значения с помощью обратного вызова Action.

Edit: * Примечание: метод загрузки или Func должен быть общедоступным;

вся ваша помощь будет очень признательна.

public class CalcParams
{
    public int CallID;
    public int Result;
    public Action<int, int> CallbackDone;
    public CalcParams(int callid, Action<int, int> callback)
    {
        CallID = callid;
        CallbackDone = callback;
    }
}
public partial class MainPage : UserControl
{
    int rand;
    public MainPage()
    {
        InitializeComponent();
        rand = 0;
    }
    public void DownloadDataInBackground(CalcParams calcparams)
    {
        WebClient client = new WebClient();
        Uri uri = new Uri(string.Format("https://www.google.com/search?q={0}", calcparams.CallID));
        client.DownloadStringCompleted += (s, e) =>
        {
            CalcParams localparams = (CalcParams)e.UserState;
            localparams.CallbackDone(localparams.CallID, e.Result.Length);
        };
        client.DownloadStringAsync(uri, calcparams);
    }

    private void button1_Click(object sender, RoutedEventArgs e)
    {
        int callid = rand++;
        Debug.WriteLine("Executing CallID #{0}", callid);
        DownloadDataInBackground(new CalcParams(callid, (c, r) =>
        {
            Debug.WriteLine("The result for the callid {0} is {1}", c, r);
        }));
        callid = 0;
    }
}

Answer 1

Исходя из ваших комментариев, я бы посоветовал вам использовать TPL, а не ваши собственные обратные вызовы, это значительно упрощает жизнь :) (сейчас у вас нет обработки исключений). В проекте ParallelExtensionsExtra, а также в .NET 4.5 вы найдете еще один веб-клиент с именем DownloadStringTask, который возвращает задачу. Предполагая, что вы не используете это, вы можете легко скопировать его. Проверьте следующий код:

    public Task<string> DownloadDataInBackground(int id)
    {
        TaskCompletionSource<string> resultTaskCompletionSource = new TaskCompletionSource<string>(id);

        WebClient client = new WebClient();

        Uri uri = new Uri(string.Format("https://www.google.com/search?q={0}", id));
        client.DownloadStringCompleted += (s, e) =>
        {
            if (e.Error != null)
                resultTaskCompletionSource.SetException(e.Error);
            else if (e.Cancelled)
                resultTaskCompletionSource.SetCanceled();
            else
                resultTaskCompletionSource.SetResult(e.Result);

        };
        client.DownloadStringAsync(uri);

        return resultTaskCompletionSource.Task;
    }


    private void button1_Click(object sender, EventArgs e)
    {
        int callid = rand++;
        Debug.WriteLine("Executing CallID #{0}", callid);

        DownloadDataInBackground(callid).ContinueWith(task =>
        {
            if (task.IsCompleted)
            {
                Debug.WriteLine("Download for {0} completed with a length of {1}", task.AsyncState, task.Result.Length);
            }
        });
    }

Answer 2

С добавлением захватов можно довольно элегантно избежать добавления дополнительного класса и / или использования userState.

public partial class MainPage : UserControl
{
    Random rand;

    public MainPage()
    {
        InitializeComponent();
        rand = new Random();
    }

    public void DownloadDataInBackground(int callId, Action<int> returnResult)
    {
        WebClient client = new WebClient();
        Uri uri = new Uri("https://www.google.com/search?q=" + callId.ToString());
        client.DownloadStringCompleted += (s, e) =>
        {
            // Do something else with result ?
            returnResult(e.Result.Length);
        };
        client.DownloadStringAsync(uri);
    }

    private void button1_Click(object sender, RoutedEventArgs e)
    {
        int callid=rand.Next();
        Debug.WriteLine("Executing CallID #{0}", callid);
        DownloadDataInBackground(callid, r =>
        {
            Debug.WriteLine("The result for the callid {0} is {1}", callid, r);
        }));
    }
}

Обратите внимание, что теперь требуется только Action<int>, callid в событии click "захватывается" для использования в обратном вызове returnResult.

Answer 3

Нет, не будет. Обратный вызов - это просто метод, имеющий некоторые параметры. Какова функция возвращаемого значения в обратном вызове (Func - это просто действие с возвращаемым значением). В типичном случае вам не стоит работать с вашим классом params и просто передавать необходимые аргументы напрямую.

DownloadDataInBackground(int callId, Action<int, int> callback);

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

Кроме того, вы можете просто захватить calcparams в DownloadDataInBackground, а не использовать для этого пользовательское состояние, например:

Answer 4

это мой первый подход, я хотел бы узнать другой, более простой или чистый способ сделать это.

public partial class MainPage : UserControl
{
    int rand;
    public MainPage()
    {
        InitializeComponent();
        rand = 0;
    }

    public Func<Action<int, int>, Action<int>> DownloadDataInBackground = (callback) =>
    {
        return (c) =>
        {
            WebClient client = new WebClient();
            Uri uri = new Uri(string.Format("https://www.google.com/search?q={0}", c));
            client.DownloadStringCompleted += (s, e2) =>
            {
                callback(c, e2.Result.Length);
            };
            client.DownloadStringAsync(uri);
        };
    };

    private void button1_Click(object sender, RoutedEventArgs e)
    {
        int callid = rand++;
        Debug.WriteLine("Executing CallID #{0}", callid);
        DownloadDataInBackground((c3, r3) =>Debug.WriteLine("The result for the callid {0} is {1}", c3, r3))(callid);
    }
}