listBoxとTimerについて C#

質問

listBoxとTimerについて　C#
ヤフーニュースのURLをlistboxへ入れます。
そしてlistBox1の中身を画面表示させたら、次にlistBox2の中身も表示したいと思っています。

最初listBoxが1つだったときはうまく行っていたのですが、listBoxを増やすとうまく行かなくなりました。

この状態だと、listBox2の中身だけ表示させて終わってしまいます。
Timerの使い方が怪しいと思うのですが、どうでしょうか？

些細なことでも何でもいいのでご意見頂ければ助かります。
-----以下コード抜粋------
public partial class Form1 : Form
{
public Form1()
{
InitializeComponent();
}
int urlindex = 0;

private void Form1_Load(object sender, EventArgs e)
{
listBox1.AllowDrop = true;
listBox1.DragEnter += new DragEventHandler(listBox1_DragEnter);
listBox1.DragDrop += new DragEventHandler(listBox1_DragDrop);
listBox2.AllowDrop = true;
listBox2.DragEnter += new DragEventHandler(listBox2_DragEnter);
listBox2.DragDrop += new DragEventHandler(listBox2_DragDrop);
}
private void listBox1_DragEnter(object sender, DragEventArgs e)
{
//URLのみ受け入れる//@ITより
if (e.Data.GetDataPresent("UniformResourceLocator"))
e.Effect = DragDropEffects.Link;
else
e.Effect = DragDropEffects.None;
}
private void listBox2_DragEnter(object sender, DragEventArgs e)
{
//URLのみ受け入れる//@ITより
if (e.Data.GetDataPresent("UniformResourceLocator"))
e.Effect = DragDropEffects.Link;
else
e.Effect = DragDropEffects.None;
}
private void listBox1_DragDrop(object sender, DragEventArgs e)
{
//ドロップされたリンクのURLを取得する//@ITより
string url = e.Data.GetData(DataFormats.Text).ToString();
//結果を表示
listBox1.Text = url;
//MessageBox.Show(url);
//ドロップされたデータがstring型か調べる
if (e.Data.GetDataPresent(typeof(string)))
{
ListBox target = (ListBox)sender;
//ドロップされたデータ(string型)を取得
string itemText =
(string)e.Data.GetData(typeof(string));
//ドロップされたデータをリストボックスに追加する
target.Items.Add(url);
//MessageBox.Show("表示");
}
}
private void listBox2_DragDrop(object sender, DragEventArgs e)
{
//ドロップされたリンクのURLを取得する//@ITより
string url = e.Data.GetData(DataFormats.Text).ToString();
//結果を表示
listBox2.Text = url;
//MessageBox.Show(url);
//ドロップされたデータがstring型か調べる
if (e.Data.GetDataPresent(typeof(string)))
{
ListBox target = (ListBox)sender;
string itemText =
(string)e.Data.GetData(typeof(string));
target.Items.Add(url);
}
}
private void goButton_Click(object sender, EventArgs e)
{
urlindex = 0;
timer1.Start();
timer2.Start();

}

private void timer1_Tick(object sender, EventArgs e)
{
timer1.Stop();
if (listBox1.Items.Count != 0 && urlindex < listBox1.Items.Count)
{
string url = (string)listBox1.Items[urlindex];
webBrowser1.Navigate(url);
urlindex++;
}
}

private void timer2_Tick_1(object sender, EventArgs e)
{

timer2.Stop();
if (listBox2.Items.Count != 0 && urlindex < listBox2.Items.Count)
{
string url = (string)listBox2.Items[urlindex];
}
}
private void webBrowser1_DocumentCompleted(object sender, WebBrowserDocumentCompletedEventArgs e)
{
timer1.Start();
timer2.Start();

}

private void listBox2_SelectedIndexChanged(object sender, EventArgs e)
{
}

回答No.3

田中 裕之（@bybalsendercase） · ベストアンサー

こんにちは

今日は日曜日でお休みでした。（笑

＞「url1」を全て表示したら、そのまま「url2」も続けて全て表示させることです。
なんとなくご質問の内容がわかってきました。
単純に「url1」を全て表示したら、そのまま「url2」も続けて全て表示するサンプルを作成しました、

単純にするためにwebBrowser1_DocumentCompletedのイベントハンドラーは使用していまん。
もし、必要な場合はお手数ですがもう一度返信してください。

int urlindex = 0;
private void Form1_Load(object sender, EventArgs e)
{
webBrowser1.ScriptErrorsSuppressed = true;//スクリプトエラーを表示しない
timer1.Interval = 3000;
timer2.Interval = 3000;
listBox1.AllowDrop = true;
listBox1.DragEnter += new DragEventHandler(listBox1_DragEnter);
listBox1.DragDrop += new DragEventHandler(listBox1_DragDrop);
listBox2.AllowDrop = true;
listBox2.DragEnter += new DragEventHandler(listBox2_DragEnter);
listBox2.DragDrop += new DragEventHandler(listBox2_DragDrop);
}
private void listBox1_DragEnter(object sender, DragEventArgs e)
{
if (e.Data.GetDataPresent("UniformResourceLocator"))
e.Effect = DragDropEffects.Link;
else
e.Effect = DragDropEffects.None;
}
private void listBox2_DragEnter(object sender, DragEventArgs e)
{
if (e.Data.GetDataPresent("UniformResourceLocator"))
e.Effect = DragDropEffects.Link;
else
e.Effect = DragDropEffects.None;
}
private void listBox1_DragDrop(object sender, DragEventArgs e)
{
string url1 = e.Data.GetData(DataFormats.Text).ToString();
　　　　　listBox1.Text = url1;
　　　　　　if (e.Data.GetDataPresent(typeof(string)))
　　　　　{
　　　 ListBox target = (ListBox)sender;
　　　　 string itemText =
　　　 (string)e.Data.GetData(typeof(string));
　　　 target.Items.Add(url1);
　　　　}
}

private void listBox2_DragDrop(object sender, DragEventArgs e)
{
string url2 = e.Data.GetData(DataFormats.Text).ToString();
listBox2.Text = url2;
if (e.Data.GetDataPresent(typeof(string)))
{
ListBox target = (ListBox)sender;
string itemText =
(string)e.Data.GetData(typeof(string));
target.Items.Add(url2);
//MessageBox.Show("表示");
}
}

private void goButton_Click(object sender, EventArgs e)
{
urlindex = 0;
timer1.Start();//timer1からスタート
}
private void timer1_Tick(object sender, EventArgs e)
{
MessageBox.Show("timer_1_TICK");
if (listBox1.Items.Count != 0 && urlindex < listBox1.Items.Count)
{
string url1 = (string)listBox1.Items[urlindex];
webBrowser1.Navigate(url1);
urlindex++;
}
//listBox1のURLの表示が終了したらtimer1を停止、timer2を起動、
//urlindex = 0にする
else if (urlindex >= listBox1.Items.Count)
{
timer1.Enabled = false;
timer2.Enabled = true;
urlindex = 0;
}
}
private void timer2_Tick(object sender, EventArgs e)
{
if (listBox2.Items.Count != 0 && urlindex < listBox2.Items.Count)
{
string url2 = (string)listBox2.Items[urlindex];
webBrowser1.Navigate(url2);
urlindex++;
}
//listBox1のURLの表示が終了したらtimer2を停止
else if (urlindex >= listBox2.Items.Count)
{
timer2.Enabled = false;
}
　　　　　　}

お礼

やってみました。
私がやりたかったことはこれです！
とても助かりました。
この次はURLをDBへ入れたいと思います。
また、わからないときは質問させて頂きますのでどうぞ宜しくお願い致します。

回答No.2

田中 裕之（@bybalsendercase）

続きです。

private void webBrowser1_DocumentCompleted(object sender, WebBrowserDocumentCompletedEventArgs e)
{
//読み込まれるURLによって処理を分ける
if (e.Url.ToString() == url1)
{
timer2.Enabled=true;
}
else if (e.Url.ToString() == url2)
{
timer1.Enabled = true;
}
}

private void listBox2_SelectedIndexChanged(object sender, EventArgs e)
{

}

よろしくお願いします。

お礼

bybalsendercase様
本当にありがとうございました。
次にLisBoxに入ったURLを表示させるのではなく、データベースに入れたいと思い買いて見ました。
また躓いてしまったので、もしお時間ございましたら回答頂けると嬉しいです。
以下URLです。↓
http://okwave.jp/qa/q8064923.html

補足

bybalsendercase様
ご回答頂きありがとうございます。
以下のように書き換えて見ました。
これで振り分けは出来るようになりました。
目的達成まであと少しのような気がします。
目的は「url1」を全て表示したら、そのまま「url2」も続けて全て表示させることです。
どう書けばいいのでしょうか？
宜しくお願い致します。
-----------------------------------
private void Form1_Load(object sender, EventArgs e)
{
listBox1.AllowDrop = true;
listBox1.DragEnter += new DragEventHandler(listBox1_DragEnter);
listBox1.DragDrop += new DragEventHandler(listBox1_DragDrop);
listBox2.AllowDrop = true;
listBox2.DragEnter += new DragEventHandler(listBox2_DragEnter);
listBox2.DragDrop += new DragEventHandler(listBox2_DragDrop);
}
private void listBox1_DragEnter(object sender, DragEventArgs e)
{
if (e.Data.GetDataPresent("UniformResourceLocator"))
e.Effect = DragDropEffects.Link;
else
e.Effect = DragDropEffects.None;
}
private void listBox2_DragEnter(object sender, DragEventArgs e)
{
if (e.Data.GetDataPresent("UniformResourceLocator"))
e.Effect = DragDropEffects.Link;
else
e.Effect = DragDropEffects.None;
}
private void listBox1_DragDrop(object sender, DragEventArgs e)
{
string url1 = e.Data.GetData(DataFormats.Text).ToString();
listBox1.Text = url1;
if (e.Data.GetDataPresent(typeof(string)))
{
ListBox target = (ListBox)sender;
string itemText =
(string)e.Data.GetData(typeof(string));
target.Items.Add(url1);
}
}
private void listBox2_DragDrop(object sender, DragEventArgs e)
{
string url2 = e.Data.GetData(DataFormats.Text).ToString();
listBox2.Text = url2;
if (e.Data.GetDataPresent(typeof(string)))
{
ListBox target = (ListBox)sender;
string itemText =
(string)e.Data.GetData(typeof(string));
target.Items.Add(url2);
//MessageBox.Show("表示");
}
}
private void goButton_Click(object sender, EventArgs e)
{
urlindex = 0;
timer1.Start();
timer2.Start();
}

private void timer1_Tick(object sender, EventArgs e)
{
timer1.Stop();
MessageBox.Show("timer_1_TICK");
if (listBox1.Items.Count != 0 && urlindex < listBox1.Items.Count)
{
string url1= (string)listBox1.Items[urlindex];
webBrowser1.Navigate(url1);
urlindex++;
}
}
private void timer2_Tick_2(object sender, EventArgs e)
{
timer2.Stop();
MessageBox.Show("timer_2_TICK");

if (listBox2.Items.Count != 0 && urlindex < listBox2.Items.Count)
{
string url2 = (string)listBox2.Items[urlindex];
webBrowser1.Navigate(url2);
urlindex++;
}
//Console.WriteLine(listBox2.Items[0]);
}

private void webBrowser1_DocumentCompleted(object sender, WebBrowserDocumentCompletedEventArgs e)
{
timer1.Start();
timer2.Start();
//読み込まれるURLによって処理を分ける
string url1 = "";
string url2 = "";
if (e.Url.ToString() == url1)
{
timer2.Enabled = true;
}
else if (e.Url.ToString() == url2)
{
timer1.Enabled = true;
}
}

private void listBox2_SelectedIndexChanged(object sender, EventArgs e)
{
}

回答No.1

田中 裕之（@bybalsendercase）

こんばんは

＞listBox1の中身を画面表示させたら、次にlistBox2の中身も表示したいと思っています。
すいません、この部分の意味がわかりませんでしたので、適当にサンプルを作成しましたので、どこがどのように違うのかできる範囲で結構ですので教えてください。

サンプルではlistBox1とlistBox2に同じ数だけURLを貼り付けて、goButtonをクリックすると交互にlistBox1とlistBox2のURLをwebBrowser1に表示されるようにしてあります。

何回か回答返信の繰り返しをしないと、もしかしたら質問の意味が掴めないかもしれません。
但し、私は明日から出勤ですので、回答できませんので、他の方に委ねます。

public Form1()
{
InitializeComponent();
}
int urlindex1 = 0;//この部分を変更
int urlindex2 = 0;//この部分を変更
private void Form1_Load(object sender, EventArgs e)
{
listBox1.AllowDrop = true;
listBox1.DragEnter += new DragEventHandler(listBox1_DragEnter);
listBox1.DragDrop += new DragEventHandler(listBox1_DragDrop);
listBox2.AllowDrop = true;
listBox2.DragEnter += new DragEventHandler(listBox2_DragEnter);
listBox2.DragDrop += new DragEventHandler(listBox2_DragDrop);

}
private void listBox1_DragEnter(object sender, DragEventArgs e)
{
//URLのみ受け入れる//@ITより
if (e.Data.GetDataPresent("UniformResourceLocator"))
e.Effect = DragDropEffects.Link;
else
e.Effect = DragDropEffects.None;
}
private void listBox2_DragEnter(object sender, DragEventArgs e)
{
//URLのみ受け入れる//@ITより
if (e.Data.GetDataPresent("UniformResourceLocator"))
e.Effect = DragDropEffects.Link;
else
e.Effect = DragDropEffects.None;
}

private void listBox1_DragDrop(object sender, DragEventArgs e)
{
//ドロップされたリンクのURLを取得する//@ITより
string url = e.Data.GetData(DataFormats.Text).ToString();
//結果を表示
listBox1.Text = url;
//MessageBox.Show(url);
//ドロップされたデータがstring型か調べる
if (e.Data.GetDataPresent(typeof(string)))
{
ListBox target = (ListBox)sender;
//ドロップされたデータ(string型)を取得
string itemText =
(string)e.Data.GetData(typeof(string));
//ドロップされたデータをリストボックスに追加する
target.Items.Add(url);
//MessageBox.Show("表示");
}
}
private void listBox2_DragDrop(object sender, DragEventArgs e)
{
//ドロップされたリンクのURLを取得する//@ITより
string url = e.Data.GetData(DataFormats.Text).ToString();
//結果を表示
listBox2.Text = url;
//MessageBox.Show(url);
//ドロップされたデータがstring型か調べる
if (e.Data.GetDataPresent(typeof(string)))
{
ListBox target = (ListBox)sender;
string itemText =
(string)e.Data.GetData(typeof(string));
target.Items.Add(url);
}
}

private void goButton_Click(object sender, EventArgs e)
{
urlindex1 = 0;//この部分を変更
urlindex2 = 0;//この部分を変更
timer1.Start();
}
string url1;
private void timer1_Tick(object sender, EventArgs e)
{
timer1.Enabled = false;
if ((listBox1.Items.Count != 0) && (urlindex1 < listBox1.Items.Count))
{
url1 = (string)listBox1.Items[urlindex1];
webBrowser1.Navigate(url1);
urlindex1++;
}
}
string url2;
private void timer2_Tick(object sender, EventArgs e)
{
timer2.Enabled = false;
if ((listBox2.Items.Count != 0) && (urlindex2 < listBox2.Items.Count))
{
url2 = (string)listBox2.Items[urlindex2];
webBrowser1.Navigate(url2);
urlindex2++;
}
}

DocumentCompletedの部分が文字制限で掲載できませんでしたので、次ページに続きます。

お礼

bybalsendercase様
ご丁寧にありがとうございます。
やってみます！