在 HttpClient 异常上显示 AlertDialog (Xamarin Android)

Question

我的 Xamarin Android 应用使用 Web 服务，它使用 HttpClient 连接到该服务。在没有连接时（例如，当用户没有手机或 WiFi 连接时），会抛出异常。我正在使用 async/await 从服务器获取数据。这是我的代码的摘录：

    public async Task<String> doLogin(string username, string password)
    {
        String url = Constants.loginEndpoint + username + "/" + password + "/";
        var uri = new Uri(string.Format(url, string.Empty));

        return_string = "";

        try
        {
            var response = await GetAsync(uri);
            if (response.IsSuccessStatusCode)
            {
                return_string = "success";
                // Process the positive response here
            else
            { }
        }
        catch (Exception ex)
        {
            throw new ConnectionException();
        }

        return return_string;
    }

我定义了一个自定义 ConnectionException 并希望向用户显示一个 AlertDialog 以通知他们请求由于没有连接而失败。用户单击确定后，我想关闭该应用程序。我尝试通过以下方式显示警报对话框，但它不起作用：

    public class ConnectionException : Exception
{
    public ConnectionException()
    {
        AlertDialog.Builder alert = new AlertDialog.Builder(myApp.Context);
        alert.SetTitle("Failure");
        alert.SetMessage("Request failed. No connection.");
        alert.SetPositiveButton("OK", (senderAlert, args) =>
        {
        });

        Dialog dialog = alert.Create();
        dialog.Show();
    }

    public ConnectionException(string message)
        : base(message)
    { }

    public ConnectionException(string message, Exception innerException)
        : base(message, innerException)
    { }
}

这是正确的方法吗？可能不会，因为它不起作用。对于如何实现这一目标，我将不胜感激。另外，我没有考虑太多，但这是处理此类异常的首选方式吗？

Answer 1

假设您的myApp.Context 是Activity 并且它没有返回堆栈，您可以调用Finish()

var context = myApp.Context; // this needs to be an Activity-based context...
context.RunOnUiThread(() =>
{
    var alertDialog = new AlertDialog.Builder(context)
        .SetTitle("Failure")
        .SetMessage("Request failed. No connection.")
        .SetPositiveButton("OK", (senderAlert, args) =>
        {
            context.Finish();
        })
        .Create();
    alertDialog.Show();
});

Answer 2

您是在多个地方重复使用此异常，还是仅此一次？ 如果您只使用一次此异常，则没有真正的理由构建您自己的异常。 您也可以只捕获异常并从您的捕获中发布您的警报。

我知道这不是编写 catch 的好方法，但如果它有效，为什么不使用它。

旁注： DisplayAlert 对您来说也可能更容易。这将是一个班轮。

例子：

await DisplayAlert("Failure","Request failed. No connection.", "Ok");

Answer 3

您处理可能出现的错误的方式包含多个问题，并且由于多种原因不是正确的方法。

首先：您的代码不遵循 C-Sharp 约定并且包含一些代码异味。我向你展示了一种更好、更被接受的风格。

1) C# 中的方法通常以大写字母开头。 doLogin 变为 Login

2) 要创建一个新的 Uri 实例，您不需要格式化您的 url 字符串。不会使用 string.Empty。所以代码可以简化成await GetAsync(new Uri(...));

3) return_string 似乎没有在方法之外以任何方式使用。它是 string.Empty 或“成功”。为什么不将其切换为布尔值？这样您就可以轻松检查登录是否成功。返回类型变为 bool 而不是 string。

该方法现在看起来像这样：

public async Task<bool> Login(string username, string password)
{
    //TODO: Do parameter check for username and password

    try
    {
        var response = await GetAsync(new Uri(Constants.loginEndpoint + username + "/" + password + "/"));

        if (response.IsSuccessStatusCode)
        {
            // Process the positive response here

            return true;
        }
        else
        {
            return false;
        }
    }
    catch (Exception ex)
    {
        throw new ConnectionException();
    }

    return false;
}

其次，正如@Jason 所提到的，异常不应包含任何 UI 或业务逻辑。请考虑以下内容，这将破坏您当前的实现。

public async Task<bool> Login(string username, string password)
{
   var connectionEx = new ConnectionException();

    try
    {
       ...
    }
    catch (Exception ex)
    {
        throw connectionEx;
    }

    ...
}

现在您的用户会看到异常，即使没有异常。

最后一件事是，我建议不要仅仅为了抛出自定义异常而捕获异常。原因是，可能还有其他事情也会引发异常。例如，肯定响应处理中的某些内容为空。

根据 Login 方法的使用方式，例如直接在 Android Activity 中，我会这样做：

public async Task Login(string username, string password)
{
    //TODO: Do parameter check for username and password

    try
    {
        var response = await GetAsync(new Uri(Constants.loginEndpoint + username + "/" + password + "/"));

        if (response.IsSuccessStatusCode)
        {
            // Process the positive response here
        }
        else
        {
            var alertDialog = new AlertDialog.Builder(context)
               .SetTitle("Failure")
               .SetMessage("Request failed.")
               .SetPositiveButton("OK", (senderAlert, args) =>
                   {
                       Finish();
                   })
               .Create();

            alertDialog.Show();
        }
    }
    catch (Exception ex)
    {
        var alertDialog = new AlertDialog.Builder(context)
            .SetTitle("Failure")
            .SetMessage("Something went wrong (" + ex.Message +")")
            .SetPositiveButton("OK", (senderAlert, args) =>
                {
                   Finish();
                })
            .Create();

        alertDialog.Show();
    }
}