XNA And Windows forms

Question

0.00/5 (No votes)

See more:

I am working on a demo(Demoscene) using XNA, but i want a window to pop out first, where you can select your settings. But when i code a button to Open a new Game1() instance, i get this(InvalidOperationException):

Det er ikke en gyldig operasjon å starte en ny meldingsløkke i en enkelt tråd. Bruk Form.ShowDialog i stedet.

Which translates in english to It is not a valid operation to start a new messageloop(?) in a single thread. use Form.-ShowDIalog instead.

I am using showDialog for my settings form. What needs change?

Source:

C#

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;

namespace Tg12
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
            button1.Text = "Start!";
        }

        private void button1_Click(object sender, EventArgs e)
        {
            Game1 game = new Game1();
            game.Run();
        }
    }
}

Program.cs:

C#

using System;

namespace Tg12
{
#if WINDOWS || XBOX
    static class Program
    {
        /// <summary>
        /// The main entry point for the application.
        /// </summary>
        static void Main(string[] args)
        {
            Form1 form = new Form1();
            form.ShowDialog();
        }
    }
#endif
}

Posted 21-Dec-11 4:07am

Liam S. Crouch

Updated 21-Dec-11 4:39am

v2

Add a Solution

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

LanFanNinja · Answer 1 · 2011-12-21T05:06:00

Solution 1

Well I am not sure what you are doing is a good idea but the only way I can see to do this right now in my tired state of mind is to start a new thread. i.e.

C#

private void button1_Click(object sender, EventArgs e)
{
    System.Threading.Thread thStartGame = new System.Threading.Thread(StartGame);
    thStartGame.Start();
}

private void StartGame()
{
    Game1 game = new Game1();
    game.Run();
}

Posted 21-Dec-11 5:06am

LanFanNinja

Updated 21-Dec-11 5:08am

v2

Comments

Liam S. Crouch 21-Dec-11 11:56am

I get
The best overloaded method match for 'System.Threading.Thread.Thread(System.Threading.ParameterizedThreadStart)' has some invalid arguments

and Cannot convert from 'void' to 'System.Threadind.parameterizedThreadStart'

LanFanNinja 21-Dec-11 12:57pm

Show me your code.

Liam S. Crouch 21-Dec-11 13:01pm

Form1.cs

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
using System;
using System.Threading;

namespace Tg12
{
public partial class Form1 : Form
{
public Form1()
{
InitializeComponent();
button1.Text = "Start!";
}

private void button1_Click(object sender, EventArgs e)
{
Game1 ogame = new Game1();
Thread theThread = new Thread(new ThreadStart(StartGame()));
}
public void StartGame()
{
Game1 game = new Game1();
game.Run();
}
}
}

Program.cs

using System;

namespace Tg12
{
#if WINDOWS || XBOX
static class Program
{
///

/// The main entry point for the application.
///

static void Main(string[] args)
{
Form1 form = new Form1();
form.ShowDialog();
}
}
#endif
}

LanFanNinja 21-Dec-11 13:42pm

Use this code exactly as it is written.

private void button1_Click(object sender, EventArgs e)
{
//Game1 ogame = new Game1();

Thread theThread = new Thread(StartGame);
// OR
//Thread theThread = new Thread(new ThreadStart(StartGame));

theThread.Start();
}

public void StartGame()
{
Game1 game = new Game1();
game.Run();
}

LanFanNinja 21-Dec-11 13:45pm

Note the lack of parenthesis from StartGame.

Your code: Thread theThread = new Thread(new ThreadStart(StartGame()));

My code: Thread theThread = new Thread(new ThreadStart(StartGame));
or
My code: Thread theThread = new Thread(StartGame);

EDIT:
Also in your button1_Click event handler you are creating an instance of Game1 called "ogame" this is not needed an instance of Game1 is being created in the StartGame() method.

In order for you to be able to create "ogame" in button1_Click and the run it in StartGame() would require more work as there would be a cross thread error.

If you need access to the game instance outside of StartGame() you will need to make it global. i.e. something like this:

public partial class Form1 : Form
{
Game1 ogame;

public Form1()
{
InitializeComponent();
button1.Text = "Start!";
}

private void button1_Click(object sender, EventArgs e)
{
Thread theThread = new Thread(StartGame);
theThread.Start();
}

public void StartGame()
{
ogame = new Game1();
ogame.Run();
}
}

Liam S. Crouch 21-Dec-11 15:06pm

I fixed it, thanks for thew help!

LanFanNinja 21-Dec-11 15:25pm

You're welcome.