Как исправить ошибку в коде на C++?

Question

[LP-DIMAN]

#include
#include
#include

using namespace std;
int main() {
setlocale(LC_ALL, "Russian");
const int n = 5;
int a[n];
cout << " Введите " << n << " целых чисел: " << endl;
for (int i = 0; i < n; i++) cin >> a[i];
for (int i = 0; i < n; i++) cout << a[i] << " ";
int imin = 0;
for (int i = imin = 0; i < n; i++) {
if (a[i] < a[imin]) imin = i;

}
cout << "\n\ Минимальный элемент массива = " << a[imin];
cout << endl;

double C[n], p = 1.0;
int i, k = 0;

srand(time(0));
for (i = 0; i < n; i++)
{
C[i] = (double)(rand() % 50) / 10;
if (C[i] >0)
{
p *= C[i];
++k;
}
}

cout << "Среднее геометрическое положительных элементов массива: ";
if (k > 0){
cout << pow(p, 1.0 / k);
cout << endl;
}
else{
cout << k;
cout << endl;
return 0;
}
cout << endl << "Обратный порядок: " << endl;;
for (int i = n - 1; i >= 0; i--)
{

cout << a[i] << endl;

}
}
Неправильно считает среднее геометрическое положительных чисел, точнее выводит рандомно. Если убираю строку, то пишет ноль . Как вывести точное значение?

Answer 1

[maaGames]

Я правильно понимаю, что ты сперва заполняешь массив С рандомными числами, а потом удивляешься, что у тебя рандомный результат?

И проверка на положительность не нужна, rand возвращает числа в диапазоне [0:6535]

Comments

[Alexander Movchan]

Проверка нужна, так как на ноль умножать не нужно. По поводу остального согласен.

[maaGames]

Александр: Век живи - век учись. Только сейчас дошло, что на ноль умножать и правда не следует.)

[Владислав Ярославлев]

rand() возвращает числа в диапазоне [0; RAND_MAX], в Visual C++ это [0; 32767]

Answer 2

[Alexander Movchan]

Эмм... Что это?

srand(time(0));
for (i = 0; i < n; i++)
{
C[i] = (double)(rand() % 50) / 10;
if (C[i] >0)
{
p *= C[i];
++k;
}
}

P.S. Используйте тэг <code lang="cpp"></code>