c++ / arduino: string zu variable parsen

in meinem arduino projekt habe ich 1000+ zeilen zum parsen von kommandos, variablen etc, die über die serielle schnittstelle hereinkommen:

Code: CPP

bool EEPROMHandler::stringToVariable(uint16_t device, char* string, uint16_t& variable)
{
	if (device == EEPROMHandler::device_type_::GENERAL)
	{
		if (strcmp_P(string, PSTR("DISPLAY_AVAILABLE")) == 0)
		{
			variable = general_variable_:: DISPLAY_AVAILABLE;
			return true;
		}

		if (strcmp_P(string, PSTR("DISPLAY_TYPE")) == 0)
		{
			variable = general_variable_:: DISPLAY_TYPE;
			return true;
		}

		if (strcmp_P(string, PSTR("DISPLAY_RES_X")) == 0)
		{
			variable = general_variable_:: DISPLAY_RES_X;
			return true;
		}

		if (strcmp_P(string, PSTR("DISPLAY_RES_Y")) == 0)
		{
			variable = general_variable_:: DISPLAY_RES_Y;
			return true;
		}

		if (strcmp_P(string, PSTR("DISPLAY_UPDATE_INTERVAL")) == 0)
		{
			variable = general_variable_:: DISPLAY_UPDATE_INTERVAL;
			return true;
		}

		if (strcmp_P(string, PSTR("DISPLAY_BRIGHTNESS")) == 0)
		{
			variable = general_variable_:: DISPLAY_BRIGHTNESS;
			return true;
		}

		if (strcmp_P(string, PSTR("DISPLAY_CONTRAST")) == 0)
		{
			variable = general_variable_:: DISPLAY_CONTRAST;
			return true;
		}
...

hat jemand eine idee wie man das noch optimieren kann? ich würde gerne bei einem human readable protokoll bleiben.

json file falls das für c++ gibt

Wie ist general_variable_:: DISPLAY_AVAILABLE definiert?

Asoo. Dann bleib ich bei meiner Variante. String-Array sauber aufstellen (ob jetzt ausm JSON, einem Textfile oder im Code, ist egal).
Mit foreach durchiterieren, wenn der strcmp matcht, return array val.
Das ist ein Dreizeiler.

Falls du deinen Enum behalten willst, obwohl enums nicht wirklich zum durchiterieren geeignet sind, koenntest du dir diese Iterator-Klasse ansehen:
http://www.drdobbs.com/when-enum-ju...ion-c/184403955

Also die schoenste Loesung waere eine Hashmap: http://stackoverflow.com/a/3578247

ich habe derzeit noch ca 160kb flash frei, aber das programm ist noch lange nicht fertig
mein eigener naiver ansatz wäre gewesen ein array von structs zu erstellen:

Code: CPP

struct command_inst_
{
	const PROGMEM char* string;
	uint16_t command;
};

command_inst_ command_list_[NUM_CMDS];

dann könnte ich einfach drüber iterieren und den zum string passenden command zurückgeben, das würde aber wahrscheinlich in sachen geschwindigkeit, speicherverbrauch und lesbarkeit nix bringen