//Editfelder auslesen, und in chars old_lp1 und damage_lp1 speichern
GetWindowText (hStartPoints1, old_lp1, 25);
GetWindowText (hLifePoints1, damage_lp1, 25);
//old_lp1 und damage_lp1 in Integer lp11 und lp22 konvertieren
lp11 = atoi (old_lp1);
lp12 = atoi (damage_lp1);
//lp12 von lp11 subtrahieren
lp1 = lp11 - lp12;
//lp1 in den char new_lp1 wandeln
sprintf (new_lp1, "%i", lp1);
//Inhalt vom Editfeld hStartPoints1 durch new_lp1 ersetzen
SetWindowText (hStartPoints1, new_lp1);

Allerdings passiert aber nichts im Editfeld, also die Zahl bleibt die drinstand. Ich habe den char-Array über ne MessageBox ausgeben lassen. Die Ausgabe lautete 0. Das kann aber nicht sein, da 12 - 3 = 9 und nicht 0 ist. Weiß jemand was ich falsch mache?

Mfg Ominion

char old_lp1[10];

char damage_lp1[10];

char new_lp1[10];

int lp1;

int lp11;
int lp12;

Die Handles sind gültig, und das ganze steckt in keiner Funktion, sondern in einer WM_COMMAND-Message.

Mfg Ominion

Mfg Ominion

dachte ich mir,

Da dran scheitern viele!

na du solltest schon schauen dass "der Buffer" genauso groß ist wie der String der den Text aufnehmen soll.

[cpp]
    int GetWindowText(

        HWND hWnd,	// handle of window or control with text
        LPTSTR lpString,	// address of buffer for text
        int nMaxCount 	// maximum number of characters to copy
        }

    CHAR lpString[10];
    nMaxCount = lstrlen(lpString);
    GetWindowText(hwnd, lpString, nMaxCount);

    // alternativ

    GetWindowText(hwnd, lpString, 10); //  :o) 

[/cpp]

Das heißt, dass nicht angegeben werden soll wieviele chars es genau sind, sondern wieviele es maximal sind. Also ist es egal ob ich mehr angebe als benötigt werden.

Habs trotzdem mal probiert. Hat auch nichts gebracht.

Mfg Ominion

Das heißt, dass nicht angegeben werden soll wieviele chars es genau sind, sondern wieviele es maximal sind.

du gibst an wieviel chars maximal in dein array kopiert werden können.

Ominion schrieb:

Also ist es egal ob ich mehr angebe als benötigt werden.

du solltest auf keinen fall mehr angeben als dein char array gross ist

Mfg Ominion

Woran liegt das?

der codeabschnitt schein korrekt zu sein. bemühe doch mal deinen debugger

int nMaxCount     // maximum number of characters to copy

Das heißt, dass nicht angegeben werden soll wieviele chars es genau sind, sondern wieviele es maximal sind. Also ist es egal ob ich mehr angebe als benötigt werden.

quatsch. theoretisch kann windows an der addresse des buffers 25 bytes auf 0 setzen oder sonst irgendwelchen quatsch machen. das bort sich dann schön in eine anderen sachen auf dem stack.

debugger ist etwas übertrieben für so ein problem. lass dir doch erstmal die einzelnen strings nach GetWindowText ausgeben und guck wo was nicht geht. oder poste mehr code.

debugger ist etwas übertrieben für so ein problem.

Mfg Ominion

--------------------Konfiguration: main - Win32 Debug--------------------
Kompilierung läuft...
Fehler beim Start cl.exe

main.exe - 1 Fehler, 0 Warnung(en)

Mfg Ominion

Mfg Ominion

~o_

LRESULT CALLBACK WndProc (HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
{
	HWND hStartPoints1;
	HWND hStartPoints2;

	HWND hLifePoints1;
	HWND hLifePoints2;

	HWND hCalc1;
	HWND hCalc2;

	char old_lp1[10];
	char old_lp2[10];

	char damage_lp1[10];
	char damage_lp2[10];

	char new_lp1[10];
	char new_lp2[10];

	int lp1;
	int lp2;

	int lp11;
	int lp12;

	int lp21;
	int lp22;

	int length11;
	int length12;

	int length21;
	int length22;

	PAINTSTRUCT ps;
	HDC hdc;
	HGDIOBJ OldObj;

	switch (message)
    {

		case WM_CREATE:
			  hStartPoints1 = CreateWindow ("Edit", NULL, WS_CHILD | WS_VISIBLE, 70, 30, 100, 17, hWnd, (HMENU)1, g_hInst, NULL); 
			  hStartPoints2 = CreateWindow ("Edit", NULL, WS_CHILD | WS_VISIBLE, 200, 30, 100, 17, hWnd, (HMENU)2, g_hInst, NULL);

			  hLifePoints1 = CreateWindow ("Edit", NULL, WS_CHILD | WS_VISIBLE, 70, 55, 100, 17, hWnd, (HMENU)3, g_hInst, NULL);
			  hLifePoints2 = CreateWindow ("Edit", NULL, WS_CHILD | WS_VISIBLE, 200, 55, 100, 17, hWnd, (HMENU)4, g_hInst, NULL);

			  hCalc1 = CreateWindow ("Button", "Berechnen", WS_CHILD | WS_VISIBLE | BS_PUSHBUTTON, 70, 80, 100, 20, hWnd, (HMENU)5, g_hInst, NULL);
			  hCalc2 = CreateWindow ("Button", "Berechnen", WS_CHILD | WS_VISIBLE | BS_PUSHBUTTON, 200, 80, 100, 20, hWnd, (HMENU)6, g_hInst, NULL);
		return 0;

		case WM_COMMAND:
			  switch (LOWORD(wParam))
			  {
				  case 5:
					  length11 = lstrlen (old_lp1);
					  length12 = lstrlen (damage_lp1);

					  GetWindowText (hStartPoints1, old_lp1, length11);
					  GetWindowText (hLifePoints1, damage_lp1, length12);

					  lp11 = atoi (old_lp1);
					  lp12 = atoi (damage_lp1);

					  lp1 = lp11 - lp12;

					  sprintf (new_lp1, "%i", lp1);

					  SetWindowText (hStartPoints1, new_lp1);
				  return 0;

				  case 6:
					  length21 = lstrlen (old_lp2);
					  length22 = lstrlen (damage_lp2);

					  GetWindowText (hStartPoints2, old_lp2,length21);
					  GetWindowText (hLifePoints2, damage_lp2, length22);

					  lp21 = atoi (old_lp2);
					  lp22 = atoi (damage_lp2);

					  lp2 = lp21 - lp22;

					  sprintf (new_lp2, "%i", lp2);

					  SetWindowText (hStartPoints2, new_lp2);
				  return 0;
			  }
		return 0;

		case WM_PAINT:
			  hdc = BeginPaint (hWnd, &ps);
			        OldObj = SelectObject (hdc, GetStockObject (DEFAULT_GUI_FONT)); 
                    SetBkMode (hdc, TRANSPARENT);
					TextOut (hdc, 10, 30, "Life Points", strlen ("Life Points"));
					TextOut (hdc, 10, 55, "Schaden", strlen ("Schaden"));
					TextOut (hdc, 100, 10, "Spieler 1", strlen ("Spieler 1"));
					TextOut (hdc, 230, 10, "Spieler 2", strlen ("Spieler 2"));
					TextOut (hdc, 10, 105, "© Software Meadow", strlen ("© Software Meadow"));
					DeleteObject (SelectObject (hdc, OldObj));
			  EndPaint (hWnd, &ps);
	    return 0;

		case WM_DESTROY:
			  PostQuitMessage (0);
		return 0;
	}

    return DefWindowProc (hWnd, message, wParam, lParam);
}

Mfg Ominion

length11 = lstrlen (old_lp1);
length12 = lstrlen (damage_lp1);

Mfg Ominion