c++ 2D-Array einfacher Denkfehler

[anfänger (Gast)]

Hallo,
habe vor kurzen mit Programmieren angefangen. Begonnen habe ich mit der 
Sprache c++. Nun hängt es an einem 2D-array, obwohl ich dachte, dass ich 
das schon begriffen hätte. Also schätze ich muss es ein einfacher 
anfänger-/denk-fehler sein.
Folgender Code ist das Problem:

#include <iostream>
#include <cstdio>
# include <string>
#include <conio.h>
#include <dos.h>
using namespace std;
int main()
{
    int feld[3][3];
    feld[0][3]=3;
    feld[1][0]=1;
    cout<<feld[0][3];

    system("PAUSE");
    cin.clear();
    cin.ignore(cin.rdbuf()->in_avail());
    cin.get();
    return 0;
}
Er gibt mir dann "1" aus. Wenn ich // vor feld[1][0]=1; schreibe gibt er 
mir "3" aus.
Diesen Zusammenhang verstehe ich nicht.
Danke für eure Hilfe

[Rufus Τ. F. (rufus)]

anfänger schrieb:
> int feld[3][3];
>     feld[0][3]=3;

Du verwendest ein Array mit drei Elementen, greifst aber auf das 
vierte Element zu. Das erste Element ist nicht [1], sondern [0].

[Tom (Gast)]

anfänger schrieb:
> int feld[3][3];
>     feld[0][3]=3;

Wie groß ist feld? Welches ist daraus folgend der kleinste und der 
größte Index?

[anfänger (Gast)]

ah ok. da war der fehler. danke.
hatte irgendwie im kopf feld[3][3] wäre jeweils von 0-3 aber heißt ja 
0-2^^.
danke

[Dennis S. (eltio)]

Davon abgesehen wäre es sinnvoll sich zunächst auf den Standard zu 
konzentrieren. Das heißt die "conio.h" und "dos.h" wegzulassen. Wenn du 
dann noch alles in der Konsole machst, dann brauchst du auch deinen 
"system("PAUSE");"-Befehl nicht mehr.

Desweiteren solltest du dir bewusst sein, dass C und C++ 
unterschiedliche Sprachen sind. Du bindest nämlich eine aus der C-Welt 
kommende Header-Datei ein ("cstdio") die für C++ "angepasst" wurde.

Zuguterletzt: wenn du Quellcode postest, solltest du entsprechende Tags 
nutzen um die Lesbarkeit zu erhöhen.

Gruß
Dennis

[anfänger (Gast)]

Okay. danke.
wo wir gerade dabei sind - bei Funktionen verstehe ich das nicht mit den 
globalen variablen.
Ich bekomme da eine Fehlermeldung bei cout<<feld[i][o]<<" "; da ich feld 
dort noch nicht definiert habe. ich meine aus dem informatikuntericht 
von vor ein paar Jahren, dass man sowas mit public int löst oder so. 
aber hier bekomme ich das nicht auf die kette.
und wenn ich mir videos/tutorials zu funktionen angucke geht es nie um 
int's oder arrays die von einer anderen funktion ausgegeben werden 
sollen.
danke für eure hilfe

#include <iostream>
# include <string>
using namespace std;
int erzeugen()
{
    for(int i =0;i<=3;i++)
    {
         for (int o=0;o<=3;o++)
         {
              cout<<feld[i][o]<<" ";
         }
         cout<<"\n";

}
int main()
{
    public int feld[4][4];
    feld[0][0]=0;
    feld[0][1]=1;
    feld[0][2]=2;
    feld[0][3]=3;
    feld[1][0]=1;
    feld[1][1]=0;
    feld[1][2]=0;
    feld[1][3]=0;
    feld[2][0]=2;
    feld[2][1]=0;
    feld[2][2]=0;
    feld[2][3]=0;
    feld[3][0]=3;
    feld[3][1]=0;
    feld[3][2]=0;
    feld[3][3]=0;
    erzeugen();
   
    cout<<"\n\n\n\n\n\n"<<endl;
    cin.clear();
    cin.ignore(cin.rdbuf()->in_avail());
    cin.get();
    return 0;
}

[Mikro 7. (mikro77)]

C/++ ist nicht die einfachste Sprache um "anzufangen". An einem guten 
Buch wirst du kaum drum herum kommen.

Zu deinem Problem: "feld" ist eine lokale Variable, auf die nur im 
lokalen Kontext zugegriffen werden kann; also hier nur in main().

Damit die Funktion "erzeugen" darauf zugreifen kann, mußt du das Feld an 
die Funktion übergeben. Das geht grundsätzlich entweder "by-value" oder 
"by-reference". Arrays sind ein doppelter Sonderfall. Da geht es nur 
"by-reference" (was du hier aber sowieso brauchst). Und um die 
Feld-Größe zu übergeben, braucht es eine "spezielle" Syntax:

#include <iostream>
#include <string>

using namespace std;

void erzeugen(int (*feld)[4][4])
{
    for (int i=0;i<=3;i++)
    {
         for (int o=0;o<=3;o++)
         {
     cout<<(*feld)[i][o]<<" ";
         }
         cout<<"\n";
    }
}
int main()
{
    int feld[4][4];
    feld[0][0]=0;
    feld[0][1]=1;
    feld[0][2]=2;
    feld[0][3]=3;
    feld[1][0]=1;
    feld[1][1]=0;
    feld[1][2]=0;
    feld[1][3]=0;
    feld[2][0]=2;
    feld[2][1]=0;
    feld[2][2]=0;
    feld[2][3]=0;
    feld[3][0]=3;
    feld[3][1]=0;
    feld[3][2]=0;
    feld[3][3]=0;
    erzeugen(&feld);
   
    cout<<"\n\n\n\n\n\n"<<endl;
    cin.clear();
    cin.ignore(cin.rdbuf()->in_avail());
    cin.get();
    return 0;
}

Alternativ deklariest du "feld" global bevor es irgendwo benutzt wird. 
Globale Definitionen sollten allerdings mit Bedacht eingesetzt werden, 
da sie zu schwer wartbaren Code führen können.

#include <iostream>
#include <string>

using namespace std;

int feld[4][4];

void erzeugen()
{
    for (int i=0;i<=3;i++)
    {
         for (int o=0;o<=3;o++)
         {
     cout<<feld[i][o]<<" ";
         }
         cout<<"\n";
    }
}
int main()
{
    feld[0][0]=0;
    feld[0][1]=1;
    feld[0][2]=2;
    feld[0][3]=3;
    feld[1][0]=1;
    feld[1][1]=0;
    feld[1][2]=0;
    feld[1][3]=0;
    feld[2][0]=2;
    feld[2][1]=0;
    feld[2][2]=0;
    feld[2][3]=0;
    feld[3][0]=3;
    feld[3][1]=0;
    feld[3][2]=0;
    feld[3][3]=0;
    erzeugen();
   
    cout<<"\n\n\n\n\n\n"<<endl;
    cin.clear();
    cin.ignore(cin.rdbuf()->in_avail());
    cin.get();
    return 0;
}