CODEWITHSUNDEEP
arrayscpointersstruct
Denis Plescan
Denis Plescan

Reputation: 3

Sum elements of array structure with a pointer to that structure

I have a problem where I have this structure

typedef struct _COMPLEX {
    double real, imag;
}COMPLEX;

and I need to declare an array of this structure and sum the elements of the array. Here is my code.

#include <stdio.h>
#include <stdlib.h>
void* xmalloc(size_t nrOcteti);

int main()
{
    COMPLEX* v = 0, *s = 0;
    int n, i;
    printf("\n n = ");
    scanf("%d", &n);
    v = (COMPLEX*)xmalloc((n)*sizeof(COMPLEX));
    s = (COMPLEX*)xmalloc(sizeof(COMPLEX));
    for(i = 0; i < n; i++)
    {
        printf("\n v[%d].real and imag:", i);
        scanf("%lf %lf", &v[i].real, &v[i].imag);
    } 
    for(i = 0; i < n; i++)
        printf("V%d after scan=%.2lf + %.2lf * i\n", i, v[i].real, v[i].imag);
    s->real = 0; s->imag = 0;
    for(i = 0; i < n; i++)
    {
        s->real = s->real + v[i].real;
        s->imag = s->imag + v[i].imag;
    }
    for(i = 0; i < n; i++)
        printf("V%d after sum=%.2lf + %.2lf * i\n", i, v[i].real, v[i].imag);
    printf("\nS=%lf + %lf\n", s->real, s->imag);
    if(s) free(s);
    s = 0;
    if(v) free(v);
    v = 0;
    return 0;
}
void* xmalloc(size_t nrOcteti)
{
    void *p = 0;
    p = malloc(sizeof(nrOcteti));
    if(!p)
    {
        fprintf(stderr, "Allocation failed!");
        exit(EXIT_FAILURE);
    }
    return p;
}

After I give the elements of the array I print them everything is alright, but just before the sum the elements of the array are changed(and for multiple test, apparently the element with the index 2 is modified) and the sum at the end is incorrect. And sometimes(I think because of different inputs) it gives me at the end Segmentation fault because of the free(). The array and sum needs to be pointers to that struct and need to be dynamically allocated. I tried many times and I can't manage to make it work properly. If somebody can help me solve this it will be a blessing =))

Upvotes: 0

Views: 162

Answers (1)

MikeCAT
MikeCAT

Reputation: 75062

The line

    p = malloc(sizeof(nrOcteti));

in the function xmalloc() is wrong. This line is ignoring what is passed as the argument and just allocating for one size_t.

It should be

    p = malloc(nrOcteti);

to allocate specified size.

Upvotes: 1

Related Questions