Как лучше всего собирать данные об исключениях в функции?

[alextrof94]

В данный момент есть функция полностью помещенная в try-catch, однако это позволяет узнать только, что в ней произошло исключение, а хотелось бы узнать об исключении все вплоть до параметров функции, вызвавшей исключение.
Что делать? Я знаю всего 2 решения проблемы:
1. ввести дополнительную строку exceptionInfo, которую заполнять перед каждым потенциально-опасным действием и выдавать ее при исключении.
2. наделать кучу try-catch.
Оба этих подхода мне не нравятся из-за их громоздкости.

Вот, кстати, нынешний код функции (лишь бы работало), который я собираюсь рефакторить (не надо в нем разбираться, ибо проблема с ним не очень связана):

private string GetWins(string Page)
		{
			string result = "";

			string strG = "";

			try
			{
				if (Page.IndexOf("<div class=\"table__rows\">") < 0)
					return "";
				List<string> gettedWon = new List<string>();
				int st = Page.IndexOf("<div class=\"table__rows\">");
				int en = Page.IndexOf("<div class=\"pagination\">");
				string inPage = Page.Substring(st, en - st);

				while (inPage.IndexOf("<div class=\"table__row-outer-wrap\">") > -1)
				{
					inPage = inPage.Substring(inPage.IndexOf("<div class=\"table__row-outer-wrap\">") + "<div class=\"table__row-outer-wrap\">".Length);
					if (inPage.IndexOf("<div class=\"table__row-outer-wrap\">") > -1)
						strG = inPage.Substring(0, inPage.IndexOf("<div class=\"table__row-outer-wrap\">"));
					else
						strG = inPage;

					strG = strG.Substring(strG.IndexOf("<a class=\"table__column__heading\" href=\"") + "<a class=\"table__column__heading\" href=\"".Length);
					strG = strG.Substring(strG.IndexOf(">") + 1);
					string str = strG.Substring(0, strG.IndexOf("<"));

					gettedWon.Add(str);
				}

				List<string> oldWon = new List<string>();
				foreach (string s in WonGames)
					oldWon.Add(s);

				//delete dublicates
				foreach (string s in oldWon)
				{
					gettedWon.Remove(s);
				}

				foreach (string s in gettedWon)
				{
					WonGames.Add(s);
					result += s + "\n";
				}
			}
			catch (Exception ex)
			{
				ErrorReporting("Error GetWins(): " + ex.Message + " | string=" + strG);
			}
			return result;
		}

[lomastr_]

что-то не понятно в чем беда?
потенциально опасный код всегда известен, исключение есть, чего еще не хватает?

[alextrof94]

Этого кода много. Например функция SubString, которая выдает исключение "Значение StartIndex не может быть меньше нуля. Имя параметра: startIndex", их у меня с десяток в одной функции, как узнать какая из них выдает исключение и при каких параметрах?
Паршу один сайтишко, и это самый легкий способ добыть инфу.

[Alex11223]

Точно самый легкий? Что насчет HtmlAgilityPack? http://www.programmersforum.ru/showp...27&postcount=3

А про проблему я не понял, но в Exception вроде ж есть инфа о методе и даже номер строки, если не удалять pdb.

[alextrof94]

Точно самый легкий. Основное облегчение в том, что как раз не надо использовать посторонние классы, которые так же будут выдавать эксепшены при отсутствии искомых элементов.
Ошибки вываливаются не у меня, а у друзей, в этом и проблема.
Попробую HtmlAgilityPack, хоть и пробовал его использовать раньше.

[Alex11223]

Ну можно не нарываться на эти экзепшны, а проверять самому найден ли элемент и кидать свое исключение с нужными данными и описанием.

А библиотеки для парсинга HTML удобнее такого парсинга хотя бы в плане надежности. Добавится в этот div еще один CSS класс для подсветки чего-нибудь и сломается парсинг Substring'ами. Ну и XPath запрос по-моему проще составлять, чем так, особенно на сложных сайтах, или когда лишние элементы попадаются и надо точнее выбирать.

[alextrof94]

Ну вот. переписал функции под HtmlAgilityPack. Стало проще, но теперь код выглядит громоздко...
И встал другой вопрос, как внедрить библиотеку в exe.
UPD: вес приложения увеличился в 3 раза =/

[lomastr_]

юзайте регексы https://msdn.microsoft.com/en-us/lib...v=vs.100).aspx

[Alex11223]

Не, регекспами тоже плохо HTML парсить, не сильно лучше первого варианта.
http://stackoverflow.com/questions/1...contained-tags

В 3 раза это сколько? Стало 150 КБ вместо 50? Серьезная проблема, да.

"Внедрить" можно ILMerge, но смысла мало. Чем вам так dll мешает? Вы ж не вирусы пишете. Отправляйте папку в zip архиве либо инсталлятор используйте.

[lomastr_]

html это текст, регекс для текста и создан, что не так?