[Solved] AccessViolationException when using RenderWindow in C++CLI

[ThreeDumps]

Hello. I have managed class MainApp. Managed class can have only other managed types:

cannot define 'something' as a member of managed 'projekt_name': mixed types are not supported

I have other class (lets call it B) (this class have sf::RenderWindow) and it is normal C++ class. MainApp need to contain B. I can't have B in managed class so i have B* and in MainApp construtor i create new object for this pointer. In B i construct sf::RenderWindow by RenderWindow (WindowHandle handle), handle come from MainApp.

In B construktor I set setMouseCursorVisible( false ); and everything is ok so far.

But when I use display() or clear() evrything is going good but! When I close application i have

An unhandled exception of type 'System.AccessViolationException' occurred in project_name

I first destroy class B, and after that i destroy rest of controls in MainApp (when I change order effect is that same).

What prabobly I made wrong? It's banned to use sf::RenderWindow in object made by new?

[zsbzsb]

Sounds like your destructors are not working properly. You should post a complete and minimal example that causes this.

[ThreeDumps]

I had hope that was fundamental problem. Well... Here is code:

B.h

#pragma once
#include <SFML/Graphics.hpp>
#include <Windows.h>

class B {
        sf::RenderWindow app;
public:
        B( HWND windowHandle ) : app(windowHandle) { }
        void draw() {
                app.clear( sf::Color::White );
                app.display();
        }
};
 

mainApp.h

#pragma once
#include "B.h"

namespace Project {

        using namespace System;
        using namespace System::ComponentModel;
        using namespace System::Collections;
        using namespace System::Windows::Forms;
        using namespace System::Data;
        using namespace System::Drawing;

        /// <summary>
        /// Summary for mainApp
        /// </summary>
        public ref class mainApp : public System::Windows::Forms::Form
        {
                B * b;  
        public:
                mainApp(void)
                {
                        InitializeComponent();
                        b = new B( static_cast<HWND>(panel3->Handle.ToPointer()) );
                }

                void draw() {      
                        b->draw();  
                }                        
        protected:
                /// <summary>
                /// Clean up any resources being used.
                /// </summary>
                ~mainApp()
                {
                        delete b;    
                        if (components)
                        {
                                delete components;
                        }
                }
        private: System::Windows::Forms::Panel^  panel3;


        private:
                /// <summary>
                /// Required designer variable.
                /// </summary>
                System::ComponentModel::Container ^components;

#pragma region Windows Form Designer generated code
                /// <summary>
                /// Required method for Designer support - do not modify
                /// the contents of this method with the code editor.
                /// </summary>
                void InitializeComponent(void)
                {
                        System::ComponentModel::ComponentResourceManager^  resources = (gcnew System::ComponentModel::ComponentResourceManager(mainApp::typeid));
                        this->panel3 = (gcnew System::Windows::Forms::Panel());
                        this->SuspendLayout();
                        //
                        // panel3
                        //
                        this->panel3->Dock = System::Windows::Forms::DockStyle::Fill;
                        this->panel3->Location = System::Drawing::Point(3, 16);
                        this->panel3->Name = L"panel3";
                        this->panel3->Size = System::Drawing::Size(160, 256);
                        this->panel3->TabIndex = 1;
                        //
                        // mainApp
                        //
                        this->AutoScaleDimensions = System::Drawing::SizeF(6, 13);
                        this->AutoScaleMode = System::Windows::Forms::AutoScaleMode::Font;
                        this->ClientSize = System::Drawing::Size(948, 566);
                        this->Name = L"mainApp";
                        this->Text = L"mainApp";
                        this->ResumeLayout(false);
                        this->PerformLayout();

                }
#pragma endregion
        };
}
 

main.cpp

#include "mainApp.h"
using namespace System;

[STAThreadAttribute]
int main()
{
        Project::mainApp mainApp;
        mainApp.Show();

        while (true) {
                System::Windows::Forms::Application::DoEvents();
                mainApp.draw();
        }
    return 0;
}
 

mainApp.h have some code, but most of it is auto generated.

[Ixrec]

Blind guess: Maybe give B a destructor that does app.close()?

[ThreeDumps]

@Ixrec, nope. Still same.

class B {
        sf::RenderWindow app;
public:
        B( HWND windowHandle ) : app(windowHandle) { }
        void draw() {
                app.clear( sf::Color::White );
                app.display();
        }
        ~B() {
                app.close();
        }
};

[zsbzsb]

Without being able to test anything, how about changing

   ~mainApp()
        {
            delete b;    
            if (components)
            {
                delete components;
            }
        }
while (true) {
        System::Windows::Forms::Application::DoEvents();
        mainApp.draw();
    }

to something like

   ~mainApp()
        {
            if (b) delete b;    
            if (components)
            {
                delete components;
            }
        }
while (mainApp.IsOpen()) {
        System::Windows::Forms::Application::DoEvents();
        mainApp.draw();
    }

[ThreeDumps]

@zsbzsb, hit!

while (true) {
        System::Windows::Forms::Application::DoEvents();
        mainApp.draw();
}
 

This loop is still working after mainApp destruction.

Don't know what to do with this topic. Problem solution have nothing common with SFML Well, thank you all for help.

[zsbzsb]

Don't know what to do with this topic. Problem solution have nothing common with SFML Well, thank you all for help.

Just let it sink into the depths of the forums.... And you could also modify to title and add "[Solved]".