ошибка error C2043: illegal break

[Ongi]

Всем доброго дня.
При компиляции программы, в подпрограмме появляется ошибка error C2043: illegal break. Подпрограмма должна найти среднее арифметическое положительных элементов с четными номерами
Вот код подпрограммы:

double first (int*mass,int x) //положительные элементы с четным номерами 
{
	int i,l=0,sum=0;
	double sa;
	printf("введите первый массив\n");
	for(i=0;i<x;i++)
	{
		printf("mass[%d]=",i);
		scanf("%d",&mass[i]);
	}
	for(i=0;i<x;i++)
		if(mass[i]>0 && i%2==0)
		{
			sum=sum+mass[i];
			l++;
		}
	if (sum==0)
	{
		printf ("подходящих элементов нет\n");
		break;
	}
	sa=sum/l;
	printf("среднее арифметическое \n");
	printf ("sa=%lf\n",sa);
	return sa;
}

[ROD]

а что break делает в теле if`а ??? он ставится только в циклах (и то, далеко не всегда он там нужен) и в switch`ах.

[Ongi]

Если включить if, где находится break, в цикл, то программа вылетает с ошибкой (приложение будет закрыто)...
Можно ли как-то по-другому остановить работу подпрограммы? Сразу после нее должна включиться другая подпрограмма.

[ROD]

если тебе надо остановить работу подпрограммы, то надо писать return; (если она ничего не возвращает (типа void)) или return 0; (или любое другое значение или переменная) в том месте, где подпрограмма должна закончить свою работу.

К тому же у вас во втором цикле for очевидная ошибка - если тело цикла не поммещено в фигурные скобки, то телом считается следующая за циклом строка


Рекомендую это

for(i=0;i<x;i++)
		if(mass[i]>0 && i%2==0)
		{
			sum=sum+mass[i];
			l++;
		}

заменить на это

for(i=0;i<x;i++)
{
		if(mass[i]>0 && i%2==0)
		{
			sum=sum+mass[i];
			l++;
		}
}

[Ongi]

ROD, спасибо вам большое за помощь.
По поводу return: загвоздка была в том, что в одном случае (если не попадала на второй if) подпрограмма должна была возвращать sa, а в другом - нет.

Решение оказалось простым )) надо было изменить условие второго if на противоположное.

double first (int*mass,int x) 
{
	int i,l=0;
	double sa;
	int sum=0;
	printf("Введите первый массив\n");
	for(i=0;i<x;i++)
	{
		printf("mass[%d]=",i);
		scanf("%d",&mass[i]);
	}
	for(i=0;i<x;i++)
	{
		if((mass[i]>0) && (i%2==0))
		{
			sum=sum+mass[i];
			l++;
		}
	}
	if (sum!=0) //было if (sum==0)
	{
	sa=sum/l;
	printf("среднее арифметическое\n");
	printf ("sa=%lf\n",sa);
	}
	else printf ("положительных элементов нет\n");
	return sa;
}

Теперь все работает как надо

[krugolet]

Здравствуйте!
Дабы не плодить лишних тем, задам свой вопрос здесь, ведь тема та же.

Вычисляю сумму ряда.
код:

#include<stdio.h>
#include<math.h>

int main()
{
	const int maxIter = 50;
	double x, eps;
	int done = 1, n;
	double ch =1, y = ch;
	printf("Input argument and precision:");
	scanf("%lf %lf", &x, &eps);
	printf("arg=%lf  eps=%lf", x, eps);

	for(n = 0; fabs(ch)>eps; n++);
	{
		ch *= x*x/((2*n+1)*(2*n+2));
		y += ch;
		if(n>maxIter) 
		{
			printf("\nRyad rashoditsya!!!\n");
			done = 0; 
			break;
		}	
	}
	if(done)
	{
		printf("The function value y = %lf for x = %lf\n", y, x);
		printf("have been calculated after %d iterations\n", n);
	}
	return 0;
}

компилятор ругается ошибкой error C2043: illegal break

break используется для аварийного выхода из цикла при превышении определенного количества итераций.

[_Bers]

Сообщение от ROD

К тому же у вас во втором цикле for очевидная ошибка - если тело цикла не поммещено в фигурные скобки, то телом считается следующая за циклом строка

Вы ошибаетесь. Ниже представленные варианты кода делают абсолютно одинаковые действия:

for(i=0;i<x;i++) if(mass[i]>0 && i%2==0) { sum=sum+mass[i]; l++; }

for(i=0;i<x;i++)
{
		if(mass[i]>0 && i%2==0) { sum=sum+mass[i]; l++; }
}

Телом цикла может быть либо блок, либо действие. Причем, блок - это кучка действий, которые компилятор будет воспринимать, как одно большое действие. Поэтому можно сказать, что туловищем цикла всегда является действие (блочное, или обычное).

В первом случае компилятор посчитает телом цикла только одно действие:

for(i=0;i<x;i++)  if(mass[i]>0 && i%2==0) { sum=sum+mass[i]; l++; } //условие, со всем своим блоком -это одно простое не_блочное действие

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

Конечно, выделять туловище цикла фигурными скобками - предпочтительнее, ибо лучше читается, проще понять, и меньше вероятность допустить ошибку. Лично я всегда заключаю туловище цикла в фигурные скобки (даже если оно в одно простое действие).

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

[_Bers]

Сообщение от krugolet

Здравствуйте!
Дабы не плодить лишних тем, задам свой вопрос здесь, ведь тема та же.

Вычисляю сумму ряда.
код:

#include<stdio.h>
#include<math.h>

int main()
{
	const int maxIter = 50;
	double x, eps;
	int done = 1, n;
	double ch =1, y = ch;
	printf("Input argument and precision:");
	scanf("%lf %lf", &x, &eps);
	printf("arg=%lf  eps=%lf", x, eps);

	for(n = 0; fabs(ch)>eps; n++); //<---- очень внимательно смотрим сюда
	{
		ch *= x*x/((2*n+1)*(2*n+2));
		y += ch;
		if(n>maxIter) 
		{
			printf("\nRyad rashoditsya!!!\n");
			done = 0; 
			break;
		}	
	}
	if(done)
	{
		printf("The function value y = %lf for x = %lf\n", y, x);
		printf("have been calculated after %d iterations\n", n);
	}
	return 0;
}

компилятор ругается ошибкой error C2043: illegal break

break используется для аварийного выхода из цикла при превышении определенного количества итераций.

1. См комментарий в коде.
2. Перечитываем сообщение написанное мною чуть выше.
3. Задаём себе вопрос: что означает символ ';' для компилятора? Зачем этот символ стоит после объявления цикла, и что подумает компилятор, когда увидит этот символ?