Ich beschäftige mich zurzeit wieder etwas mit C++ und nun habe ich eine Optimierungsfrage, für einen besseren Code.

Es handelt sich um eine Bewegungssteuerung. Der Benutzer legt fest, dass er das Objekt mit den Tasten WSAD speichert.

Nun speichere ich in forward_key, left_key, right_key, backward_key (char) die Tasten ab.

Nun habe ich noch eine Eventschnittstelle, die die Tasteneingabe erkennt. Es wird geschickt ob die Taste losgelassen oder gedrückt wurde und die Taste (als char) wird übergeben.

Zurzeit sieht es so aus:

void KeyboardEvent(bool status, char key)
{
 if(key==forward_key){forward=status}
 if(key==left_key){forward=status}
 if(key==right_key){forward=status}
 if(key==backward_key){forward=status}
}

Da es nicht nur 4 Tasten zur Steuerung gibt, sondern eine Menge mehr, würde ich das gerne in ein Array packen, aber ich weiß nicht wie ich das handhaben soll... jemand eine Idee?

Also in der Art wie:

void KeyboardEvent(bool status, char key)
{
//controller ist das Array
controller[key]=status;
}

Jedoch muss der char Key für forward, left usw... variabel sein.

Sollte es jemand noch nicht ganz klar sein, was ich meine, versuch ich es noch einmal genauer zu beschreiben.

namespace Key
{
enum Key
{
  Q,W,E,R,T,Z,U,...
};
}

void KeyboardEvent(bool status, Key::Key key)
{
  if (key == Key::Q)
  if (key == Key::W)
  ..
}

//...
std::map<char, bool> your_map;
//...
//dort packst du deine elemente rein und kannst das so drauf zu greifen
void KeyboardEvent(bool status, char key)
{
    your_map[key] = status;
    //als darstellung, würde das z.b. so aussehen   your_map['c'] = true;
}

Warum nicht ein Array?

Gute Frage. Letzter Versuch:

namespace Key
{
enum Key
{
  Q,W,E,R,T,Z,U,..., KEY_COUNT
};
}
bool key_stats[Key::KEY_COUNT] = {0}; // = {false} ?

void KeyboardEvent(bool status, Key::Key key)
{
  // Wert von key prüfen?
  key_stats[key] = status;
}

Jetzt alle zufrieden?

Jetzt alle zufrieden?

Nö, warum ein enum?

cooky451 schrieb:

Jetzt alle zufrieden?

Nö, warum ein enum?

Was würdest du denn nehmen wollen? char? Und was machst du dann, wenn Ctrl gedrückt wird? (Oder Shift, Enter, etc.)

Also Zahl (enum) wäre nicht so gut, da ich diese Key-Codes bekomme: http://msdn.microsoft.com/en-us/library/dd375731(v=VS.85).aspx

Also Zahl (enum) wäre nicht so gut, da ich diese Key-Codes bekomme: http://msdn.microsoft.com/en-us/library/dd375731(v=VS.85).aspx

Also wäre die Arraygröße 256? Geht doch noch.

array[keycode]

Da brauchst Du nichts umformen.

besser als ein bool array[256] und dann char als Index zu missbrauchen

Dem Link zufolge, der oben gepostet wurde, scheints eher Richtung unsigned char zu gehen, zumal Werte von 0x01-0xFE angegeben werden. Das ist dann schon wieder ne andere Situation.

__declspec(dllexport) LRESULT CALLBACK Test::KeyboardProc ( int nCode, WPARAM wParam, LPARAM lParam)
{
 if(buffer[0]=='C' && buffer[2]=='n' && buffer[5]=='a' && nCode == HC_ACTION)
 {
  KBDLLHOOKSTRUCT hookstruct = *((KBDLLHOOKSTRUCT*)lParam);
  // Find the letter of the typed key
  char lettre=(char)hookstruct.vkCode;
  ...
 }
 ...
}

vkCode ist ja ein DWord...
Edit: Okay, ich kann es ja auch zu einen Integer casten. xD