Læs url fra fil

Et eksempel på hvordan det kan gøres, hvis hver URL er separeret med et linjeskift:

List<string> urlList = new List<string>();
using (FileStream stream = File.OpenRead(@"c:\temp\test.txt"))
{
    using (StreamReader reader = new StreamReader(stream))
    {
        while (reader.Peek() > 0)
        {
            urlList.Add(reader.ReadLine());
        }
        reader.Close();
    }
}

Jeg er ikke helt sikker på at jeg forstår, hvorfor starte en tråd inde i din kode! Kan du forklare det, så jeg er sikker på jeg ikke misforstår noget.

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
using System.Threading;
using System.IO;

namespace WindowsFormsApplication1
{
    public partial class Form1 : Form
    {
        string[] url;
        private delegate void change(string URL);
        WebBrowser browser = new WebBrowser();
        bool continue_ = true;


        private void ChangeUrl(string URL)
        {
            if (browser.InvokeRequired)
            {
                change d = new change(ChangeUrl);
                browser.Invoke(d, new object[] { URL });
            }
            else
            {
                    browser.Navigate(URL);
            }         
        }

        private void thread()
        {
            while (continue_)
            {
                for (int i = 0; i < url.Length; i++)
                {
                    ChangeUrl(url[i]);

                    System.Threading.Thread.Sleep(10000);
                }
            }           
        }

        public Form1()
        {
            InitializeComponent();
            browser = webBrowser1;
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            url = null;

            StreamReader rdr = new StreamReader(@"c:\test.txt");

            string text = rdr.ReadToEnd();

            url = text.Split(Convert.ToChar(","));

            //browser.Navigate(url[0]);
            Thread thread = new Thread(new ThreadStart(this.thread));
            thread.Start();
        }
    }
}

Jeg vil råde dig til ikke at bruge multithreading til noget så simpelt som dette. Ovenstående kode vil helt sikkert give problemer.

eksempelvis (med mindre jeg overser et eller andet) kan du ikke umiddelbart lukke programmet, da der ikke er noget der fortæller tråden at den skal stoppe. Så når du lukker dit program, vil det i værste fald ikke lukke før 10sek multipliceret med antal url'er. Dette er hvis formen altså ikke kaster en Exception, fordi Invoke() metoden kaldes på formen efter den er disposed.

Ovenstående kode gør verdens simpleste problem kompliceret. Hvorfor?

Kun brug multithreading der det er nødvendigt. Ovenstående kode kan gøres meget simplere med en Timer. Hvis man absolut vil bruge multithreading, bør man bruge en BackgroundWorker.

Jeg vil mene dette er en bedre løsning. Koden går ud fra at du har trukket en WebBrowser og en Timer fra toolbox'en, ind på din form.

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Windows.Forms;
using System.IO;

namespace WindowsFormsApplication1
{
    public partial class urltest : Form
    {
        private List<string> urlList = new List<string>();
        int currentUrlIndex = 0;

        public urltest()
        {
            InitializeComponent();

            using (FileStream stream = File.OpenRead(@"c:\temp\test.txt"))
            {
                using (StreamReader reader = new StreamReader(stream))
                {
                    while (reader.Peek() > 0)
                    {
                        urlList.Add(reader.ReadLine());
                    }
                    reader.Close();
                }
            }

            timer1_Tick(this.timer1, EventArgs.Empty);
            timer1.Interval = 10000;
            this.timer1.Enabled = true;
        }

        private void timer1_Tick(object sender, EventArgs e)
        {
            this.webBrowser1.Navigate(this.urlList[currentUrlIndex++]);
            if (currentUrlIndex >= urlList.Count)
                currentUrlIndex = 0;
        }
    }
}

aaberg_cc har ret, min kode er mere kompliceret, og den er ikke testet, men vil ihvertfald give fejl når programmet lukkes.

Og som han siger, er der ingen grund til at gøre det mere besværligt end det egentlig er....!!!

aaberg_cc: Perfekt! Smider du ikke et svar?! :o)

:-)