Thread: cannot initialize structure variables

Hi,

i have a problem that is rather basic but since i am not a frequent developper this presents a difficulty to me that i do not understand. So what i have is :

Code:

#include <stdio.h>
#include "w.h"


void main(){

WheelCnt *wheel;

wheel->x =0;
wheel->y = 0;

printf("%d\n",wheel->y);

}

and w.h that looks like this:

Code:

#define MAXWSIZE 200



typedef struct WheelCnt{   
	int q[MAXWSIZE];
	int x;
	int y ;
}WheelCnt;

the error i get when i run the program is :

Segmentation fault (core dumped)

and this is because i am trying to do this:

Code:

wheel->x =0;
wheel->y = 0;

Can someone help


thank you !!!

wheel is just a pointer to a WheelCnt structure, there is no actual structure. The pointer has a garbage value, and you are accessing memory you don't own. That results in undefined behavior, which often manifests as a seg fault.

You need to declare it as a struct and use the . (dot) operator:

Code:

WheelCnt wheel;
wheel.x = 0;

or malloc space for it and use the -> operator:

Code:

WheelCnt *wheel;
wheel = malloc(sizeof(*wheel));
wheel->x = 0;

Warning!

Code:

void main(){  }

That hurts.If i was the compiler i guess i would cry.
The right thing to write is this

Code:

int main(void)
{
    ..
    return 0;
}

Also about the malloc do not forget to include stdlib.h (malloc is written there).And i would suggest you to write (if trying to using the malloc function)

Code:

WheelCnt *wheel;
wheel = malloc(sizeof(WheelCnt ));
wheel->x = 0;

because i think that it is more clear,as when you define a struct ,it's like you have your own type of data (like the language has it's primitive types by default int,char,etc).When you malloc memory you use the sizeof(int) -in a more complex statement usually-,so this is why i suggest it.

The advantage (in my opinion) to using sizeof object rather than sizeof (type) is that you have one less place to change if you decide to use another type later on: the compiler knows the type of the object.

Code:

int *x;
/* ... */
x = malloc(100 * sizeof *x); // if the type changes, this line can be left alone

Code:

double *x;
/* ... */
x = malloc(100 * sizeof *x); // if the type changes, this line can be left alone

Compare with

Code:

int *y;
/* ... */
y = malloc(100 * sizeof (int)); // if the type changes, this line must be changed too

Code:

double *y;
/* ... */
y = malloc(100 * sizeof (double)); // if the type changes, this line must be changed too

Originally Posted by std10093

Warning!

Code:

void main(){  }

That hurts.If i was the compiler i guess i would cry.

It is possible that the OP is writing code for an embedded device, in which case "void main()" is correct. I suspect this might be the case due to the variable names - "WheelCnt" might stand for "Wheel Control," and the int members 'x' and 'y' sound like coordinates - but I might very well be wrong.

> That hurts.If i was the compiler i guess i would cry.
hahahaha xD

But you should seriously check out this link.

Wonderful things in this post.Bravo to all!