[C] Zeichenfolge aus Funktion zurückgeben

[Luckie]

Folgender Code:

Code:

#include <stdio.h>
#include <windows.h>


TCHAR* SysErrorMessage(int ErrorCode) {
   TCHAR szBuf[80]; 
    LPVOID lpMsgBuf;
    DWORD dw = GetLastError(); 

    FormatMessage(
        FORMAT_MESSAGE_ALLOCATE_BUFFER | 
        FORMAT_MESSAGE_FROM_SYSTEM,
        NULL,
        dw,
        MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT),
        (LPTSTR) &lpMsgBuf,
        0, NULL );

   wsprintf(szBuf, (LPTSTR)lpMsgBuf);
   LocalFree(lpMsgBuf);
   return szBuf;     
}

int main(int argc, char* argv[])
{
   printf(SysErrorMessage(5));
   return 0;
}

Leider gelingt es mir nicht eine ganz normale Zeichenfolge beliebiger Länmge aus der Funktion zurückzugeben, so dass ich sie mit printf ausgeben kann.

Zusatzaufgabe: Wie mache ich das ganze Unicode sicher?

[Vjay]

Benutzt du dort den GNU-C Compiler?

Mh, ohne jetzt eine genaue Ahnung zu haben, würde ich versuchen einen pChar zurückzugeben.

Grad mal in die Hilfe zu FormatMessage geguckt:

Zitat:

pBuffer

Points to a buffer for the formatted (and null-terminated) message. If dwFlags includes FORMAT_MESSAGE_ALLOCATE_BUFFER, the function allocates a buffer using the LocalAlloc function, and places the address of the buffer at the address specified in lpBuffer.

Denke ich weiss worauf du hinaus willst, aber Standard-C hat nicht die Super-Delphi-Stringbehandlung.

[ste_ett]

Code:

#include <stdio.h>
#include <windows.h>


TCHAR* SysErrorMessage(int ErrorCode) {
   TCHAR szBuf[80];
   LPVOID lpMsgBuf;
//   DWORD dw = GetLastError();
   DWORD dw = ErrorCode;

   FormatMessage(
      FORMAT_MESSAGE_ALLOCATE_BUFFER |
      FORMAT_MESSAGE_FROM_SYSTEM,
      NULL,
      dw,
      MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT),
      (LPTSTR)&lpMsgBuf,
      0, NULL );

   wsprintf(szBuf, (LPTSTR)lpMsgBuf);
   LocalFree(lpMsgBuf);
   return szBuf;     
}

int main(int argc, char* argv[])
{
   printf("%s", SysErrorMessage(5));
   return 0;
}

- übergebenen Wert aus "ErrorCode" wird jetzt zur Umwandlung genommen
- Ausgabe geändert in "printf("%s", SysErrorMessage(5));"

Ich würde einen Char-Array als Parameter übergeben, in den du schreibst, oder du holst Speicher innerhalb der Funktion und gibst ihn nach dem Aufrufen wieder frei. :)

[Luckie]

Zitat:

Zitat von ste_ett

Ich würde einen Char-Array als Parameter übergeben, in den du schreibst, oder du holst Speicher innerhalb der Funktion und gibst ihn nach dem Aufrufen wieder frei. :)

So komplizierte Aufrufe wollte ich eigentlich vermeiden.

Deine Version kompiliert jetzt zwar, aber es wird nur Datenmüll ausgegeben. Später will ich die Funktion SysErrorMessage auch bei einer Messagebox oder so benutzen, da geht das ja dann nicht mehr mit printf("%s", ...);.

[Flocke]

Ganz grober Schnitzer: szBuf liegt auf dem Stack, ist also nach dem Aufruf der Funktion eigentlich nicht mehr gültig. Wenn der Inhalt noch drin steht, dann hast du Glück. Ein erster Bugfix wäre, static TCHAR szBuf[80]; zu benutzen.

Allerdings ist es in C üblich, den Ergebnisparameter zusammen mit der maximalen Länge zu übergeben, wie ste_ett schon schrieb.

[Luckie]

Also so was:

Code:

void SysErrorMessage(int ErrorCode, char* Buffer, int lenBuffer)

Nur bin ich in C noch nicht so tritt fest, wie müsste denn dann die Implementation und der Aufruf aussehen? Und ambesten noch Unicode fähig. Wir wollen ja gleich up to date sein. ;)

[ste_ett]

Code:

#include <stdio.h>
#include <windows.h>

DWORD SysErrorMessage(const int ErrorCode, wchar_t* szBuf, const DWORD ArrayLength)
{
   DWORD result = 0;
   LPVOID lpMsgBuf = NULL;
   DWORD dw = ErrorCode;

   result = FormatMessage(FORMAT_MESSAGE_ALLOCATE_BUFFER | FORMAT_MESSAGE_FROM_SYSTEM, NULL, dw,
                     MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT), (LPWSTR)&lpMsgBuf, 0, NULL);

   if (result <= ArrayLength)
   {
      wsprintf(szBuf, (LPWSTR)lpMsgBuf);
   }
   else
   {
      result = (DWORD)-1; // ErrCode
   }

   LocalFree(lpMsgBuf);

   return result;
}

int main(int argc, char* argv[])
{
   TCHAR szBuf[80];
   ZeroMemory(szBuf, sizeof(szBuf));

   if (SysErrorMessage(5, szBuf, sizeof(szBuf) / 2))
   {
      wprintf(L"%s", szBuf);
   }

   return 0;
}

z.B. so :)

[Flocke]

Etwa so:

Code:

TCHAR *
SysErrorMessage (DWORD dwErrorCode, TCHAR *pszBuf, int nLenBuf)
{
  FormatMessage(
    FORMAT_MESSAGE_FROM_SYSTEM | (pszBuf ? FORMAT_MESSAGE_ALLOCATE_BUFFER : 0),
    NULL,
    dwErrorCode ? dwErrorCode : GetLastError(),
    MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT),
    pszBuf ? pszBuf : (LPTSTR)&pszBuf,
    pszBuf ? nLenBuf : 0,
    NULL);
  return pszBuf;
}

// Nachtrag: noch erweitert

[Luckie]

Ah, ja die Version von ste_ett funktioniert und ich habe sie verstanden. Aber wäre es nicht sinnvoll die benötigte Länge des Buffers zurückzugeben, so dass man im Bedarfsfall entsprechend mehr Speicher reservieren kann? aslo:

Code:

len = SysErrorMessage(5, NULL, 0);
GetMem(szBuff, len)
SysErrorMessage(5,szBuf, len);
FreeMem(szBuf);

Das ist jetzt Delphi Pseudocode.

Und wie kann ich den Code in eine eigene Quellcodedatei auslagern? Ich habe ihn in eine eigene Datei kopiert MpuTools.cpp und eine entsprechende headerdatei angelegt: MpuTools.h, die ich dann mit include eingebunden habe:

Code:

#include <stdio.h>
#include <windows.h>
#include "MpuTools.h"

int main(int argc, char* argv[])
{
   TCHAR szBuf[80];
   ZeroMemory(szBuf, sizeof(szBuf));

   if (SysErrorMessage(5, szBuf, sizeof(szBuf)) >0)
   {
      wprintf(L"%s", szBuf);
     MessageBoxW(0, szBuf, L"Test", 0);
   }

   return 0;
}

Das mag der Compiler jedoch nicht:

Zitat:

Fehler 1 error C2144: Syntaxfehler: 'int' sollte auf ';' folgen c:\dokumente und einstellungen\mp\eigene dateien\visual studio 2005\projects\mputools\mputools_test\mputools_test .cpp 5

OK, in der Header-Datei hat ein Semikolon gefehlt. Jetzt bekomme ich aber die fehlermeldung:

Zitat:

Fehler 1 error LNK2019: Verweis auf nicht aufgelöstes externes Symbol ""unsigned long __cdecl SysErrorMessage(int,wchar_t *,unsigned long)" (?SysErrorMessage@@YAKHPA_WK@Z)" in Funktion "_main". MpuTools_Test.obj

:gruebel:

[Luckie]

OK, das Problem ist gelöst: Ich musste noch die dateien zum Projekt hinzufügen. Ich dachte das passiert irgendwie automatisch, wenn ich die entsprechende Header-Datei einbinde. Damit wäre alles geklärt. Besten Dank für eure Hilfe.