Number of call occurrences exceeds threshold

Количество вызовов функции в функции больше заданного порога вхождения вызова функции

Описание

Этот дефект возникает, когда тело функции содержит больше вызовов функций, чем заданный порог вхождения вызова функции. Для получения дополнительной информации о том, как Polyspace^® вычисляет количество вхождений вызова функции, см. Number of Call Occurrences

Polyspace использует порог 50 по умолчанию, если вы не задаете порог. Чтобы задать файл выбора, в котором можно задать порог, используйте Set checkers by file (-checkers-selection-file). Также см. «Уменьшение сложности программного обеспечения при помощи Polyspace Checkers».

Когда вы импортируете комментарии из предыдущих анализов при помощи polyspace-comments-importPolyspace копирует любую рецензирующую информацию о метрике кода Number of Call Occurrences в предыдущем результате этой проверки в текущем результате. Если текущий результат содержит ту же метрику кода, информация проверки также копируется в метрику кода.

Риск

Нарушение этой проверки может указать, что:

Функция выполняет слишком много задач.
Функция может иметь высокую взаимозависимость с несколькими другими функциями.
Изменения в некоторой другой функции могут оказать неожиданное влияние на помеченную функцию.

Эти факторы затрудняют поддержание и отладку функции.

Зафиксировать

Чтобы исправить эту проверку, либо измените код, либо измените порог проверки. При рефакторинге кода спроектируйте функции в вашем коде так, чтобы:

Каждая функция выполняет одну конкретную задачу.
Функции имеют минимальные побочные эффекты на другие функции.

Лучшая практика состоит в том, чтобы проверить сложность модуля на ранних стадиях разработки, чтобы избежать дорогостоящего рефакторинга после разработки.

Примеры

расширить все

Избегайте функций, которые имеют слишком много вызовов функций

 long long power(double x, int n){
	 long long BN = 1;
	 for(int i = 0; i<n;++i){
		 BN*=x;
	 }
	 return BN;
 }
long long factorial(int n){
	 long long BN = 1;
	 for(int i = 1; i<=n;++i){
		 BN*=i;
	 }
	 return BN;
 }
 
 double AppxIndex(double m, double f){//Noncompliant
	 double U = (power(m,2) - 1)/(power(m,2)*factorial(2));
	 double V = (power(m,4) + 27*power(m,2)+38)/factorial(3)*(2*power(m,2)+3);
	 return (1+2*f*power(U,2)*(1+power(m,2)*factorial(static_cast<int>(U/V))
       + power(m,3)/power(m,3)*(U-V)))/( (1-2*f*power(U,2)*(1+power(m,2)*
        factorial(static_cast<int>(U/V)) + power(m,3)/power(m,3)*(U-V)))); }

Функция AppxIndex содержит 17 различных вызовов функций, что больше заданного порога вхождения вызовов, равного 10. Такое высокое вхождение указывает, что эта функция выполняет слишком много задач. Это также указывает, что функция имеет высокие взаимозависимости с другими функциями. Эти факторы делают AppIndex трудно поддерживать, тестировать или отладка.

Коррекция - Код рефактора

Одной из возможных коррекций является рефактор функции. Здесь функция разделена на меньшие функции, которые выполняют определенные задачи.

  long long power(double x, int n){//Compliant
	long long BN = 1;
	for(int i = 0; i<n;++i){
		BN*=x;
	}
	return BN;
}
long long factorial(int n){//Compliant
	long long BN = 1;
	for(int i = 1; i<=n;++i){
		BN*=i;
	}
	return BN;
}

double CalculateU(double m){//Compliant
	return (power(m,2) - 1)/(power(m,2)*factorial(2));
}

double CalculateV(double m){//Compliant
	return (power(m,4) + 27*power(m,2)+38)/factorial(3)*(2*power(m,2)+3);
}

double CalculateMid(double m, double f){//Compliant
	double U = CalculateU(m);
	double V = CalculateU(m); 
	return (2*f*power(U,2)*(1+power(m,2)*factorial(static_cast<int>(U/V))
             + power(m,3)/power(m,3)*(U-V)));
}

double AppxIndex(double m, double f){//Compliant
	
	return (1+CalculateMid(m,f))/( (1-CalculateMid(m,f)));
}

Проверяйте информацию

Группа: Комплексность программного обеспечения

Язык: C | C++

Акроним: SC06

Порог по умолчанию: 50

См. также

Check Guidelines (-guidelines)

Темы

Введенный в R2021a

Документация