Help....Trying to use extern / static to no avail....

I have four global variables defined before main in file1.c

Code:

---

boolean moreThanOneSwitch = F, moreThanOneSwitchTwoSec = F;unsigned int switchHits[12] = {0}, button = 0;

---

I declare these variables 'extern' in file1.h

Code:

---

extern unsigned int switchHits[12], button;extern boolean moreThanOneSwitchTwoSec, moreThanOneSwitch;

---

I place them into a struct in file1.h

Code:

---

struct swParams {    unsigned int switchHits[12];
    unsigned int button;
    boolean moreThanOneSwitch;
    boolean moreThanOneSwitchTwoSec;
};

---

file1.h also contains a declaration to a function that returns that struct so that main can act upon the results.

Code:

---

struct swParam swMath(unsigned int *p);

---

Within main I call the function which happens to exists in file2.c
This function gets called on a timer and modifies the variables
After returning from the function in main I look at these variables and decide whether or not (through logic) on how to proceed. At different points within the logic I may or may not clear the variables. (I do not want to clear them within the function)....Can someone please point me in the right direction on how to do this?? I'm very lost here.....Basically I want to initialize variables in main, call a function periodically that can modify these variables, use them in main and allow main to also possibly modify them....If I place everything in main all works fine

Thanks

Quote:

---

Originally Posted by ridgerunnersjw

I place them into a struct in file1.h

---

No, you didn't. You declared a struct that happens to have four members of the same names and types as those four global variables, but which have nothing to do with the global variables.

Since it is clear that you are insistent on having global variables even though I am not convinced that they are necessary for the problem that you're trying to solve, I'll just suggest this:

In file1.h:

Code:

---

struct swParams {
    unsigned int switchHits[12];
    unsigned int button;
    boolean moreThanOneSwitch;
    boolean moreThanOneSwitchTwoSec;
};

extern struct swParams switchParams;

void swMath(unsigned int *p);

---

Notice that I declared swMath as returning void. You do not want to return a struct swParams object because the one that you're working with is global. But honestly, I still don't see why you need a global variable. You can declare swMath like this instead:

Code:

---

void swMath(struct swParams *switchParams, unsigned int *p);

---

then in the main function:

Code:

---

struct swParams switchParams;
swMath(&switchParams, x);

---

Anyway, if you insist on a global variable, then in file1.c:

Code:

---

#include "file1.h"

struct swParams switchParams = {{0}, 0, F, F};

---

Thank you! I look forward tomorrow giving your stuff a try.....It's complicated from this end because this is embedded code on a microprocessor so hardware ISRs are constantly going off and both main and the function have the opportunity to change the states of the booleans....It all depends on user input from switches....I am constantly scanning the switches and running the function every 2 seconds or so and then in main I act upon these switches and transmit RF data....depending on the switches thrown main might make changes to the boolean states...I cannot clear them within the function .....

I just want to say Thank you! The code examples you sent work great.....