goto

[ds.Dante]

Я не парсер XML пишу, а обычный подстановщик цифры по одному тегу. И вместо одного простого метода делать несколько, да ещё с рекурсией - не резон. Как раз получится в ущерб читаемости и поддерживаемости. Учтите также время, потраченное на написание кода: если я на каждый метод буду уделять столько внимания, меня уволят с работы за разгильдяйство. :)

[Utkin]

Ну тут как бы решить надо по обстоятельствам - быстро написать неудобный код и потом его расхлебывать через пару лет (Вы же уже расхлебываете чужой , то есть опять же рекурсия), либо сразу через "нехочу" сделать как положено и не мучаться ни с отладкой ни с сопровождением.

если я на каждый метод буду уделять столько внимания, меня уволят с работы за разгильдяйство.

Очень странное поведение начальства. Либо Ваши отношения с ним. Объяснитесь по поводу функции. Мое уже понимает, что если я бегаю казалось бы на пустом месте, значит это ж-ж-ж-ж не спроста.

[ds.Dante]

Всё уже, с благославления начальства я изменил интерфейсы класса, и этот метод практически весь полетел в мусорку, а то, что осталось, переписано с регексами, с которыми я уже успел освоиться. Так что правильно, что я не стал здесь заниматься научной работой.

Сообщение от Utkin

сразу через "нехочу" сделать как положено и не мучаться ни с отладкой ни с сопровождением.

В каком именно месте могли бы возникнуть проблемы с отладкой или сопровождением?

[Utkin]

В каком именно месте могли бы возникнуть проблемы с отладкой или сопровождением?

В самом неожиданном. Я не знаю всех условий задачи, но чисто абстрактно - может изменится шаблон строки...

[ds.Dante]

И какое это имеет отношение к сабжу? Несколько функций с рекурсией проще отлаживать и рефакторить?

[BOBAH13]

Не читал все посты, но сразу из примера вашего ясно - разбить на несколько более мелких функций. Самый верный вариант, а не goto и/или дублирование.

UPD:

public static bool GetFree (String Expression, IEnumerable<String> Exists, out String Result)

Кстати то же зло! Я про out Result. Почему не сделать так?

public static String GetFree (String Expression, IEnumerable<String> Exists)

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

Почему не сделать так?

согласен, если не удалось ничего выпарсить, то возвращаем null.

[ds.Dante]

Сейчас я так и сделал, но на тот момент (нет доступа ко всем зависимым проектам, сигнатуры методов менять нельзя) то был единственный вариант.

Сообщение от BOBAH13

Не читал все посты, но сразу из примера вашего ясно - разбить на несколько более мелких функций.

На какие именно (примерно)?

[BOBAH13]

Сообщение от ds.Dante

Сейчас я так и сделал, но на тот момент (нет доступа ко всем зависимым проектам, сигнатуры методов менять нельзя) то был единственный вариант.


На какие именно (примерно)?

Прошу прощения, но вы действительно не замечаете сходства? (дублирование)

if (Tmp.StartsWith (NumberType.Any.ToString ()))
			{
				#region Any
				Tmp = Tmp.Substring (3);
				Type = NumberType.Any;
				if (!String.IsNullOrEmpty (Tmp))
				{
					if (!Tmp.StartsWith ("(") || !Tmp.EndsWith (")"))
						return false;

					Tmp = Tmp.Substring (1, Tmp.Length - 2);
					if (!String.IsNullOrEmpty (Tmp))
					{
						int p;
						if (!int.TryParse (Tmp, out p))
							return false;

						Param = p;
						UseParam = true;
					}
				}
				#endregion
			}
			else if (Tmp.StartsWith (NumberType.Next.ToString ()))
			{
				#region Next
				Tmp = Tmp.Substring (4);
				Type = NumberType.Next;
				if (!String.IsNullOrEmpty (Tmp))
				{
					if (!Tmp.StartsWith ("(") || !Tmp.EndsWith (")"))
						return false;
					Tmp = Tmp.Substring (1, Tmp.Length - 2);
					if (!String.IsNullOrEmpty (Tmp))
					{
						int p;
						if (!int.TryParse (Tmp, out p))
							return false;

						Param = p;
						UseParam = true;
					}
				}
				#endregion
			}

[BOBAH13]

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

согласен, если не удалось ничего выпарсить, то возвращаем null.

Лучше не null, а пустую строку.