2-dimensionales Feld c++/cli

[Anti-Programmierer (Gast)]

Hey,
habe ein Problem mit einem 2-dimensionalen Integer-Feld.
Ich deklariere das Array:

array<int ^, 2> ^ Spielstein;

In der Form1 lege ich das Array dann an:

Spielstein = gcnew array<int ^, 2>(8, 8);

Nun möchte ich in ein bestimmtes Feld eine "1" schreiben:

Buttons[i + 1,j]->BackgroundImage = Image::FromFile("Bilder\\Gelb.jpg");
   Spielstein[i +1,j] = 1;

Gleichzeitig liegt dahinter ein Button-Array, das mir ein Spielfeld 
simuliert. Ist also das Button-Hintergrundbild "Gelb" dann soll ne 1 
stehen.

Für das Button-Feld habe ich einen gemeinsamen Event-Handler. I und J 
sind auch richtig jedoch bei der Abfrage

if(Spielstein[i,j] == 1)

komme ich nie rein...
i und j stimmen, es müsste also eine 1 im Feld stehen....

mfg

[Frank M. (aktenasche)]

poste mal mehr code

[Anti-Programmierer (Gast)]

Das ist eig mein ganzer Code:

  private: System::Void Form1_Load(System::Object^  sender, System::EventArgs^  e) {

      this->Location = Point(100,20);
      this->Size = System::Drawing::Size(1116,666);
      Spielfeld->Size = System::Drawing::Size(600,600);
      Spielfeld->Location = Point(0,27);

      Spielstein = gcnew array<int ^, 2>(8, 8);
      Buttons = gcnew array<Windows::Forms::Button ^, 2>(8, 8);
      

      for(int i = 0; i < 8; i++)
        {
          for(int j = 0; j < 8; j++)
          {
            Buttons[i,j] = gcnew Windows::Forms::Button();
            Spielstein[i,j] = gcnew int;
            // Spielstein 1: Gelber Spielstein
            // Spielstein 2: Gelbe Dame
            // Spielstein 3: Roter Spielstein
            // Spielstein 4: Rote Dame

            Buttons[i,j]->Size = System::Drawing::Size(75,75);
            Buttons[i,j]->FlatStyle = FlatStyle::Flat;
            Buttons[i,j]->Margin = System::Windows::Forms::Padding(0);
            Buttons[i,j]->FlatAppearance->BorderSize = 0;
            Buttons[i,j]->BackgroundImageLayout = ImageLayout::Stretch;
            Buttons[i,j]->Click += gcnew EventHandler(this, &Form1::Buttons_Clicked);

            Spielfeld->Controls->Add(Buttons[i,j], i, j);
      
            if(i % 2 == 0 && j % 2 == 0 || i % 2 == 1 && j % 2 == 1)
              Buttons[i,j]->BackColor = Color::Black;
            else
              Buttons[i,j]->BackColor = Color::White;
          }
        }
       }
  private: System::Void Buttons_Clicked(Object ^sender, EventArgs ^e) {
        
        Button ^clickedButton = gcnew Button();
        clickedButton = dynamic_cast<Button^>(sender);

        for(int i = 0; i < 8; i++)
        {
          for(int j = 0; j < 8; j++)
          {
            if(clickedButton == Buttons[i,j])
            {
              if(zug == 1)
              {
                if(Spielstein[i,j] == 1)
                {
                  this->Text = "Hi";
                }
                else
                {
                  this->Text = Convert::ToString(i);
                }
              }
            }
          }
        }
          // Spiel spielen
          // Spielmechanik: Wenn Spieler 1 am Zug ist und einen Spielstein auswählt:
          // Dann abfragen ob es ein richtiger Button ist.
          // Wenn ja, dann abfragen ob das angewählte Feld, das voraus liegt
          // Leer ist, bzw das 2fach entfernte Feld leer ist und dazwischen das Button-Hintergrundbild
          // des anderen Spielers ist. Wenn ja, dann ist Zug gültig. das alte Feld wird nullptr 
          // gesetetz und wenn ein Gegner dazwischen lag, wird dieser entfernt. 
          // OPTIONAL: Abfrage ob von diesem Spielstein ein weiterer gegnerischer Spielstein
          //       übersprungen werden kann, wenn ja, dann darf nochmal gesprungen werden.
       }
  private: System::Void neuesSpielToolStripMenuItem_Click(System::Object^  sender, System::EventArgs^  e) {

         // Spielfeld initialisieren

         for(int i = 0; i < 8; i = i + 2)
         {
          for(int j = 0; j < 3; j++)
          {
            if(j == 1)
            {
              Buttons[i + 1,j]->BackgroundImage = Image::FromFile("Bilder\\Gelb.jpg");
              Spielstein[i +1,j] = 1;
            }
            else
            {
              Buttons[i,j]->BackgroundImage = Image::FromFile("Bilder\\Gelb.jpg");
              Spielstein[i,j] = 1;
            }
          }
            Buttons[i,4]->BackgroundImage = nullptr;
            Buttons[i + 1,3]->BackgroundImage = nullptr;
          for(int j = 5; j < 8; j++)
          {
            if(j == 6)
            {
              Buttons[i,j]->BackgroundImage = Image::FromFile("Bilder\\Rot.jpg");
              Spielstein[i,j] = 3;
            }
            else
            {
              Buttons[i + 1,j]->BackgroundImage = Image::FromFile("Bilder\\Rot.jpg");
              Spielstein[i +1,j] = 3;
            }
          }
         }
         zug = 1;
         freigabe = true;
       }

[Thaala (Gast)]

Das hier ist es meiner Meinung nach:

      Spielstein = gcnew array<int ^, 2>(8, 8);

(Das ist ein zweidimensionales Array auf den Typ intptr!)
...

           Spielstein[i,j] = gcnew int;

Besser wäre:

      Spielstein = gcnew array<int, 2>(8, 8);

(Das ist ein zweidimensionales Array auf den Typ int!, der Speicher für 
die 64 Werte ist bereits reserviert!)


kommst Du denn hier überhaupt rein ?

                if(Spielstein[i,j] == 1)
                {
                  this->Text = "Hi";
                }
                else
                {
                  this->Text = Convert::ToString(i);
                }

Hier würde das genügen:

private: System::Void Buttons_Clicked(Object ^sender, EventArgs^ e)         
        Button ^clickedButton = dynamic_cast<Button^>(sender);

Gruß T.