В чем ошибка при вычислении суммы последовательности чисел?

Question

[raycheel]

Решаю задачу, необходимо определить сумму чисел не пустой последовательности за которой следует 0. Посчитать среднее арифметическое и геометрическое значения. Завис на сумме чисел, консоль показывает не верные результаты подсчета, например, ввожу 2 2 0, ответ компилятора: 11271656 .
Благодарен за любую помощь!

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

void main()
{
	int x;
	float sum = 0, prois = 1, sumg, suma, n = 0;

	do
	{
		scanf("%f", &x);
		n = n++;
		sum = sum + x;
		prois = prois*x;
		suma = sum / n;
		sumg = pow(prois, 1 / n);
	} while (x != 0);

	printf("Suma %d", &sum);
	getch();
}

Answer 1

[jcmvbkbc]

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

void main()
{
	int n = 0;
	float sum = 0, prois = 1, sumg, suma, x;

	for (;;) {
		scanf("%f", &x);
		if (x == 0)
			break;
		n++;
		sum = sum + x;
		prois = prois*x;
	}
	if (n > 0) {
		suma = sum / n;
		sumg = pow(prois, 1. / n);
	}
	printf("Suma %f", sum);
	getch();
}

Comments

[raycheel]

Спасибо! Почему у меня не правильно считается средняя геометрическая? Постоянно пишет 1 и все... Спасибо еще раз!

[jcmvbkbc]

Поправил код ещё раз.

[jcmvbkbc]

...и ещё раз.

[raycheel]

@jcmvbkbc Один маленький вопрос, что означает for(;;) - не стандартная запись какая-то. Спасибо!

[jcmvbkbc]

что ж тут нестандартного, бесконечный цикл

Answer 2

[OvLab]

Судя по тому, что "int x;" должно быть "scanf("%d", &x);". Произведение будет всегда равно нулю, так как на него будет сначала умножаться, а уже потом выход из цикла.

Answer 3

[raycheel]

@jcmvbkbc
Благодарю за внимание! Я пытаюсь вывести значение средней геометрической, но она в любом случае равна 1-це. Никак не могу разобраться.

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

void main()
{
	int n = 0;
	float sum = 0, prois = 1, sumg, suma, x;

	for (;;) {
		scanf("%f", &x);
		if (x == 0)
			break;
		n++;
		sum = sum + x;
		prois = prois*x;
	}
	if (n > 0) {
		suma = sum / n;
		float step = 1 / n;
		sumg = pow(prois, step);
	}
	printf("Suma %f", sum);
	printf("Cped. arifmetich %f \n ", suma);
	printf("Cped. geometri4eskaya %f", sumg);
	getch();
}

Comments

[aschep]

у тебя n - int, и 1 - тоже int. поэтому step всегда 0. поставь хотя бы точку после 1: float step = 1. / n;

[aschep]

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