Xamarin Formen, Fehler, wenn ein webhttp-Link

Question

in diesem Beispiel zu öffnen versuchen, öffnen Sie eine Adresse http wie folgt aussehen: http://www.url.com/ und ich empfangen eine Fehlermeldung:

Invalid URI: The format of the URI could not be determined.

Dies ist der Code, den ich verwende. Und was, wenn ein Benutzer eine schlechte Adresse aufsetzt, wie zum Beispiel htp:/ohne zwei t's, wie kann ich einen Absturz vermeiden?

Dies ist mein Code:

foreach (var currentItem in getInfofromDB["results"]) { 

    URLstring = currentItem ["url"].ToString(); 


URLButton.Clicked += (object sender, EventArgs e) => { 

    Device.OpenUri(new Uri(URLstring)); //Invalid URI: The format of the URI could not be determined. 

    }; 

} 

Answer 1

1. Können Sie überprüfen, welche Zeichenfolge Sie tatsächlich in Ihrem Ereignisanruf erhalten?
2. Versuchen Sie, jedes Mal eine neue Variable für den Zyklus zu erstellen. anstatt eine externe String-Variable zu verwenden.

Grund: Sie mehrere Schließungen in Ihrem Zyklus erstellen, aber Sie verwenden eine globale unveränderliche Variable für diese Sache (Ihr urlString hat keinen Grund, überhaupt zu existieren).Ich kann mir nicht vorstellen, was dort vor sich geht, aber du könntest zum Zeitpunkt des Ereignisses etwas erhalten.

foreach (var currentItem in results) 
{ 
    var s = currentItem ["url"].ToString(); 
    URLButton.Clicked += (object sender, EventArgs e) => { 
     Debug.WriteLine(s); 
     Device.OpenUri(new Uri(s)); }; 
} 

Answer 2

Sie gültig UriTryCreate mit erstellen können.

MSDN

Creates a new Uri. Does not throw an exception if the Uri cannot be created.

Uri = outvalue; 

if(Uri.TryCreate("http://www.google.com", UriKind.Absolute, out outvalue)) 
{ 
    //logic 
} 

Answer 3

Ich nehme an, das irgendwo außerhalb Ihres geschrieben Code existiert:

string URLString; 
Button URLButton; 

Mit Ihrer Schleife, Sie einige Dinge falsch machen.

Zuerst fügen Sie einen Clicked-Handler für jedes Element in Ihrer "Ergebnis" -Liste hinzu. Wenn die Schaltfläche angetippt wird, wird versucht, sie alle einzeln auszulösen (Event-Handler werden als Multicast-Delegaten implementiert). Wenn einer von ihnen fehlschlägt, werden die verbleibenden nicht ausgeführt.

Zweitens referenzieren Sie URLString innerhalb des Clicked-Handlers. Dies bedeutet, dass ALLE Clicked-Handler mit dem gleichen Wert ausgeführt werden.

Drittens, da URLString außerhalb der Schleife definiert ist und Sie seinen Wert innerhalb der Schleife zuweisen, bedeutet dies, dass nach Abschluss der Schleife NUR der Wert des letzten Elements in den "Ergebnissen" angezeigt wird. Liste.

Vielleicht gibt es hier mehr Code, den Sie uns nicht gezeigt haben - wie URLString und URLButton tatsächlich innerhalb des Schleifenkörpers deklariert werden? Diese Probleme müssen jedoch behoben werden, bevor Sie erwarten können, dass Ihr Code wie vorgesehen funktioniert.

Ich habe meine "bereinigte" Version Ihres Codeausschnitts hier aktualisiert. Es ist das gleiche wie die aktualisierte Schnipsel aber nicht stumm stattdessen eine Ausnahme zu werfen, wenn eine schlechte URL schlagen:

foreach (var currentItem in getInfofromDB["results"]) { 
    URLstring = currentItem ["url"].ToString(); 
    URLButton.Clicked += (object sender, EventArgs e) => { 
     Uri uri; 
     if (Uri.TryCreate(URLstring, UriKind.Absolute, out uri)) { 
      Device.OpenUri(uri); 
     } 
    }; 
} 

Answer 4

die mit URL nur versuchen lassen, ohne die foreach:

URLstring = getInfofromDB["results"][0]["url"].ToString(); 

URLButton.Clicked += (object sender, EventArgs e) => { 
    Uri uri; 
    if (Uri.TryCreate(URLstring, UriKind.Absolute, out uri)) { 
     Device.OpenUri(uri); 
    } 
}; 

Mit dem foreach, Sie weisen den URLButton-Ereignishandler so oft wie die Anzahl der Ergebnisse zu. Es ist nicht gut, zu viele URLs zu öffnen, oder?