Ist das eher etwas, das man vermeiden sollte, oder gängige Praxis?

GetModuleHandle

ohje...nich gut...

ich verweise hier mal auf Martin Richters Blog:

http://blog.m-ri.de/index.php/2007/12/12/die-unsitte-immer-getmodulehandlenull-fuer-hinstance-in-createwindow-und-registerclass-zu-verwenden/

dieser aufschlussreicher eintrag sollte denkanstöße geben

es gibt wie immer bei der WinAPI (oder überhaupt) mehrere möglichkeiten, etwas zu tun. so auch mit dem Wert von hInstance.
so schlimm ist in diesem fall eine globale variable nicht.
du kannst sie allerdings auch jeder funktion, die sie brauch, übergeben, so sparst du dir auch die globale variable. oder du vermeidest den direkten zugriff auf die globale variable über eine funtion:

// WARNUNG: ohne jegliche fehlerüberprüfung

// data.h
void InitInstance(HINSTANCE hInst);
HINSTANCE MyGetInstance(void);

// data.c
HINSTANCE g_hInst = NULL;

void InitInstance(HINSTANCE hInst) {
    g_hInst = hInst;
}

HINSTANCE MyGetInstance(void) {
    return g_hInst;
}

InitInstance rufst du dann z.b. in WinMain auf ... vorteil dabei -> egal ob du code für eine .exe oder .dll schreibst, oder code für eine .exe später in eine .dll auslagst, so du musst dich um keine globale variable kümmern, du musst nicht alle, aufrufe von GetModuleHandle(NULL), wie rofler erwähnte, ersetzen in der .dll. kurz: du hast eine einheitliche API.

grüße, xantus

Ist das eher etwas, das man vermeiden sollte, oder gängige Praxis?

Hi Bruder !
Wenn du bedenken hast, das etwas von ausserhalb an deiner globalen Variable knabbern könnte, dann mach die doch einfach static.