ich bin gerade dabei, einen Texteditor zu programmieren und habe mithilfe dieses Forums schon die eine oder andere schwere Hürde überwunden. Leider tut sich vor mir gerade die nächste auf. Es geht um das Einlesen einer TXT-Datei in das Textfeld. Nun liefert mir der Compiler einige Fehlermeldungen à la "invalid conversion from void to char". Hier mal der Code (wie immer aufs Wesentliche beschränkt):

LRESULT CALLBACK AppWndProc(HWND hwnd, UINT uiMsg, WPARAM wParam, LPARAM lParam) 
{

   HDC hdc;
   PAINTSTRUCT ps;

   EnableWindow(g_hButton2, FALSE);
   EnableWindow(g_hButton3, FALSE);

   static HWND hEdit;

   switch(uiMsg) 
   { 
      case WM_CREATE:
         Create_GUI_Interface(hwnd);

         FILE *fz;
         char *buffer = NULL;
         int iFileSize;

         fz = fopen("Bla.txt", "rb");
         if( fz != NULL)
         {
             fseek(fz, 0, SEEK_END);
             iFileSize = ftell(fz);
             buffer = malloc(iFileSize);

             fseek(fz, 0, SEEK_SET);
             fread(buffer, 1, iFileSize, fz);
             fclose(fz);
         }

         // Erzeugt im Fenster ein Editfeld.
         hEdit = CreateWindowEx(WS_EX_CLIENTEDGE,
                 "EDIT",
                 buffer,
                 WS_CHILD | WS_VISIBLE | WS_VSCROLL | ES_MULTILINE | ES_AUTOVSCROLL,
                 10,                                                         // Abstand zum linken Rand
                 100,                                                        // Abstand zum oberen Rand
                 400,                                                        // Feldbreite
                 350,                                                        // Feldhöhe
                 hwnd,
                 NULL,
                 g_hInstance,
                 NULL);

         free(buffer);
         return 0;

      case WM_PAINT:
           hdc = BeginPaint (hwnd, &ps);
                 SetBkMode (hdc, TRANSPARENT);

                 LFTahoma.lfHeight = 80;
                 LFTahoma.lfWeight = 0; // FW_BOLD, FW_ITALIC, FW_STRIKEOUT usw.
                 hFont = (HFONT)SelectObject (hdc, CreateFontIndirect (&LFTahoma));
                 TextOut (hdc, 20, 5, "Test", 4);
                 DeleteObject (SelectObject (hdc, hFont));

                 LFArial.lfHeight  = 15;
                 LFArial.lfWeight  = 0; // FW_BOLD, FW_ITALIC, FW_STRIKEOUT usw.
                 hFont = (HFONT)SelectObject (hdc, CreateFontIndirect (&LFArial));
                 TextOut (hdc, 4, 527, "(c) by Itachi", 13);
                 DeleteObject (SelectObject (hdc, hFont));
                 EndPaint (hwnd, &ps);
      return 0;

      case WM_COMMAND: 
         switch( LOWORD(wParam) ) 
         {
            case ID_BUTTON:
               if( HIWORD(wParam) == BN_CLICKED )
               {
                  int Confirm = MessageBox (NULL, "Programm wirklich beenden?", g_lpszAppTitle, MB_YESNO | MB_ICONEXCLAMATION);
                  {
                      if (IDYES == Confirm)
                      {
                                SendMessage(hwnd, WM_CLOSE, 0, 0);
                      }
                  } 
               } 
               break;

            case ID_BUTTON2:
               if( HIWORD(wParam) == BN_CLICKED )
               { 
                  ShowMessage("Dieser Button ist deaktiviert!");
               } 
               break;

            case ID_BUTTON3:
               if( HIWORD(wParam) == BN_CLICKED )
               { 
                  ShowMessage("Dieser Button ist deaktiviert!");
               } 
               break;
         } 
         return 0;

      case WM_CLOSE:
           FILE *fz;
           char *buffer = NULL;
           int iLength;

           iLength = GetWindowTextLength(hEdit);

           buffer = malloc(iLength);

           GetWindowText(hEdit, buffer, iLength+1);

           fz = fopen("Bla.txt", "wb");
           fwrite(buffer, 1, iLength, fz);
           fclose(fz);

           free(buffer);
      return 0;

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

   return DefWindowProc(hwnd, uiMsg, wParam, lParam); 
}

Ich suche bereits stundenlang sowohl hier im Forum, als auch via Google und in der MSDN. Doch da ich im Bezug auf das Einlesen von Dateien noch nicht so bewandert bin, weiß ich auch nicht recht, nach welchen Begriffen ich eigentlich suchen muss. Zurzeit finde ich nämlich nicht das, was ich suche. Würde mich daher über Hilfe freuen

Gruß

buffer = malloc(iFileSize);

Edit: Eigentlich müsste es heißen:

buffer = (char*)malloc((iFileSize + 1) * sizeof(buffer));

Das ist hier nur egal, da es sowieso auf ANSI läuft (kein UNICODE).

Edit-2: Das +1 bei malloc bezieht sich aber nur auf das 2. malloc (unter WM_CLOSE).

PS: Poste auch mal die Error-Log des Compilers.

okay, hier mal die Errors:

In function LRESULT AppWndProc(HWND__*, UINT, WPARAM, LPARAM)': 188 invalid conversion from \void*' to `char*'
213 jump to case label
180 crosses initialization of char*buffer' 233 jump to case label 180 crosses initialization ofcharbuffer'
266 jump to case label
180 crosses initialization of char*buffer' 267 redeclaration ofFILEfz'
179 FILE*fz' previously declared here 268 redeclaration ofcharbuffer'
180 char*buffer' previously declared here 273 invalid conversion from \void' to `char*'
285 jump to case label
180 crosses initialization of `char*buffer'
[Build Error] [Test.o] Error 1

Ich hoffe, das hilft dir

Gruß

case WM_CLOSE: {
           FILE *fz;
           char *buffer = NULL; // <- deswegen!
           int iLength;

           iLength = GetWindowTextLength(hEdit);

           buffer = malloc(iLength);

           GetWindowText(hEdit, buffer, iLength+1);

           fz = fopen("Bla.txt", "wb");
           fwrite(buffer, 1, iLength, fz);
           fclose(fz);

           free(buffer);
           } return 0;

Weiter sage ich es noch einmal ....
TU ENDLICH CASTEN, WENN DEIN COMPILER MECKERT

void* ist eine Adresse im Speicher, ohne zu wissen welcher Type dahinter steckt, sag dem Compiler per cast was es ist!

FILE *fz;

haste sicher mal Global auch, oder?
und char* buffer sicher auch noch.

Mensch Junge, schön das du an einem Editor zum Anfang dich ranmachst, denoch solltest du einige Grundlegende C/C++ Kenntnisse, besser gesagt überhaupt bei allen Sprachen, erst das Grundwissen erlenen.

Zeiger (void* / long*)
*zeiger = new Speicherbereich der Klasse etc. [erstellen]
*zeiger = &zeiger2 [adresse kopieren / zuweisen]

Die Syntax und den Umgang solltest du erst lernen, dann sind diese Fragen Überflüssig

In der MSDN findest du Beispiele für korrektes WinAPI-Dateihandling unter dem Stichwort CreateFile.

Bei dieser Gelegenheit solltest du dich auch gleich von malloc, alloc, calloc, free und Co. verabschieden. Diese Funktionen haben in einem Windowsprogramm nichts verloren (und funktionieren z.B. im Zusammenhang mit Zwischenablage erst gar nicht). Für den globalen Heap sind GlobalAlloc(), GlobalLock(), GlobalReAlloc(), GlobalSize() und GlobalFree() die geeigneteren Kandidaten (auch hier: vergl. MSDN).

open, fwrite, fclose etc. wurden aus Gründen der Abwärtskompatibilität in Win32 übernommen.

Hä? Das sind Funktionen der C Laufzeitbibliothek

fopen, fwrite, fclose etc. wurden aus Gründen der Abwärtskompatibilität in Win32 übernommen. Von ihrer Verwendung in neuen Programmen wird abgeraten, da diese Funktionen veraltet sind. In Win32-Anwendungen sollte man Dateien mit der leistungsfähigeren Funktion CreateFile() erstellen oder öffnen. Dieser Empfehlung von Jeffrey Richter ist nichts hinzuzufügen.

In der MSDN findest du Beispiele für korrektes WinAPI-Dateihandling unter dem Stichwort CreateFile.

Bei dieser Gelegenheit solltest du dich auch gleich von malloc, alloc, calloc, free und Co. verabschieden. Diese Funktionen haben in einem Windowsprogramm nichts verloren (und funktionieren z.B. im Zusammenhang mit Zwischenablage erst gar nicht). Für den globalen Heap sind GlobalAlloc(), GlobalLock(), GlobalReAlloc(), GlobalSize() und GlobalFree() die geeigneteren Kandidaten (auch hier: vergl. MSDN).

Höhr auf zu saufen und gehe pennen!
Und der Spruch mit der Zwischenablage kann man nicht beantworten, dazu reicht die Smilie auswahl nicht.....

mach mal überall bei char *blub = NULL, das = NULL weg;
jeder Zeiger bekommt den wert 0 und dadurch kreuzen die sich, welche Idiot hat dir eigentlich gesagt, das man Zeigern einen Wert geben muß, ausser durch new oder Instanzt zuweisung?
der *blub zeiger bekommt seinen Zeiger doch noch!

Weiter sage ich es noch einmal ....
TU ENDLICH CASTEN, WENN DEIN COMPILER MECKERT

void* ist eine Adresse im Speicher, ohne zu wissen welcher Type dahinter steckt, sag dem Compiler per cast was es ist!

Bei allem Respekt, aber bleib bitte mal sachlich und auf dem Teppich. Ich lerne jeden Tag etwas dazu, aber alles braucht seine Zeit. Wie wäre es denn, wenn du mir einfach mal erklärst, wie ich caste? Das ist im WinAPI-Tutorial nämlich nicht ersichtlich. Ich beherrsche durchaus die Grundlagen, nur ist mein Wissen halt noch löchrig. Wenn ich hier im Forum suche, finde ich nämlich nur andere Themen, aber nichts, was mir direkt erklärt, was "cast" ist.

Und ich glaube kaum, dass ich den Editor so, wie er jetzt ist, ohne Grundwissen hinbekommen hätte. Ich hätte halt nur gerne gewusst, wie ich diese Fehler ausbessere. Denn was für euch banal erscheint, ist für mich nicht ganz so einfach, also bitte ich um etwas Verständnis. Würde mich also über eine Antwort freuen

Gruß

Wenn du nicht weiß, was ein cast ist, beherschst du offensichtlich nicht die Grundlagen. Das hier ist das WinAPI Forum, Leute.

Die Grundlagen bestehen nur aus "cast"? Wenn dem so ist, hast du Recht. Da dem aber nicht so ist, kannst du dir deine sinnlosen Posts sparen. Anonym meckern kann ich auch. Ich bin aber ein Neuling auf dem WinAPI-Gebiet und bitte um Hilfe. Wenn ich gewusst hätte, dass man hier sofort alles verstehen muss und schon schief angeguckt wird, wenn man mal etwas nicht weiß, dann hätte ich mich in einem anderen Forum registriert. Ich konnte ja nicht wissen, dass das Nachfragen verboten ist. Wenn du mir nicht helfen kannst/willst sei doch bitte so lieb und sag einfach gar nichts. Aber falls es hier noch irgendjemanden gibt, der einem Neuling unter die Arme greifen kann/möchte, dann bitte ich ihn, mir zu helfen

Gruß

danke :D. Ich hab mir das noch mal ganz genau angeschaut und malloc gecastet:

buffer = (char*)malloc((iFileSize + 1) * sizeof(buffer));

Nun funktioniert das Einlesen auch. Bis zu diesem Punkt bin ich schon mal zufrieden (an dieser Stelle danke an CodeFinder ;)). Nur leider verursacht mein Programm nun eine hohe Systemauslastung und lässt sich zunächst auch nicht vom Taskmanager beenden. Dieser springt erst ein, wenn die Systemauslastung 100% erreicht hat. Zudem beschreibt mein Editor die Textdatei mit irgendwelchen Zeichen. Des Weiteren lässt sich das Programm auch nicht mehr über meinen Button beenden. Ich habe mal einen Screen gemacht: http://s9.photobucket.com/albums/a87/24maniac/editor.jpg

Wobei das "YÚ" nicht in die Datei gehört. Sorry, falls ich nerven sollte, aber ich möchte das so gut es geht verstehen

Gruß

...
 buffer = (char *) malloc((iFileSize+1) * sizeof(buffer));
 buffer [iFileSize] = '\0'; // <- so oder so ähnlich
...

Warum der Schliessen-Button versagt, lässt sich aus der Ferne nicht sagen. Möglicherweise hast Du in der WM_CLOSE ein "return 0" als letzte Anweisung.
Poste mal die AppWndProc () in ihrer aktuellsten Version (mit allen Änderungen).

du hast kein DestroyWindow in der WM_CLOSE und gibts 0 zurück du Holzkopf!

Einfach unfassbar, was man sich hier alles an den Kopf werfen lassen muss. Dieses Newbie-feindliche Verhalten kotzt mich dermaßen an, das glaubst du gar nicht. Wenn du mich nur beleidigen willst, kannst du dir weitere Postings getrost sparen. Komisch, dass sich das alle nur anonym trauen... Naja, sei es drum.

@ merker:

Nun stürzt das Programm nicht mehr ab und lässt sich auch normal beenden, danke schonmal ;). Bleibt noch das Problem mit den Zeichen in der TXT. Der Text wird nämlich auch nicht gespeichert, wenn ich ihn ändere. Hier mal der Code:

LRESULT CALLBACK AppWndProc(HWND hwnd, UINT uiMsg, WPARAM wParam, LPARAM lParam) 
{

   HDC hdc;
   PAINTSTRUCT ps;

   EnableWindow(g_hButton2, FALSE);
   EnableWindow(g_hButton3, FALSE);

   static HWND hEdit;

   switch(uiMsg) 
   { 
      case WM_CREATE:
         Create_GUI_Interface(hwnd);

         FILE *fz;
         char *buffer;
         int iFileSize;

         fz = fopen("Bla.txt", "rb");
         if( fz != NULL)
         {
             fseek(fz, 0, SEEK_END);
             iFileSize = ftell(fz);
             buffer = (char*)malloc((iFileSize + 1) * sizeof(buffer));
             buffer [iFileSize] = '\0';

             fseek(fz, 0, SEEK_SET);
             fread(buffer, 1, iFileSize, fz);
             fclose(fz);
         }

         // Erzeugt im Fenster ein Editfeld.
         hEdit = CreateWindowEx(WS_EX_CLIENTEDGE,
                 TEXT("EDIT"),
                 TEXT(buffer),
                 WS_CHILD | WS_VISIBLE | WS_VSCROLL | ES_MULTILINE | ES_AUTOVSCROLL,
                 10,                                                         // Abstand zum linken Rand
                 100,                                                        // Abstand zum oberen Rand
                 400,                                                        // Feldbreite
                 350,                                                        // Feldhöhe
                 hwnd,
                 NULL,
                 g_hInstance,
                 NULL);

         free(buffer);
         return 0;

      case WM_PAINT:
           hdc = BeginPaint (hwnd, &ps);
                 SetBkMode (hdc, TRANSPARENT);

                 LFTahoma.lfHeight = 80;
                 LFTahoma.lfWeight = 0;
                 hFont = (HFONT)SelectObject (hdc, CreateFontIndirect (&LFTahoma));
                 TextOut (hdc, 85, 5, "Blubb", 5);
                 DeleteObject (SelectObject (hdc, hFont));

                 LFArial.lfHeight  = 15;
                 LFArial.lfWeight  = 0;
                 hFont = (HFONT)SelectObject (hdc, CreateFontIndirect (&LFArial));
                 TextOut (hdc, 4, 527, "(c) by Itachi", 13);
                 DeleteObject (SelectObject (hdc, hFont));
                 EndPaint (hwnd, &ps);
      return 0;

      case WM_COMMAND: 
         switch( LOWORD(wParam) ) 
         {
            case ID_BUTTON:
               if( HIWORD(wParam) == BN_CLICKED )
               {
                  int Confirm = MessageBox (NULL, "Programm wirklich beenden?", g_lpszAppTitle, MB_YESNO | MB_ICONEXCLAMATION);
                  {
                      if (IDYES == Confirm)
                      {
                                SendMessage(hwnd, WM_CLOSE, 0, 0);
                      }
                  } 
               } 
               break;

            case ID_BUTTON2:
               if( HIWORD(wParam) == BN_CLICKED )
               { 
                  ShowMessage("Dieser Button ist deaktiviert!");
               } 
               break;

            case ID_BUTTON3:
               if( HIWORD(wParam) == BN_CLICKED )
               { 
                  ShowMessage("Dieser Button ist deaktiviert!");
               } 
               break;
         } 
         return 0;

      case WM_CLOSE:
      {
           FILE *fz;
           char *buffer;
           int iLength;

           iLength = GetWindowTextLength(hEdit);

           buffer = (char*)malloc((iLength + 1) * sizeof(buffer));
           buffer [iFileSize] = '\0';

           GetWindowText(hEdit, buffer, iLength+1);

           fz = fopen("Bla.txt", "wb");
           fwrite(buffer, 1, iLength, fz);
           fclose(fz);

           free(buffer);
      }

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

   return DefWindowProc(hwnd, uiMsg, wParam, lParam); 
}

buffer = (char*)malloc((iLength + 1) * sizeof(buffer));
           buffer [iFileSize] = '\0';

1. Du hast ein Fallthrough in deiner WM_CLOSE Behandlung und 2. du hast meinen Ratschlag nicht umgesetzt.

1. Hilft es mir recht wenig, wenn du irgendwelche Begriffe verwendest, die mir nicht geläufig sind,

2. war dein Post eine Beleidigung und kein Ratschlag...
__________________________________________________________

Eine genauere Formulierung wäre sehr hilfreich...

............................................................. ↑↑↑↑↑↑↑↑↑

was soll den das werden? Bitte mal schnell nachdenken wie viel Speicher du da Reservierst! (Filesize mal 4 ?)

änder das bitte, setzte den 0-Terminate nach dem einlesen der Datei, den so übergibts du ein 4mal größeren block.

setzte das PostQuitMessage in WM_CLOSE

#define IDC_EDT_TEXT	104
#define IDC_BTN_CLOSE	105
#define IDC_BTN_UNUSED1 106
#define IDC_BTN_UNUSED2	107

#include <string>
#include <sstream>
#include <fstream>

LRESULT CALLBACK AppWndProc(HWND hWnd, UINT msg, WPARAM wParam, LPARAM lParam)
{
	switch (msg)
	{
	case WM_CREATE:
		{
			Create_GUI_Interface(hwnd);
			HWND hWndEdit = CreateWindowExA(WS_EX_CLIENTEDGE, "edit", NULL,  WS_CHILD | WS_VISIBLE | WS_VSCROLL |
											ES_MULTILINE | ES_AUTOVSCROLL, 10, 100, 400, 350, hWnd, reinterpret_cast<HMENU>(IDC_EDT_TEXT),
											reinterpret_cast<HINSTANCE>(GetWindowLongA(hWnd, GWL_HINSTANCE)), NULL);
			if (hWndEdit == NULL)
			{
				MessageBoxA(hWnd, "Fehler beim erstellen des Fensters!", NULL, 0);
				return S_FALSE;
			}

			if (__argc > 1)
			{
				std::ifstream file_stream(__argv[1]);
				std::stringstream ss;
				ss << file_stream.rdbuf();
				SendMessageA(hWndEdit, WM_SETTEXT, 0, reinterpret_cast<LPARAM>(ss.str().c_str()));
			}
		} break; 
	case WM_COMMAND:
		{
			switch (LOWORD(wParam))
			{
				case IDC_BTN_CLOSE:
					{
						if (HIWORD(wParam) == BN_CLICKED && 
							MessageBoxA(hWnd, "Programm wirklich beenden?", "MyApp", MB_YESNO | MB_ICONEXCLAMATION) == IDYES)
							SendMessageA(hWnd, WM_CLOSE, 0, 0);
					} break;
				case IDC_BTN_UNUSED1:
					{
						if (HIWORD(wParam) == BN_CLICKED
							MessageBoxA(hWnd, "Dieser Button ist deaktiviert!", NULL, 0);
					} break;
				case IDC_BTN_UNUSED2:
					{
						if (HIWORD(wParam) == BN_CLICKED)
							MessageBoxA(hWnd, "Dieser Button ist deaktiviert!", NULL, 0);
					} break;
			}
		} break;
	case WM_CLOSE:
		{
			std::string file_name(__argc > 1 ? __argv[1] : "output.txt");
			std::ofstream file_stream(file_name.c_str());
			if (!file_stream)
				return S_FALSE;

			unsigned int len = GetWindowTextLengthA(GetDlgItemA(hWnd, IDC_EDT_TEXT)) + 1;
			char* buffer = new char[len];
			GetWindowTextA(GetDlgItemA(hWnd, IDC_EDT_TEXT), buffer);
			file_stream << buffer << std::endl;
			delete [] buffer;
		} break;
	case WM_DESTROY:
		{
			PostQuitMessage(0);
		} break;
	default:
		return DefWindowProc(hWnd, msg, wParam, lParam);
	}
	return S_OK;
}

Na ok C++-Style würde ich noch anders machen ... aber ist schonmal so nen bissel schöner

Bleibt noch das Problem mit den Zeichen in der TXT. Der Text wird nämlich auch nicht gespeichert, wenn ich ihn ändere.

Du hast in der WM_CLOSE eine Variable verwechselt :

case WM_CLOSE: 
{ 
 int iLength; // <- der hier ist es
...
 buffer = (char*)malloc((iLength + 1) * sizeof(buffer)); 
// buffer [iFileSize] = '\0'; 
 buffer [iLength] = '\0'; // <- also so hier
...
}

Beachte auch orr lol's Beitrag (... und nicht seine Wortwahl !) was es mit dem "sizeof(buffer)" auf sich hat.