dottrace и оптимизирующий метод с indexof - PullRequest
Новая
       29

dottrace и оптимизирующий метод с indexof

0 голосов
/ 26 января 2012 
public static string[] GetStringInBetween(string strBegin, string strEnd, string strSource, bool includeBegin, bool includeEnd)
        {
            string[] result = { "", "" };
            int iIndexOfBegin = strSource.IndexOf(strBegin);

            if (iIndexOfBegin != -1)
            {
                // include the Begin string if desired
                if (includeBegin)
                    iIndexOfBegin -= strBegin.Length;

                strSource = strSource.Substring(iIndexOfBegin + strBegin.Length);
                int iEnd = strSource.IndexOf(strEnd);

                if (iEnd != -1)
                {
                    // include the End string if desired
                    if (includeEnd)
                        iEnd += strEnd.Length;

                    result[0] = strSource.Substring(0, iEnd);

                    // advance beyond this segment
                    if (iEnd + strEnd.Length < strSource.Length)
                        result[1] = strSource.Substring(iEnd + strEnd.Length);
                }
            }

            return result;
        }

использование: 

string[] result = null;
result = HtmlHelper.GetStringInBetween(bits[0], bits[1], tagValuePair.Value, true, true);

Я использую dottrace, и этот метод использует 33% моего процессора.Как я могу оптимизировать это.Из-за этого происходит сбой моего приложения или не хватает памяти.Разумно ли, что этот метод статичен?

dottrace показывает 30% использования процессора на этом: 

System.String.IndexOf(String, Int32, Int32, StringComparison)

enter image description here

РЕДАКТИРОВАТЬ: 

    GetStringInBetween(string strBegin, string strEnd, string strSource, bool includeBegin, bool includeEnd)

strBegin = "<td class=\"m92_t_col2\">"
strEnd = "</td>"
strSource = "xxxxxxxx<td class=\"m92_t_col2\">Di. 31.01.12</td>xxxxxxxxxxxxxx
includeBegin = true
includeEnd = true

then i will get result
result[0] = "<td class=\"m92_t_col2\">Di. 31.01.12</td>"

надеюсь, это поможет тому, что делает этот метод.Попробуйте найти строку между strBegin и strEnd ...

Ответы [ 2 ]

1 голос
/ 26 января 2012

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

Кроме того, зная, что эти строки не локализованы, вы можете получить их, используя порядковый компаратор в IndexOf.

Что-то вроде 

public static string[] GetStringInBetween(string strBegin, string strEnd, string strSource, bool includeBegin, bool includeEnd)
{
    string[] result = { "", "" };
    int iIndexOfBegin = strSource.IndexOf(strBegin, StringComparison.Ordinal);

    if (iIndexOfBegin != -1)
    {
        int iEnd = strSource.IndexOf(strEnd, iIndexOfBegin, StringComparison.Ordinal);

        if (iEnd != -1)
        {
            result[0] = strSource.Substring(
                iIndexOfBegin + (includeBegin ? 0 : strBegin.Length), 
                iEnd + (includeEnd ? strEnd.Length : 0) - iIndexOfBegin);

            // advance beyond this segment
            if (iEnd + strEnd.Length < strSource.Length)
                result[1] = strSource.Substring(iEnd + strEnd.Length);
        }
    }

    return result;
}
0 голосов
/ 26 января 2012

Для вашего примера ввода, кажется, вы работаете с фрагментами HTML.

Я предлагаю использовать HTML Agility Pack для анализа HTML - он предоставляет результат в виде простого запроса, используя синтаксис LINQ to XML или XPath. Это быстрая и эффективная библиотека.

...