# V814. Decreased performance. The 'strlen' function was called multiple times inside the body of a loop.

Анализатор обнаружил конструкцию, которую потенциально можно оптимизировать. При каждой итерации цикла вызывается функция strlen(S) или аналогичная ей. Строка 'S' не меняется, и значит, её длину можно вычислить заранее. В некоторых случаях это может дать существенный прирост производительности.

Рассмотрим первый пример.

    for (;;) {
    {
      ....
      segment = next_segment + strlen("]]>");
      ....
    }

Внутри цикла много раз вычисляется длина строки "]]>". Конечно, строка короткая и функция strlen() Работает быстро. Но если цикл выполняет миллионы итераций, можно получить замедление "на ровном месте". Этот недочет легко исправить:

    const size_t suffixLen = strlen("]]>");
    for (;;) {
    {
      ....
      segment = next_segment + suffixLen;
      ....
    }

Ещё лучше использовать макрос, на подобии этого:

    #define LiteralStrLen(S) (sizeof(S) / sizeof(S[0]) - 1)
    ....
    segment = next_segment + LiteralStrLen("]]>");

Если это Си++, то можно сделать шаблонную функцию:

    template <typename T, size_t N>
    char (&ArraySizeHelper(T (&array)[N]))[N];

    template <typename T, size_t N>
    size_t LiteralStrLen(T (&array)[N]) {
      return sizeof(ArraySizeHelper(array)) - 1;
    }
    ....
    segment = next_segment + LiteralStrLen("]]>");

Рассмотрим второй пример.

    for(j=0; j<(int)lstrlen(text); j++)
    {
      if(text[j]=='\n')
      {
        lines++;
      }
    }

Показанный фрагмент кода считает количество строк в тексте. Этот пример не надуманный, а взят из реального приложения.

Если текст будет большим, этот алгоритм будет крайне неэффективен. На каждой итерации цикла, будет вычисляться его длина, чтобы сравнить с ней значение переменной 'j'.

Быстрый вариант:

    const int textLen = lstrlen(text);
    for(j=0; j<textLen; j++)
    {
      if(text[j]=='\n')
      {
        lines++;
      }
    }