Bsp. Zeichenktte vom server "string1_string2string3_{string4|string1}string2~string3|...."

Source:

void HandleCommand(char* data)
{
	char Command[sizeof(data)];
	strcpy(Command,data);
	char *CommandBuffer[sizeof(data)];
	char *pch;
	int i = 0;

	pch = strtok(Command,"~");
	while (pch != NULL)
	{
		CommandBuffer[i] = pch;
		pch = strtok (NULL, "~");
		i++;
	}
	if(!lstrcmp(CommandBuffer[0], "11"))
	{
		MessageBox ( NULL, CommandBuffer[1], "Test", MB_OK);
	}
}

void StartCommandManagement(char *WebpanelRequest,char *Domain)
{ 
	while (true)
	{
		char *request = HttpRequest(WebpanelRequest,Domain,80);
		char *pch;

		pch = strtok(request,"|");
		while (pch != NULL)
		{
			if(strstr(pch, "~")!=0){
				//Start Handle Command
				HandleCommand(pch);
			}
			pch = strtok(NULL, "|");
		}
		request[0] = '\0';
		break;
	}
}

void HandleCommand(char* data)
{
	char Command[sizeof(data)];
	strcpy(Command,data);
	char *CommandBuffer[sizeof(data)];

Auch in deinem reinen C-Code liefert sizeof(data) nicht dir Länge des Arrays! Das kann also nur schief gehen.

das ja wyn kann ja auch 50 oder so reinschreiben aber wo ist der fehler das er immer nur die 1. zeichenkette bearbeitet?

? wyn?

Nachmal darüber nachdenken was für einen Wert du mit sizeof(char*) erhälst
Und wieviele Elemente demnach Command und CommandBuffer haben.

void HandleCommand(char* data)
{
	char Command[5];
	strcpy(Command,data);
	char *CommandBuffer[5];
	char *pch;
	int i = 0;

	pch = strtok(Command,"~");
	while (pch != NULL)
	{
		CommandBuffer[i] = pch;
		pch = strtok (NULL, "~");
		i++;
	}
	if(!lstrcmp(CommandBuffer[0], "11"))
	{
		MessageBox ( NULL, CommandBuffer[1], "Test", MB_OK);
	}
}

void StartCommandManagement(char *WebpanelRequest,char *Domain)
{ 
	while (true)
	{
		char *request = HttpRequest(WebpanelRequest,Domain,80);
		char *pch;

		pch = strtok(request,"|");
		while (pch != NULL)
		{
			if(strstr(pch, "~")!=0){
				//Start Handle Command
				HandleCommand(pch);
			}
			pch = strtok(NULL, "|");
		}
		request[0] = '\0';
		break;
	}
}

Bsp. Zeichenktte vom server "string1_string2string3_{string4|string1}string2~string3|...."

spike1993 schrieb:

char Command[5];
    strcpy(Command,data)

Du machst deine Buffer viel zu kurz und wunderst dich dann, wenn du dir den Stack zerschießt und merkwürdiges Verhalten rauskriegst? Das mit der Speicherverwaltung solltest du besser noch mal üben. Mit zu wenig Speicher zu arbeiten ist keine Lappalie!

Ansonsten sieht mir der Code eigentlich so aus, als gehöre er mehr ins C- als ins C++-Forum. In C++ ginge das beispielsweise so:

void HandleCommand(std::string const &data)
{
  std::istringstream tokenizer(data);
  std::string token;
  std::vector<std::string> commands;

  while(std::getline(tokenizer, token, '~')) {
    commands.push_back(token);
  }

  if(commands.size() > 1 && commands[0] == "11") {
    MessageBox(NULL, commands[1].c_str(), "Test", MB_OK);
  }
}

void StartCommandManagement(std::string const &WebpanelRequest, std::string const &Domain)
{
  while (true) {
    std::string request = HttpRequest(WebpanelRequest.c_str(), Domain.c_str(), 80);
    std::istringstream tokenizer(request);
    std::string token;

    while(std::getline(tokenizer, token, '|')) {
      if(token.find('~') != std::string::npos) {
        HandleCommand(token);
      }
    }

    request.clear();
    break;
  }
}

...und c_str() auch nur, weil mir das nach einem C-API aussieht, um das man nicht herumkommt.

Fehler:

1>  CommControl.cpp
1>CommControl.cpp(10): error C2079: 'tokenizer' verwendet undefiniertes class 'std::basic_istringstream<_Elem,_Traits,_Alloc>'
1>          with
1>          [
1>              _Elem=char,
1>              _Traits=std::char_traits<char>,
1>              _Alloc=std::allocator<char>
1>          ]
1>CommControl.cpp(10): error C2440: 'Initialisierung': 'const std::string' kann nicht in 'int' konvertiert werden
1>          Kein benutzerdefinierter Konvertierungsoperator verfügbar, der diese Konvertierung durchführen kann, oder der Operator kann nicht aufgerufen werden
1>CommControl.cpp(12): error C2065: 'vector': nichtdeklarierter Bezeichner
1>CommControl.cpp(12): error C2275: 'std::string': Ungültige Verwendung dieses Typs als Ausdruck
1>          C:\Program Files\Microsoft Visual Studio 10.0\VC\include\xstring(2063): Siehe Deklaration von 'std::string'
1>CommControl.cpp(12): error C2065: 'commands': nichtdeklarierter Bezeichner
1>CommControl.cpp(14): error C2780: 'std::basic_istream<_Elem,_Traits> &std::getline(std::basic_istream<_Elem,_Traits> &,std::basic_string<_Elem,_Traits,_Alloc> &)': Erwartet 2 Argumente - 3 unterstützt
1>          C:\Program Files\Microsoft Visual Studio 10.0\VC\include\string(479): Siehe Deklaration von 'std::getline'
1>CommControl.cpp(14): error C2784: "std::basic_istream<_Elem,_Traits> &std::getline(std::basic_istream<_Elem,_Traits> &,std::basic_string<_Elem,_Traits,_Alloc> &,const _Elem)": template-Argument für "std::basic_istream<_Elem,_Traits> &" konnte nicht von "int" hergeleitet werden.
1>          C:\Program Files\Microsoft Visual Studio 10.0\VC\include\string(468): Siehe Deklaration von 'std::getline'
1>CommControl.cpp(14): error C2780: 'std::basic_istream<_Elem,_Traits> &std::getline(std::basic_istream<_Elem,_Traits> &&,std::basic_string<_Elem,_Traits,_Alloc> &)': Erwartet 2 Argumente - 3 unterstützt
1>          C:\Program Files\Microsoft Visual Studio 10.0\VC\include\string(448): Siehe Deklaration von 'std::getline'
1>CommControl.cpp(14): error C2784: "std::basic_istream<_Elem,_Traits> &std::getline(std::basic_istream<_Elem,_Traits> &&,std::basic_string<_Elem,_Traits,_Alloc> &,const _Elem)": template-Argument für "std::basic_istream<_Elem,_Traits> &&" konnte nicht von "int" hergeleitet werden.
1>          C:\Program Files\Microsoft Visual Studio 10.0\VC\include\string(395): Siehe Deklaration von 'std::getline'
1>CommControl.cpp(14): fatal error C1903: Weiterverarbeitung nach vorherigem Fehler nicht möglich; Kompilierung wird abgebrochen.
1>
1>Fehler beim Erstellen

hab ich irg was vergessen?