Не могу понять в чем ошибка(задача на С)?

Question

[gleendo]

В чем ошибка? указывает на
sum = sum + arr[i];
product = product * arr[i];
double arr[10] = {1, 2, 3, 4, 5, 6, 7, 8, 9};

void array(arr) {
  int i;
double sum = 0, product = 1;
for (i = 0; i < strlen(arr); i++) {
sum = sum + arr[i];
product = product * arr[i];
}
printf("%f", sum);
printf("%f", product);
}


double arr[10] = {1, 2, 3, 4, 5, 6, 7, 8, 9};
array(arr);

Answer 1

[Евгений]

1. укажите корректный тип параметра для функции, например void array(double* arr)
2. strlen - это функция для работы со строками и не подходит для double массивов, поэтому нужно добавить второй параметр в вызов функции - размер передаваемого массива.

void array(double *arr, int size) {
  int i;
double sum = 0, product = 1;
for (i = 0; i < size; i++) {
sum = sum + arr[i];
product = product * arr[i];
}
printf("%f", sum);
printf("%f", product);
}

int main() {
double arr[10] = {1, 2, 3, 4, 5, 6, 7, 8, 9};
array(arr, sizeof(arr)/sizeof(arr[0]));
return 0;
}

P.S отредактировал ответ с учетом комментариев

Comments

[gleendo]

Евгений Указал, теперь осталась одна из 3-х ошибок. Указывает на double arr[10] = {1, 2, 3, 4, 5, 6, 7, 8, 9};
Пишет Declaration is not allowed here

Что нужно сделать?

[jcmvbkbc]

> array(arr, sizeof(arr));
вы передали размер массива в байтах, а ожидали количество элементов

[gleendo]

jcmvbkbc: И как порешать? помогите пожалуйта

[jcmvbkbc]

> И как порешать?
идиоматичное решение -- array(arr, sizeof(arr) / sizeof(*arr));

[gleendo]

jcmvbkbc: Ничего не помогает. До сих пор пишет что Declaration is not allowed here (на это строчке double arr[10] = {1, 2, 3, 4, 5, 6, 7, 8, 9};)

[jcmvbkbc]

con_ написанное мною не относится к этой ошибке компиляции.
Весь код ниже функции array нужно поместить в функцию, чтобы он вообще скомпилировался.

Answer 2

[Армянское Радио]

В функции у вас arr типа int (если тип не указан, C считает, что это - int),
вы пытаетесь применить к int strlen, и, естественно, получаете ругань.

Правильно написать void array(double *arr,const size_t count) и передать в функцию еще и количество элементов в массиве.

Дальше вы передаете в функцию указатель на массив double, что тоже не здорово.