Xamarinフォーム、エラーこの例ではwebhttpリンク

Question

を開こうとすると、私はこのように見ているのhttpアドレスを開きます。http://www.url.com/と私のエラーを受け取る：

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

は、これは私が使用していたコードです。そして、ユーザーが2tなしでhtp：/のような悪いアドレスを設定した場合、どのようにしてクラッシュを避けることができますか？

これは私のコードです：

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. イベントコール内で実際にどの文字列を確認できますか？
2. サイクルごとに新しい変数を作成してみてください。外部の文字列変数を使用する代わりに。

理由：あなたのサイクルで複数のクロージャを作成しているが、あなたはこの事のために一つのグローバル不変変数を使用します（あなたのURLstringがまったく存在する理由がありません）。私はそこで何が起こっているのか想像することはできませんが、イベントコールの時に何かを得るかもしれません。

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

あなたはTryCreateを使用して、有効なUriを作成することができます。

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

私は、これはどこかにあなたの投稿コードの外に存在すると仮定します。あなたのループで

string URLString; 
Button URLButton; 

、あなたは間違っていくつかのことをやっています。

まず、 "結果"リストの各項目にClickedハンドラを追加します。ボタンをタップすると、一度に1つずつ（イベントハンドラはマルチキャストデリゲートとして実装されます）、すべてのイベントを起動しようとします。それらのANYが失敗すると、残りのものは実行されません。

次に、Clickedハンドラ内からURLStringを参照しています。これは、すべてのクリックされたハンドラが同じ値を使用して実行されることを意味します。

第3に、URLStringはループ外で定義されており、ループ内からその値を割り当てているため、ループが終了すると、結果セットの最後の項目の値のみを持ちます。リスト。

おそらく、URLStringとURLButtonが実際にループ本体内で宣言されているように、私たちが表示していないコードがいくつかありますか？しかし、これらの問題は、コードが意図したとおりに動作することを期待する前に修正する必要があります。

ここでコードスニペットの「クリーンアップ」バージョンを更新しました。それはあなたの更新スニペットと同じであるが、代わりに悪いURLを打ったときに例外をスローで静かに失敗します。

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

はのは、foreachのなしで、唯一のURLを試してみましょう：

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); 
    } 
}; 

のforeachで、結果の数だけURLButtonイベントハンドラを割り当てます。あまりにも多くのURLを開くのは良いことではありませんか？