Крошечный способ получить первые 25 символов

public static Take(this string s, int i)
{
    if(s.Length <= i)
        return s
    else
        return s.Substring(0, i) + "..."
}

public string ShortDescription
{
    get { return this.Description.Take(25); }
}

return this.Description.Substring(0, Math.Min(this.Description.Length, 25));

Не имеет ... часть. Ваш способ является, вероятно, лучшим на самом деле.

Путем Вы сделали, это кажется прекрасным мне, за исключением того, что я использовал бы магическое число 25, у меня будет это как константа.

Вы действительно хотите сохранить это в своем бобе хотя? По-видимому, это для дисплея где-нибудь, таким образом, Ваш рендерер должен быть вещью, делающей усечение вместо объекта данных

Хорошо я знаю, что уже существует ответ, принятый, и я могу быть замучен для вывода регулярного выражения здесь, но это - то, как я обычно делаю это:

//may return more than 25 characters depending on where in the string 25 characters is at
public string ShortDescription(string val)
{
    return Regex.Replace(val, @"(.{25})[^\s]*.*","$1...");
}
// stricter version that only returns 25 characters, plus 3 for ...
public string ShortDescriptionStrict(string val)
{
    return Regex.Replace(val, @"(.{25}).*","$1...");
}

Это имеет хорошую дополнительную льготу не вырезания слова в половине, поскольку это всегда останавливается после первого пробельного символа прошлые 25 символов. (Конечно, если Вам нужен он для усечения текста, входящего в базу данных, которая могла бы быть проблемой.

Оборотная сторона, хорошо я уверен, что это не возможное быстрое решение.

Править: замененный … с "..." с тех пор не уверенный, если это решение для сети!

без.... этого должно быть самым коротким:

public string ShortDescription
{
    get { return Microsoft.VisualBasic.Left(this.Description;}
}

Я думаю, что подход является звуковым, хотя я рекомендовал бы несколько корректировок

• Переместите магическое число в a const или значение конфигурации
• Используйте постоянного клиента if условное выражение, а не тернарный оператор
• Используйте a string.Format("{0}...") вместо + "..."
• Имейте всего одну точку возврата от функции

Так:

public string ShortDescription
{
    get
    {
        const int SHORT_DESCRIPTION_LENGTH = 25;

        string _shortDescription = Description;

        if (Description.Length > SHORT_DESCRIPTION_LENGTH)
        {
            _shortDescription = string.Format("{0}...", Description.Substring(0, SHORT_DESCRIPTION_LENGTH));
        }

        return _shortDescription;
    }
}

Для более общего подхода Вы хотели бы перемещать логику в дополнительный метод:

public static string ToTruncated(this string s, int truncateAt)
{
    string truncated = s;

    if (s.Length > truncateAt)
    {
        truncated = string.Format("{0}...", s.Substring(0, truncateAt));
    }

    return truncated;
}

Править

Я использую тернарный оператор экстенсивно, но предпочитаю избегать его, если код становится достаточно подробным, что это начинает расширять прошлые 120 символов или около этого. В этом случае я хотел бы перенести его на несколько строк, поэтому найдите что постоянный клиент if условное выражение более читаемо.

Edit2

Для типографской правильности Вы могли также рассмотреть использование символа замещающего знака (…) в противоположность трем точкам/периодам/точкам (...).

Один способ сделать это:

int length = Math.Min(Description.Length, 25);
return Description.Substring(0, length) + "...";

Существует две строки вместо одной, но более короткие :).

Править: Как указано в комментариях, это получает Вас... все время, таким образом, ответ был неправильным. Исправление его означает, что мы возвращаемся к исходному решению.

На данном этапе я думаю с помощью строковых расширений, единственная опция сократить код. И это имеет смысл только, когда тот код повторяется по крайней мере в нескольких местах...

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

Я придерживался бы с тем, что у Вас есть tbh, но так же, как альтернатива, если у Вас есть LINQ к объектам, Вы могли

new string(this.Description.ToCharArray().Take(25).ToArray())

//And to maintain the ...
+ (this.Description.Length <= 25 ? String.Empty : "...")

Как другие сказали, Вы, вероятно, хотели бы сохранить 25 в константе

Я не могу думать ни о ком, но Ваш подход не мог бы быть лучшим. Вы добавляете логику представления в свой объект данных? Раз так затем я предполагаю, что Вы помещаете ту логику в другом месте, например, статический класс StringDisplayUtils с GetShortStringMethod (интервал maxCharsToDisplay, представьте stringToShorten в виде строки).

Однако тот подход не мог бы быть большим также. Что относительно различных шрифтов и наборов символов? Необходимо было бы начать измерять фактическую длину строки с точки зрения пикселей. Проверьте свойство AutoEllipsis на класс Маркировки winform (Вы будете, prob должен установить AutoSize на ложь при использовании этого). Свойство AutoEllipsis, когда верный, укоротит строку и добавит '...' символы для Вас.

Необходимо видеть, можно ли сослаться на Microsoft. VisualBasic DLL в Ваше приложение, таким образом, можно использовать "Левую" функцию.