Anfänger - Programm funktioniert, aber....



  • @peroja sagte in Anfänger - Programm funktioniert, aber....:

    GetOpenFileName(&ofn);
    if(GetOpenFileName(&ofn))
        display_file(ofn.lpstrFile);
    

    Du darfst natürlich nur einmal GetOpenFileName(&ofn) aufrufen (gilt analog auch für GetSaveFileName(&ofn))!!!

    PS: Wegen Warnungen s. Einstellungen für Code::Blocks ("Compiler-Flags einstellen")



  • @yahendrik Ah ja besten Dank, ja dass ich mit Unicode das kompilieren wollte war natürlich ein grober Schnitzer.



  • Du darfst natürlich nur einmal GetOpenFileName(&ofn) aufrufen (gilt analog auch für GetSaveFileName(&ofn))!!!

    Ja klar... autsch jetzt wo es ... oh mann (peinlich)
    ok, hab ne faire aurede: ich hab erst vor ca 2 wochen angefangen mit C++ und bin aber etwa 3std. am tag dabei



  • nextsteps:
    1)
    ich versuche herauszufinden wie ich PNG und JPG anstelle BMP images einbinden kann. (nicht so wichtig)

    ich erzeuge ein modalfenster mit checkbox, radio und ein select dropdown und speichere die values in ein txt file.
    dieses soll beim nächsten aufruf des modals wieder gelesen werden und die checkbox, radio und select mit den entsprechenden values wieder geladen werden. (ein versuch später ein config file zu basteln)

    ich geh mal davon aus dass ich fopen() und fread() auch ohne dialogfenster nutzen kann....
    ich versuche erst mal und wenns nicht klappt frage ich.



  • @peroja sagte in Anfänger - Programm funktioniert, aber....:

    ich nutze die IDE CODEBLOCKS

    Selber schuld.

    Ne, ernsthaft? Vergiss die WinAPI und lern' erstmal C++.



  • @Swordfish ganz bestimmt werd ich das tun nur weil DU es sagst. ironieoff
    ich hab anständig gefragt und auch anständig geantwortet. und was ist an CODE::BLOCKS so schlimm?
    ich hab es eben jetzt als meine IDE zum lernen gewählt und nicht VS , obwohl ich das auch hab.
    wenn du mir beim lernen helfen willst dann dankeschön. wenn nicht, dann geh bitte wo anders stänkern. bei mir funzt sowas nicht.



  • @peroja sagte in Anfänger - Programm funktioniert, aber....:

    wenn nicht, dann geh bitte wo anders stänkern.

    ok

    // edit: Heck, alleine schon an dem Code oben sieht man wie viel blödsinn Du Dir mit der WinAPI "beibringst". Aber ist ja Dein Bier. Wer nicht hören will ...



  • @peroja sagte in Anfänger - Programm funktioniert, aber....:

       while ( GetMessageW(&msg,NULL,0,0))
       {
          TranslateMessage(&msg);
          DispatchMessage(&msg);
       }
    

    ist übrigens kaput.



  • / edit: Heck, alleine schon an dem Code oben sieht man wie viel blödsinn Du Dir mit der WinAPI "beibringst". Aber ist ja Dein Bier. Wer nicht hören will ...

    ja? wirklich? weil du seit einigen jahren codest und sicher mehr erfahrung hast als einer der grad mal 2 wochen dabei ist, bedeutet für dich dass du mich von oben herab behandeln kannst? bevor du einen auf lehrer machst, lern gefälligst etwas anstand und respekt. du hast auch mal angefangen.
    auch wenn du recht hast und mein code vielleicht nicht optimal ist, anstatt zu kritisieren kannst du mir freundlich zeigen WO ich WAS falsch mache und erklären warum was wie ist. die anderen konnten es doch auch, oder?
    im übrigen bin ich einem tutorial von einem inder gefolgt. wenn mein kompiler meinen code zwar mit warnungen kompiliert und die exe dann auch macht was sie soll, wie soll ich wissen dass ich etwas falsch mache? ich bin totaler anfänger. und C++ konsolensachen hab ich übrigens n ganzen monat gebastelt und die grundsachen kennen zu lernen. aus einem buch "Grundkurs C" von Jürgen Wolf

    also... mein code ist "kaputt", kann sein. sag mir bitte (freundlich) was kaputt ist und weshalb. und wenn du mir ein beispiel geben könntest wie man es richtig macht, dann wäre ich sogar dankbar und hätte, stell dir mal vor, etwas von dir gelernt.



  • Hör bitte auf mit dieser trotzigen Art.

    Globale Variablen sind gaga und schau Dir an was GetMessage() zurückgibt.

    @peroja sagte in Anfänger - Programm funktioniert, aber....:

    aus einem buch "Grundkurs C" von Jürgen Wolf

    Ja, wegwerfen. (Kannst auch freundlich wegwerfen und dabei lächeln, aber bitte wegwerfen.)



  • @Swordfish sagte in Anfänger - Programm funktioniert, aber....:

    Hör bitte auf mit dieser trotzigen Art.

    Globale Variablen sind gaga und schau Dir an was GetMessage() zurückgibt.

    hab ich jetzt mal nachgesehen und da hat jemand das ganze etwas anders geschrieben. hab ich kopiert und funzt

    while(GetMessage(&msg, NULL, 0, 0) > 0)
    {
        TranslateMessage(&msg);
        DispatchMessage(&msg);
    }
    

    ich versteh es aber nicht was du meinst mit GetMessage() ausgabe? was soll es zurückgeben? und wo sehe ich das?

    @peroja sagte in Anfänger - Programm funktioniert, aber....:

    aus einem buch "Grundkurs C" von Jürgen Wolf

    Ja, wegwerfen. (Kannst auch freundlich wegwerfen und dabei lächeln, aber bitte wegwerfen.)

    du kennst das buch? so schlimm? nun ja ich kann das nicht beurteilen. die beispiele machen für mich (als rookie) sinn.



  • @peroja sagte in Anfänger - Programm funktioniert, aber....:

    hab ich kopiert und funzt

    Ja, so funktioniert C++ nicht.

    @peroja sagte in Anfänger - Programm funktioniert, aber....:

    ich versteh es aber nicht was du meinst mit GetMessage() ausgabe? was soll es zurückgeben? und wo sehe ich das?

    GetMessage()

    @peroja sagte in Anfänger - Programm funktioniert, aber....:

    du kennst das buch? so schlimm?

    Ich kenne den Autor. Schlimm genug.



  • und siehst du auf die schnelle wo ich den fehler mit dem window titel mache? da kommt nur M anstatt My Window



  • die beste art für mich zu lernen ist wenn jemand meinen code korrigiert und // kommentiert
    kompiliert und testets und danach postet.

    ich weiss ist viel verlangt, aber für mich die ideale art zu lernen, weill dann sehe ich was ich falsch gemacht habe

    mann kan ja meine zeilen auch auskommentieren und neu schreiben. dann sehe ich den unterschied

    🙂



  • @peroja sagte in Anfänger - Programm funktioniert, aber....:

    die beste art für mich zu lernen ist wenn jemand meinen code korrigiert und // kommentiert
    kompiliert und testets und danach postet.

    ich weiss ist viel verlangt, aber für mich die ideale art zu lernen, weill dann sehe ich was ich falsch gemacht habe

    mann kan ja meine zeilen auch auskommentieren und neu schreiben. dann sehe ich den unterschied

    🙂

    Die beste Art wäre, vernünftig anzufangen mit C wenn du alte WinApi programmieren möchtest.
    Für C++ gibt es Wrapper wie wxWidgets, Qt, oder alterndes MFC den WinApi32 ist alles andere als C++ und die Basics solltest du schon können bevor du mit Winapi rumhandtierst.

    mFg Ara

    ps: jeder hat mal angefangen doch ohne grundlagen landest du mit jedem code hier 😉



  • @peroja sagte in Anfänger - Programm funktioniert, aber....:

    und siehst du auf die schnelle wo ich den fehler mit dem window titel mache? da kommt nur M anstatt My Window

    Ich würds mal auf das Mischen der A- und W-Varianten und damit char und wchar_t schieben. WinMain etwas umgeschrieben:

    int WINAPI WinMain(HINSTANCE hInst, HINSTANCE hPrevInst, LPSTR args, int ncmdshow)
    {
    	WNDCLASS wc = {0};
    
    	wc.hbrBackground = (HBRUSH)COLOR_WINDOW ;
    	wc.hCursor = LoadCursor(NULL, IDC_ARROW);
    	wc.hInstance = hInst;
    	wc.lpszClassName = TEXT("myWindowClass");
    	wc.lpfnWndProc = WindowProcedure;
    
    	if(!RegisterClass(&wc))
    	  return -1;
    
    	registerDialogClass(hInst);
    
    	CreateWindow(TEXT("myWindowClass"), TEXT("My Window"), WS_OVERLAPPEDWINDOW|WS_VISIBLE,0,0,900,600,NULL,NULL,NULL,NULL);
    
    	MSG msg ={0};
    
    	while(GetMessage(&msg,NULL,0,0))
    	{
    	  TranslateMessage(&msg);
    	  DispatchMessage(&msg);
    	}
    
    	return 0;
    }
    

    SetWindowText/GetWindowText/AppendMenu solte ebenfalls das A rangestellt werden, wenn du mit chars arbeitest. Wie geschrieben, besser wäre es natürlich auf TCHARs umzustellen (siehe oben).

    Und die Messageloop ist völlig okay, wenn weder ein Windowhandle noch ein Rangefilter angegeben wurde. Siehe hier.



  • @peroja sagte in Anfänger - Programm funktioniert, aber....:

    [sinngemäß] schreib meinen code um und versehe ihn mit Kommentaren.

    Sonst geht es Dir noch halbwegs gut?

    @peroja sagte in Anfänger - Programm funktioniert, aber....:

    also... mein code ist "kaputt"

    Ja. Lies die verdammte Dokumentation.

    @yahendrik sagte in Anfänger - Programm funktioniert, aber....:

    	while(GetMessage(&msg,NULL,0,0))
    	{
    	  TranslateMessage(&msg);
    	  DispatchMessage(&msg);
    	}
    

    ist immer noch kaput.



  • @peroja: Ich sehe auch das Mischen von ANSI und WideString als das Problem an.
    Hier ein paar (englische) Artikel dazu:

    Du solltest also (bei WinAPI-Projekten) immer mit "UNICODE" kompilieren (außer du willst noch für Win98 kompilieren ;- )
    Und generell auf die explizite Angabe von A und W verzichten und, wie schon geschrieben, String-Literale mit dem _T- oder TEXT-Makro benutzen. Und dann TCHAR als Datentyp (anstatt explizit char oder wchar_t).



  • Hallo,

    Ich hab mich natürlich nicht Ermutigen lassen und geh unbeirrt meinen Weg. Ich kenne die Grundregeln von C/C++
    Ich bin 2 Wochen lang jeden Tag mehrere Stunden drann gewesen und ich denke schon dass man sich da so einiges Aneignet.
    Ich bleibe bei WIN32 API und ich hab jetzt auch ein bisschen Microsoft Literatur geschmöckert, aber das war ehrlich gesagt nicht sehr Hilfreich. Ich hab mir auch SFML angesehen, was ich Geil finde für Games, aber das möchte ich nicht.
    Das was ich machen will, könnte ich wohl easy in VB coden, jedoch das ist mir wiederum zu "billig". Mich reizen komplizierte herausforderungen. Deshalb wird es bei C++ Win32 bleiben.

    Das Einzige was ich ändere ist meine IDE. Ich werde auf Visual Studio zurückgreifen. CodeBlocks ist geil aber irgendwie fehlt da was.

    Ich hab unterdessen gelernt wie man Radiobuttons und Dropdown Selectoren benutzt und deren Variable übergibt



  • @peroja sagte in Anfänger - Programm funktioniert, aber....:

    Ich kenne die Grundregeln von C/C++
    Ich bin 2 Wochen lang jeden Tag mehrere Stunden drann gewesen und ich denke schon dass man sich da so einiges Aneignet.
    Ich bleibe bei WIN32 API und ich hab jetzt auch ein bisschen Microsoft Literatur geschmöckert,

    Kannst du gern machen, will dich auch gar nicht davon abbringen.
    Ich will dir aber nur aus eigener Erfahrung sagen, dass das bei mir ganz schlecht funktioniert hat. Ich konnte schon andere Programmiersprachen, und dachte mir, C++ ist auch wieder das gleiche... Hab mit MFC angefangen, kaum was verstanden. Und es wurde "nichts" erklärt. Weil C++ eben nicht MFC ist, und MFC Bücher halt auch wenig C++ erklären (sonst wären das schlechte MFC Bücher). Mit der Win API ist das auch so ähnlich.
    Das Ergebnis war, dass ich danach so etwa zehn Jahre auf Kriegsfuß mit C++ stand, bis ich das endlich richtig verstanden hatte.


Log in to reply