Как инициализировать массив?

Question

[artgor]

Задание следующее: имеется три массива, два из них уже инициализированы, третий нужно инициализировать следующим образом: каждый элемент массива должен равняться сумме соответствующих по индексу элементов двух инициализированных массивов.

Вот мой код:

void func(double * arr1, double * arr2, double * arr3, int len)
{
    int index;

    for (index = 0; index < len; index++)
        *(arr3 + index) = *(arr1 + index) + *(arr2 + index);
}

Я инициализировал два массива одинаковыми значениями: 1, 2, 3. Следовательно, третий массив должен иметь следующие значения: 2, 4, 6. Но при выводе содержимого массива вывод выглядел по другому: 2, 3, 5. Что происходит не так?

Comments

[Александр Ананьев]

В этом фрагменте все вглядит правильно. Ищите ошибку дальше по коду.

Answer 1

[Василий Дёмин]

Приведите весь код. Ваша функция правильно складывает числа, скорее всего ошибка в другой части программы. Этот код выводит на экран "arr3 = {5.0, 7.0, 9.0}":

#include <stdio.h>

void func(double *arr1, double *arr2, double *arr3, int len) {
    for (int index = 0; index < len; index++)
        *(arr3 + index) = *(arr1 + index) + *(arr2 + index);
}

int main() {
    double arr1[3] = {1.0, 2.0, 3.0};
    double arr2[3] = {4.0, 5.0, 6.0};
    double arr3[3];
    
    func(arr1, arr2, arr3, 3);
    printf("arr3 = {%f, %f, %f}", arr3[0], arr3[1], arr3[2]);
}

Comments

[artgor]

void init_array(double * array, int len);
void func(double * arr1, double * arr2, double * arr3, int len);
void print_array(double * array, int len);
int main(void)
{
    int len;
    double element;
    double element2;
    double element3;
    double * arr1 = &element;
    double * arr2 = &element2;
    double * arr3 = &element3;

    printf("\nPrint array's length: ");
    scanf("%d", &len);
    init_array(arr1, len);
    init_array(arr2, len);
    func(arr1, arr2, arr3, len);
    print_array(arr3, len);
}

void init_array(double * array, int len)
{
    int index;

    printf("\nInitialize the array:\n");
    for (index = 0; index < len; index++)
        scanf("%lf", array + index);
}

void func(double * arr1, double * arr2, double * arr3, int len)
{
    int index;

    for (index = 0; index < len; index++)
        *(arr3 + index) = *(arr1 + index) + *(arr2 + index);
}

void print_array(double * array, int len)
{
    int index;

    for (index = 0; index < len; index++)
        printf(" %lf", *(array + index));
}

[Василий Дёмин]

artgor261, Нельзя так делать, потому что ваши указатели ссылаются на область памяти, выделенную под одно значение double.

double element;
    double element2;
    double element3;
    double * arr1 = &element;
    double * arr2 = &element2;
    double * arr3 = &element3;
    init_array(arr1, len);
    init_array(arr2, len);

Чтобы выделить память под несколько значений типа double, используйте malloc:

double *arr1 = malloc(sizeof(double) * len);
double *arr2 = malloc(sizeof(double) * len);
double *arr3 = malloc(sizeof(double) * len);

И ещё небольшое стилистическое замечание. Вместо

*(arr3 + index) = *(arr1 + index) + *(arr2 + index);

Можно писать:
arr3[index] = arr1[index] + arr2[index];

[Василий Дёмин]

artgor261, И в циклах for можно объявлять переменную, необязательно делать это до цикла:

for(int index = 0; index < len; index++)
    arr3[index] = arr1[index] + arr2[index];

[artgor]

Василий Дёмин, я пишу массивы таким образом, чтобы лучше понимать, как всё работает изнутри

[skyazimuth]

Василий Дёмин, и не забыть в конце для каждого массива free() вызвать. По уму, не мешало бы ещё проверять что возвращает malloc()
Ну и в конце main() return(0);

[Василий Дёмин]

skyazimuth, return 0 можно в конце main не писать - стандарт позволяет. Про free забыл написать, но в данном случае это не обязательно, так как после завершения программы ОС сама освободит всю память процесса.