В решении Джеффа есть несколько вещей, которые можно улучшить.
if (String.IsNullOrEmpty(title)) return "";
ИМХО, не место для проверки этого. Если функция получает пустую строку, то все равно что-то пошло не так. Выкинь ошибку или вообще не реагируй.
// remove any leading or trailing spaces left over
… muuuch later:
// remove trailing dash, if there is one
Дважды работа. Учитывая, что каждая операция создает совершенно новую строку, это плохо, даже если производительность не является проблемой.
// replace spaces with single dash
title = Regex.Replace(title, @"\s+", "-");
// if we end up with multiple dashes, collapse to single dash
title = Regex.Replace(title, @"\-{2,}", "-");
Опять же, в основном, вдвое больше работы: во-первых, используйте регулярные выражения для замены нескольких пробелов одновременно. Затем снова используйте regex для замены нескольких тире одновременно. Два выражения для разбора, два автомата для построения в памяти, итерация дважды по строке, создание двух строк: все эти операции могут быть сведены к одной.
Вне моей головы, без каких-либо испытаний, это было бы эквивалентное решение:
// make it all lower case
title = title.ToLower();
// remove entities
title = Regex.Replace(title, @"&\w+;", "");
// remove anything that is not letters, numbers, dash, or space
title = Regex.Replace(title, @"[^a-z0-9\-\s]", "");
// replace spaces
title = title.Replace(' ', '-');
// collapse dashes
title = Regex.Replace(title, @"-{2,}", "-");
// trim excessive dashes at the beginning
title = title.TrimStart(new [] {'-'});
// if it's too long, clip it
if (title.Length > 80)
title = title.Substring(0, 79);
// remove trailing dashes
title = title.TrimEnd(new [] {'-'});
return title;
Обратите внимание, что этот метод использует строковые функции вместо функций регулярных выражений и символьные функции вместо строковых функций.