ok where is this left bracket

[thebigrobo]

OK codes going under this under the code will be the one fatal error im using Microsoft visual c++ 2010.

#include <vector>
#pragma once




using namespace System;
using namespace System::ComponentModel;
using namespace System::Collections;
using namespace System::Windows::Forms;
using namespace System::Data;
using namespace System::Drawing;
int fillrectangle;
int xcos = 160;

int bullet, bullets;

struct bullets{
int x;
int y;
int speed;
void act(){y=speed;}
};

/// <summary>
/// Summary for Form1
/// </summary>
public ref class Form1 : public System::Windows::Forms::Form
{

public:
Form1(void)
{
InitializeComponent();
//
//TODO: Add the constructor code here
//
}

protected:
/// <summary>
/// Clean up any resources being used.
/// </summary>
~Form1()
{
if (components)
{
delete components;
}
}
private: System::Windows::Forms::PictureBox^ starship;
private: System::Windows::Forms::PictureBox^ bullets;
protected: 







protected: 

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(Form1::typeid));
this->starship = (gcnew System::Windows::Forms::PictureBox());
this->bullets = (gcnew System::Windows::Forms::PictureBox());
(cli::safe_cast<System::ComponentModel::ISupportInitialize^ >(this->starship))->BeginInit();
(cli::safe_cast<System::ComponentModel::ISupportInitialize^ >(this->bullets))->BeginInit();
this->SuspendLayout();
// 
// starship
// 
this->starship->Image = (cli::safe_cast<System::Drawing::Image^ >(resources->GetObject(L"starship.Image")));
this->starship->Location = System::Drawing::Point(232, 385);
this->starship->Name = L"starship";
this->starship->Size = System::Drawing::Size(164, 245);
this->starship->TabIndex = 1;
this->starship->TabStop = false;
this->starship->Click += gcnew System::EventHandler(this, &Form1::starship_Click);
// 
// bullets
// 
this->bullets->Image = (cli::safe_cast<System::Drawing::Image^ >(resources->GetObject(L"bullets.Image")));
this->bullets->Location = System::Drawing::Point(230, 303);
this->bullets->Name = L"bullets";
this->bullets->Size = System::Drawing::Size(165, 74);
this->bullets->TabIndex = 2;
this->bullets->TabStop = false;
this->bullets->Click += gcnew System::EventHandler(this, &Form1::bullets_Click);
// 
// Form1
// 
this->AutoScaleDimensions = System::Drawing::SizeF(6, 13);
this->AutoScaleMode = System::Windows::Forms::AutoScaleMode::Font;
this->ClientSize = System::Drawing::Size(632, 642);
this->Controls->Add(this->bullets);
this->Controls->Add(this->starship);
this->Name = L"Form1";
this->Text = L"Form1";
this->KeyPress += gcnew System::Windows::Forms::KeyPressEventHandler(this, &Form1::keyDown);
(cli::safe_cast<System::ComponentModel::ISupportInitialize^ >(this->starship))->EndInit();
(cli::safe_cast<System::ComponentModel::ISupportInitialize^ >(this->bullets))->EndInit();
this->ResumeLayout(false);

}
#pragma endregion
private: System::Void pictureBox1_Click(System::Object^ sender, System::EventArgs^ e) {
}




private: System::Void keyDown(System::Object^ sender, System::Windows::Forms::KeyPressEventArgs^ e) {
if ((e->KeyChar == 'd')&&(xcos<450))
{
xcos+=5;
starship->Location = System::Drawing::Point (xcos, 399);
}
if ((e->KeyChar=='a')&&(xcos>25))
{
xcos -=5;
starship->Location = System::Drawing::Point(xcos, 399);
}
}
private: System::Void starship_Click(System::Object^ sender, System::EventArgs^ e) {
}
private: System::Void bullets_Click(System::Object^ sender, System::EventArgs^ e) {
for (1=0:1<bullets.size();1++)
{
(G> fillrectangle (brushes::red, bullets[i].x, bullets[i]y, 10, 10));

} 
for (1=0:1<bullets.size();1++)
{
Bullets[I].act();
if (bullets[i].y<o)
{
Bullets.erase(1);
}



};

now for the error
1>------ Build started: Project: space game, Configuration: Debug Win32 ------
1> space game.cpp
1>space game.cpp(19): fatal error C1075: end of file found before the left brace '{' at 'c:\users\spencer\desktop\c++\space game\space game\Form1.h(139)' was matched
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========

[ne555]

If you indent your code properly, you'll see the error clearly

[S G H]

This should go under Windows programming.
This isn't regular C++.
But anyways, yes, you should copy-paste indented code.

[thebigrobo]

niether of those help me im a beginner wtf is my error

[S G H]

You should indent your code.

http://indentcode.net/

Try this.
Copy-paste your code and press Send.

You should be able to see how, in the last function, you're missing some "}" (right brace) somewhere.


And...
for (1=0:1<bullets.size();1++)
Did you try questioning yourself how is this supposed to be working?

[ats15]

It is your last for loop that has unbalanced curly brackets, Your open curly bracket is on line 146, the if statement has balanced ones, and the closing curly bracket on line 155 you would think it's from the Void bullets_Click function, but you did not close the curly bracket for the for loop.

[jumper007]

Yep, as @EssGeEich mentioned:

for (1=0:1<bullets.size();1++)

I guess you wanted to write this instead:

for (int i=0:i<bullets.size();i++)

You cannot say 1 = 0 . 1 is a number. There, you need a variable, let's say i. Try changing that on both line 140 and 145.

[MikeyBoy]

niether of those help me

Yes, they do help you. If you adopted a sensible indentation style, you would find it much easier to see where you've missed out a closing brace. It also helps you see the flow of control through your code much more easily.

Not only does their advice help you find the problem you're currently having, it will help you not make similar mistakes in future, and when you do, it will help you find them.

[Lachlan Easton]

I guess you wanted to write this instead:

for (int i=0:i<bullets.size();i++)

First colon should be a semicolon; for (int i=0;i<bullets.size();++i)

[jumper007]

>

@Lachlan Easton
First colon should be a semicolon; for (int i=0;i<bullets.size();++i)

Yes indeed. I just copy/pasted his code and yeah, I missed that one. :)