Struct Array Crashing Program C

Question

The program runs but every time I do it just ends up crashing it after. I also get "Segmation Fault" Error when running a debug

         enemy mob[2];

         for(i=0; i<2; i++){

            mob->mobAttack[i] = 25;
            mob->mobHealth[i] = 150;
            mob->mobName[i] = "Growler";

        }

Structure

typedef struct{

char * mobName;
int * mobHealth;
int * mobAttack;

} enemy;

You don't know how to allocate memory? Time to log off Stackoverflow and use your time to read a basic C book or tutorial. — kaylum, Apr 26 '17 at 23:07
You could change your struct definition so you don't need to allocate any more memory. — M.M, Apr 26 '17 at 23:07
@Syntax You say C, but is it really C or is it C++ compiler and you use just C part? — ipavlu, Apr 26 '17 at 23:08
I think you wanted it to be `mob[i]->mobAttack = 25;` And I also wouldn't use pointers for mobHealth and mobAttack. — ThunderStorm, Apr 26 '17 at 23:08
@Syntax For C please look on method malloc() and free() in libraries... — ipavlu, Apr 26 '17 at 23:09
Did you intend for 1 mob to have multiple health and attack values? — M.M, Apr 26 '17 at 23:09

pm100 · Answer 1 · 2017-04-26T23:09:27.333

1

you need

    typedef struct{

    char * mobName;
    int  mobHealth;
    int  mobAttack;
} enemy

and

  mob[i].mobAttack = 25;

etc. I assume the compiler was complaining big time at you , dont ignore warnings

edited Apr 26 '17 at 23:09

answered Apr 26 '17 at 23:08

pm100

48,078
23
82
145

In original request it is pointer, but pm100 realized that it is unnecessary and he is right:). And provided correct code:). – ipavlu Apr 26 '17 at 23:10
Wow it worked. Thanks man! – DeadAccount Apr 26 '17 at 23:13

Struct Array Crashing Program C

1 Answers1