Нужно оптимизировать код

[OmegaBerkut]

Всем доброго времени суток.
Есть следующий код:

void LoadingThread()
{
	HttpWebRequest rq=(HttpWebRequest)WebRequest.Create(requeststring); // глобальная string, содержащая браузерную строку страницы
	HttpWebResponse rs=(HttpWebResponse)rq.GetResponse();
	Stream loader=rs.GetResponseStream();
	int ch=0;
	loadingdata=""; // глобальная string, в которую грузится код страницы
	while (!loaded) // глобальная bool, нужна для продолжения работы за пределами этого потока
	{
		try
		{
			ch=loader.ReadByte(); // оптимизировать нужно это
		} catch
		{
			ch=-1;
			loadingdata="";
		}
		if (ch!=-1)
			loadingdata+=(char)ch;
		else
			loaded=true;
	}
	loader.Close();
}
// потом с загруженными данными я творю то, что мне нужно

Собственно, именно так я поступил потому, что не знаю длину потока (loader.Length выдаёт исключение). Побайтное чтение - это оочень долго, при том, что мне нужно загрузить всю страницу.
Как можно это дело улучшить ?
P. S. Кириллица так не распознаётся - это я знаю. В принципе - для меня это не проблема. Хотя, если есть простой способ исправить - я не буду возражать.

[Пепел Феникса]

оберните в StreamReader с выбором нужной кодировки и у него уже ReadToEnd()
переделаю мою процедуру, думаю поможет.

	private static string GetToString(string url, string referer = null)
	{
		try
		{
			var client = (HttpWebRequest)HttpWebRequest.Create(url);
			client.UserAgent = "Opera/9.80 (Windows NT 6.2; Win64; x64) Presto/2.12.388 Version/12.17";
			client.Accept = "text/html, application/xml;q=0.9, application/xhtml+xml, image/png, image/webp, image/jpeg, image/gif, image/x-xbitmap, */*;q=0.1";
			client.Headers["Accept-Language"] = "ru-RU,ru;q=0.9,en;q = 0.8";
			client.Headers["Accept-Encoding"] = "gzip, deflate";
			client.AutomaticDecompression = DecompressionMethods.GZip | DecompressionMethods.Deflate;
			if (referer != null)
			{
				client.Referer = referer;
			}

			using (var resp = (HttpWebResponse)client.GetResponse())
			{
				if (resp.StatusCode == HttpStatusCode.OK)
				{
					using (var web = resp.GetResponseStream())
					{
						using (var sr = new StreamReader(str, Encoding.UTF8)) //Вот тут идет выбор кодировки, у меня стоит UTF8
						{
							return sr.ReadToEnd();
						}
					}
				}
			}
		}
		catch
		{
			return nulll;
		}

		return null;
	}

данный код так же обрабатывает и сжатые страницы.(и косит под браузер)
(надеюсь не напутал нигде при переносе под вашу ситуацию.

[OmegaBerkut]

Пепел Феникса
У меня это изначально должен быть поток. Обычный Thread не принимает функции с параметрами (или я чего то не знаю).

[Пепел Феникса]

вам кто-то запрещает вызвать мою функцию из кода потока или переделать ее чисто под поток?

[OmegaBerkut]

Сообщение от Пепел Феникса

вам кто-то запрещает вызвать мою функцию из кода потока или переделать ее чисто под поток?

Чисто под поток - это разве что все параметры перегнать в глобальные. А так - всё ок.

[Пепел Феникса]

в поток можно передать параметр между прочим.
глобальные переменные ограничивают вас одним потоком.

вам нужен один параметр, второй параметр функции у вас будет null.
его можно и вырезать в принципе, просто в моем коде он есть.

[OmegaBerkut]

Пепел Феникса
Такой вопрос:

using (var sr = new StreamReader(str, Encoding.UTF8))
{
	return sr.ReadToEnd();
}

Что за str в параметрах вызова конструктора ?
Я посмотрел по документации - это "путь к файлу для чтения".
Собственно я и не могу понять, зачем это ... И с чем его есть.

[Пепел Феникса]

это я при адаптации таки накосячил
там web должен быть.

private static string GetToString(string url, string referer = null)
        {
                try
                {
                    var client = (HttpWebRequest)HttpWebRequest.Create(url);
                    client.UserAgent = "Opera/9.80 (Windows NT 6.2; Win64; x64) Presto/2.12.388 Version/12.17";
                    client.Accept =
                        "text/html, application/xml;q=0.9, application/xhtml+xml, image/png, image/webp, image/jpeg, image/gif, image/x-xbitmap, */*;q=0.1";
                    client.Headers["Accept-Language"] = "ru-RU,ru;q=0.9,en;q = 0.8";
                    client.Headers["Accept-Encoding"] = "gzip, deflate";
                    client.AutomaticDecompression = DecompressionMethods.GZip | DecompressionMethods.Deflate;
                    if (referer != null)
                    {
                        client.Referer = referer;
                    }
                    using (var resp = (HttpWebResponse)client.GetResponse())
                    {
                        if (resp.StatusCode == HttpStatusCode.OK)
                        {
                            using (var web = resp.GetResponseStream())
                            {  
                    using (var sr = new StreamReader(web, Encoding.UTF8))  //Вот тут идет выбор кодировки, у меня стоит UTF8
                    {
                        return sr.ReadToEnd();
                    }
                            }
                        }
                    }
                }
                catch
                {
                    return nulll;
                }
            return null;
        }

просто исходный код немного сложнее, вам это ни к чему

[Alex11223]

Видимо туда надо было передать web созданный выше, а не str.

А вообще лучше не париться с HttpWebRequest и взять WebClient или HttpClient (.NET 4.5).

[OmegaBerkut]

В общем, я модифицировал код под себя и свои нужды (всё работает).

private void LoadingThread()
{
	try
	{
		HttpWebRequest client=(HttpWebRequest)HttpWebRequest.Create(requeststring);
		client.UserAgent="Opera/9.80 (Windows NT 6.2; Win64; x64) Presto/2.12.388 Version/12.17";
		client.Accept=
				"text/html, application/xml;q=0.9, application/xhtml+xml, image/png,"+
				"image/webp, image/jpeg, image/gif, image/x-xbitmap, */*;q=0.1";
		client.Headers["Accept-Language"]="ru-RU,ru;q=0.9,en;q = 0.8";
		client.Headers["Accept-Encoding"]="gzip, deflate";
		client.AutomaticDecompression=(DecompressionMethods.GZip | DecompressionMethods.Deflate);
		HttpWebResponse resp=(HttpWebResponse)client.GetResponse();
		if (resp.StatusCode==HttpStatusCode.OK)
		{
			Stream loader=resp.GetResponseStream();
			StreamReader reader=new StreamReader(loader);
			loadingdata=reader.ReadToEnd(); // loadingdata - глобальная string
		} else {}
	}
	catch {}
}

Кириллицу грузит, почти всё хорошо. Единственная проблема - это поведение при разрыве связи: по завершении потока почему то выполнение потока начинается сначала. Но это не проблема: моя конструкция позволяет пресекать такие ситуации - поток убивается принудительно, если он работает больше определённого времени.