MCDzienny

You forgot one important piece of your url....

http://

wait nevermind, I fixed it lol, it had to do with already having the cmdloaded

As ismellike suggested, it probably would be a good idea to use a try/catch statement as we're reading from online.

So just change this:

Code: Select all

string content = new System.Net.WebCient().DownloadString(URL);


To:

Code: Select all

try
{
   string content = new System.Net.WebCient().DownloadString(URL);
}
catch
{
   Player.SendMessage(p, "An error occurred retrieving server news.");
}


Hmm perfect! Thanks guys!

Leeizazombie wrote:Hmm perfect! Thanks guys!

No problem

Oh one more thing, is there a possible way to use something like File.ReadAllLines in the webclient version?

Wow. This is now a usefull thread for everyone(I didn't know about File.ReadAllText)!
Anyway you could also do it with streamreader, but it will require more code so it is useless.

Leeizazombie wrote:Oh one more thing, is there a possible way to use something like File.ReadAllLines in the webclient version?

I am not sure about this myself. I would do something like adding an uncommon symbol in the text when I want it to be a new line. There is probably a much more simpler way but I can only think of this sorry. So I'll show you what I mean.

I would store this as my textfile online:

Hello welcome to my server$Please read the rules$You can type /rules$Also check out the /help menu

Then in my code I would do something like:

Code: Select all

public override void Use(Player p, string message)
{
   try
   {
      string content = new System.Net.WebClient().DownloadString("http://conor.com/news.txt");
     
      foreach(string line in content.Split('$'))
      {
          Player.SendMessage(p, line);
      }
   }
   catch
   {
      Player.SendMessage(p, "Error getting news.");
   }
}


The final code I put together for you nerds,

lol, wtf? this post was made more than a year ago, why are you necroposting?