CString:Format()

_matze schrieb:

Du hast test bislang nicht initialisiert. Ich vermute mal, dass bei einem leeren CString und einer (impliziten) Konvertierung nach char* einfach 0 rauskommt, daher die Fehlermeldung. Weise test deinen String aus dem Beispiel zu (da war es noch str), dann sollte es klappen.

test wird in GetResource() mit einem CString belegt im UTF-8 Format. Also leer ist der String auf keinen Fall.

Hier mal die GerResource(). scheint ja, dass hier anscheinend etwas nicht stimmt im Bezug auf CString.Format().

static bool GetResource(const int& idrHTML, CString& rString)
{
bool retVal = false;
	try
	{		
		HRSRC hSrc = FindResource(NULL, MAKEINTRESOURCE(idrHTML), RT_HTML);
		if (hSrc != NULL)
		{
			HGLOBAL hHeader = LoadResource(NULL, hSrc);
			if (hHeader != NULL)
			{
				LPSTR lpcHtml = static_cast<LPSTR>(LockResource(hHeader));
				if (lpcHtml != NULL)
				{
					rString = CString(lpcHtml);
					retVal = true;
				}
				UnlockResource(hHeader);
			}
			FreeResource(hHeader);
		}
	}
	catch (CMemoryException* e)
	{
		SetLastError(ERROR_FUNCTION_FAILED);
		e->ReportError();
		e->Delete();
		retVal = false;
	}

}

_matze

Johannes K. schrieb:

_matze schrieb:

Du hast test bislang nicht initialisiert. Ich vermute mal, dass bei einem leeren CString und einer (impliziten) Konvertierung nach char* einfach 0 rauskommt, daher die Fehlermeldung. Weise test deinen String aus dem Beispiel zu (da war es noch str), dann sollte es klappen.

test wird in GetResource() mit einem CString belegt im UTF-8 Format. Also leer ist der String auf keinen Fall.

Siehe meine EDITs...

Du rufst hier GetResource auf. Dann rufst du darin GetResource auf. Dann rufst du... endlose Rekursion. Du musst schon irgendeine Abbruchbedingung definieren.

EDIT: Ach sorry, hab nicht richtig hingesehen. Das war nur ein Prototyp dort oben...

_matze schrieb:

EDIT2: Herrje, die Funktion dort heißt ja auch GetResource! Und du führst innerhalb dieser ohne Bedingung wieder GetResource aus. Das dürfte eine schöne Endlosrekursion sein...

Den Satz verstehe ich jetzt leider nicht. Ich führe einmal die Funktion GetResource() aus, danach ist der Cstring gefüllt und anschließend rufe ich einmal CString.Format() auf bzw. ich möchte es gern aufrufen..

Wo hab ich hier jetzt eine Endlosrekursion?

_matze

Johannes K. schrieb:

_matze schrieb:

EDIT2: Herrje, die Funktion dort heißt ja auch GetResource! Und du führst innerhalb dieser ohne Bedingung wieder GetResource aus. Das dürfte eine schöne Endlosrekursion sein...

Den Satz verstehe ich jetzt leider nicht. Ich führe einmal die Funktion GetResource() aus, danach ist der Cstring gefüllt und anschließend rufe ich einmal CString.Format() auf bzw. ich möchte es gern aufrufen..

Wo hab ich hier jetzt eine Endlosrekursion?

Wie gesagt, ich habe mich da verlesen. Ich dachte, du hättest eine Funktion GetResource gepostet, dabei stand bei dir oben nur der Prototyp der Funktion (geschweifte Klammer statt Semikolon gelesen).

_matze

Also ich kann das Fehlverhalten nicht nachvollziehen. Ich habe mir gerade schnell eine Mini-MFC-Anwendung erstellt und deine Funktion ausgeführt (leicht abgeändert, mit RT_DIALOG). Und egal, wie ich es drehe, ich bekomme keinen Fehler.

EDIT: Du solltest versuchen, dem Fehler selbst auf die Schliche zu kommen. Und zwar, in dem du deinen Code soweit reduzierst, dass mit minimaler Größe dein Fehler noch auftritt. Dann ist er quasi isoliert und du kannst per Debugger vermutlich ganz schnell sehen, wo genau der Fehler ist.

_matze schrieb:

Also ich kann das Fehlverhalten nicht nachvollziehen. Ich habe mir gerade schnell eine Mini-MFC-Anwendung erstellt und deine Funktion ausgeführt (leicht abgeändert, mit RT_DIALOG). Und egal, wie ich es drehe, ich bekomme keinen Fehler.

Es ist auch seltsam, dass z.B. CString.FormatMessage() einwandfrei funktioniert aber CString.Format() ebend nicht...

_matze

Ein Problem ist auch, dass ich nicht sicher sein kann, dass dein geposteter Code wirklich dem von dir verwendeten gleich ist. Du hast nun schon zweifach fehlerhaften Code gepostet (Klammer fehlte, return-Statement fehlte). Dein Code, sinnvoll von Fehlern befreit, läuft in meiner Testanwendung problemlos. Ich weiß nicht, was ich dazu noch sagen soll. Befolge meine Ratschläge (letzter Post, EDIT) und versuche so, dem Fehler auf die Schliche zu kommen. Merke: der Debugger ist dein Freund!

ich glaube es liegt am Format von test-String

if(GetResource(id, test)
{

  LPTSTR lpszData = new TCHAR[test.GetLength()+1];

}

Der CString test beinhaltet genau den UTF-8 Code den ich als Resource auch abgespeichert habe.

lpszData beinhaltet nur noch eckige Kästchen - daher vielleicht auch die Fehlermeldung "incorrect format specifier". Aber wie muss ich es abändern, dass es funktionieren kann?

_matze

Johannes K. schrieb:

ich glaube es liegt am Format von test-String
if(GetResource(id, test)
{

  LPTSTR lpszData = new TCHAR[test.GetLength()+1];

}
Der CString test beinhaltet genau den UTF-8 Code den ich als Resource auch abgespeichert habe.

lpszData beinhaltet nur noch eckige Kästchen - daher vielleicht auch die Fehlermeldung "incorrect format specifier". Aber wie muss ich es abändern, dass es funktionieren kann?

Du initialisierst lpzData ja auch nicht in dem Beispiel (du allozierst lediglich Speicher, beschreibst diesen dann aber nicht), das ist so also nicht aussagekräftig...