1. Diese Seite verwendet Cookies. Wenn du dich weiterhin auf dieser Seite aufhältst, akzeptierst du unseren Einsatz von Cookies. Weitere Informationen

Problem mit dynamischem mehrdimensionalem Array

Dieses Thema im Forum "C/C++" wurde erstellt von Termos, 1. Mai 2012.

  1. Termos

    Termos Mitglied

    Hallo Community,

    Folgendes Problem - Sobald ich ein dynamisches Array in der Funktion "ein" mit werten Fülle spinnt der Destruktor (~zweidim) in der Funktion ... Wo ist der Fehler?

    Code (Text):
    1. #include "stdafx.h"
    2. #include <iostream>
    3. #include <cstring>
    4.  
    5. using namespace std;
    6.  
    7.  
    8. class zweidim
    9. {
    10.     private:
    11.         double **d;
    12.         size_t ax, ay;
    13.     public:
    14.         zweidim(int Zeilen=1, int Spalten=1)
    15.         {
    16.             ax = Zeilen;
    17.             ay = Spalten;
    18.            
    19.             if(ax>0&&ay>0)
    20.             {
    21.                 d = new double*[ax];
    22.                 for(int co=0; co<ax; co++)
    23.                 {
    24.                         d[co] = new double[ay];
    25.                 }
    26.  
    27.                 for(int i=0;i<ax;i++)
    28.                 {
    29.                     for(int n=0;n<ay;n++)
    30.                     {
    31.                         d[i][n]=0.0;                   
    32.                     }
    33.                 }
    34.  
    35.             }
    36.             else
    37.             {
    38.                     cout<<"Falsche Eingabe "<<ax<<","<<ay<<"\n";
    39.                     d = NULL;
    40.             }
    41.         }
    42.        double* operator[] (int ax)
    43.         {
    44.             return & d[ax][ay];
    45.         }
    46.        
    47.  
    48.         ~zweidim()
    49.         {
    50.             int i,n;
    51.             for (i = 0; i < this->ax; i++)
    52.                 {
    53.                         delete [] d[i];
    54.                 }
    55.             delete [] d;
    56.         }
    57.          
    58.  
    59.  
    60.  
    61.         double summe(size_t ax0=0, size_t ax1=0,size_t ay0=0,size_t ay1=0)
    62.         {
    63.             double summe=0;
    64.             double temp;
    65.  
    66.             //test ob ax/y0 > ax/y1
    67.             if(ax0>ax1)
    68.             {
    69.                 temp=ax0;
    70.                 ax0=ax1;
    71.                 ax1=temp;
    72.             }
    73.  
    74.             if(ay0>ay1)
    75.             {
    76.                 temp=ay0;
    77.                 ay0=ay1;
    78.                 ay1=temp;
    79.             }
    80.             //test ende
    81.  
    82.            
    83.  
    84.  
    85.  
    86.                        
    87.             for(int i=0;i<ax1;i++)
    88.                 {
    89.                     for(int n=0;n<ay1;n++)
    90.                     {
    91.                         summe=summe+d[i][n];
    92.                     }
    93.                 }
    94.            
    95.             return summe;
    96.         }
    97.  
    98.                
    99.        
    100.  
    101.        
    102.         //Rückgabe von ax
    103.         size_t get_ax()
    104.         {
    105.             return this->ax;
    106.         }
    107.         //Rückgabe von ay
    108.         size_t get_ay()
    109.         {
    110.             return this->ay;
    111.         }
    112.         //Wert an Position x,y setzen
    113.         void set(size_t ax=0, size_t ay=0, double wert=0.0)
    114.         {
    115.             if(d!=NULL && ax<this->ax && ay<this->ay)
    116.             {
    117.                 cout<<"Es wurde "<<wert<<" an Position "<<ax<<","<<ay<<" eingefeugt\n";
    118.                 d[ax][ay]=wert;
    119.             }
    120.             else
    121.             {
    122.                 cout<<"Wert konnte nicht gesetzte werden, objekt nicht existent oder Position oberhalb des Arrays\n";
    123.             }
    124.  
    125.         }
    126.  
    127.         //Wert holen // Anzeigen
    128.         double get(size_t ax=0, size_t ay=0)
    129.         {
    130.             if(ax>this->ax || ay>this->ay)
    131.             {
    132.                 cout<<"Unzulässige Werte x y \n";
    133.                 return 0;
    134.             }else
    135.             {
    136.                
    137.             return  d[ax][ay];
    138.             }
    139.         }
    140.  
    141.         //Anzeigen Aller Elemente
    142.         void show(size_t ax0=0, size_t ax1=0, size_t ay0=0, size_t ay1=0)
    143.         {
    144.             //test auf 0>1 und auflösen bei fehler
    145.             int temp;
    146.             if(ax0>ax1)
    147.             {
    148.                 temp=ax0;
    149.                 ax0=ax1;
    150.                 ax1=temp;
    151.             }
    152.  
    153.             if(ay0>ay1)
    154.             {
    155.                 temp=ay0;
    156.                 ay0=ay1;
    157.                 ay1=temp;
    158.             }//test ende
    159.            
    160.             //Ausgabe Schleife
    161.             for(int i=0;i<ax1;i++)
    162.                 {
    163.                     for(int n=0;n<ay1;n++)
    164.                     {
    165.                         cout<<d[i][n]<<"\t";
    166.                     }
    167.                     cout<<"\n";
    168.                 }
    169.         }//Ende Show
    170.  
    171.         void ein(size_t ax0=0, size_t ax1=0, size_t ay0=0, size_t ay1=0)
    172.         {
    173.             double wert=0;
    174.                         //test auf 0>1 und auflösen bei fehler
    175.             int temp;
    176.             if(ax0>ax1)
    177.             {
    178.                 temp=ax0;
    179.                 ax0=ax1;
    180.                 ax1=temp;
    181.             }
    182.  
    183.             if(ay0>ay1)
    184.             {
    185.                 temp=ay0;
    186.                 ay0=ay1;
    187.                 ay1=temp;
    188.             }//test ende
    189.  
    190.             for(int i=0;i<ax1;i++)
    191.                 {
    192.                     for(int n=0;n<ay1;n++)
    193.                     {
    194.                         cout<<"Eingabe "<<i<<","<<n;
    195.                         cin>>wert;
    196.                         d[i][n]=wert;
    197.                         cout<<"\n";
    198.                     }
    199.                     cout<<"\n";
    200.                 }
    201.        
    202.  
    203.  
    204.         /*do {
    205.         cout<<"d = "; cin>>d
    206.         if(cin.eof()) break;
    207.         if(cin.fail() || (cin.peek() != '\n')){
    208.         cin.clear(); cin.ignore(INT_MAX,'\n');
    209.         continue;
    210.         }
    211.         break;
    212.         } while(true);
    213.         */
    214.         }
    215. };
    216.  
    217.  
    218. int main()
    219. {
    220.     zweidim arr(3,3);
    221.     cout<<"Ax="<<arr.get_ax()<<"\t Ay="<<arr.get_ay()<<"\n";
    222.     arr.set(1,2,20);
    223.     cout<<"Summe = "<<arr.summe(1,3,1,3)<<"\n";
    224.     arr.ein(1,3,1,3);
    225.     arr.show(1,3,1,3);
    226.  
    227.  
    228.     arr.~zweidim();
    229.     cin.get();
    230.     return 0;
    231. }
     
    Zuletzt bearbeitet: 1. Mai 2012
  2. sheel

    sheel I love Asm Administrator

    Hi

    Ändere
    Code (C++):
    1. double* operator[] (int ax)
    2. {
    3.     return & d[ax][ax];
    4. }
    zu
    Code (C++):
    1. double*&operator[] (int ax)
    2. {
    3.     return d[ax];
    4. }
    Noch was: ruf den Destruktor im main nicht selbst auf, das geht schon automatisch.

    Gruß
     
  3. ibafluss

    ibafluss Erfahrenes Mitglied

    Was genau passiert denn beim Destruktor?

    Vorweg nur 2 Sachen: Beim operator[] übergibst du ein ax, ein ay aber nicht und da es eine Membervariable gibt, die auch so heißt, nimmt die Funktion das ay der class, ist das so beabsichtigt? Ich glaube eher nicht oder?

    Und in der main() rufst du den Destruktor explicit auf, das darfst du nicht machen, er wird vom Compiler am Ende von main() noch einmal aufgerufen und 2mal den Destruktor für dasselbe Objekt verwenden macht Probleme!

    Lg
     
  4. Termos

    Termos Mitglied

    Super vielen dank, hat funktioniert

    jetzt hab ich das Problem das ich beim Füllen der Elemente einen Fehler im Speicher erzeuge ...


    Ist einfach die schleife falsch und ich versuche auf unreservierten speicher zuzugreifen?

    Code (Text):
    1. void ein(size_t ax0=0, size_t ax1=0, size_t ay0=0, size_t ay1=0)
    2.         {
    3.             int wert=0;
    4.                         //test auf 0>1 und auflösen bei fehler
    5.             int temp;
    6.             if(ax0>ax1)
    7.             {
    8.                 temp=ax0;
    9.                 ax0=ax1;
    10.                 ax1=temp;
    11.             }
    12.  
    13.             if(ay0>ay1)
    14.             {
    15.                 temp=ay0;
    16.                 ay0=ay1;
    17.                 ay1=temp;
    18.             }//test ende
    19.  
    20.             for(int i = ax0;i <=ax1;i++)
    21.                 {
    22.                     for(int n = ay0;n <=ay1;n++)
    23.                     {
    24.                         cout<<"Eingabe ["<<i<<"],["<<n<<"] = ";
    25.                         cin>>wert;
    26.                         d[i][n]=wert;
    27.                         cout<<"\n";
    28.                     }
    29.                     cout<<"\n";
    30.                 }
    31.        
    32.  
    33.  
    34.         /*do {
    35.         cout<<"d = "; cin>>d
    36.         if(cin.eof()) break;
    37.         if(cin.fail() || (cin.peek() != '\n')){
    38.         cin.clear(); cin.ignore(INT_MAX,'\n');
    39.         continue;
    40.         }
    41.         break;
    42.         } while(true);
    43.         */
    44.         }
    Code (Text):
    1. int main()
    2. {
    3.     zweidim arr(3,3);
    4.     cout<<"Ax="<<arr.get_ax()<<"\t Ay="<<arr.get_ay()<<"\n";
    5.     arr.set(1,2,20);
    6.     cout<<"Summe = "<<arr.summe(1,3,1,3)<<"\n";
    7.    
    8.     arr.ein(1,3,1,3);
    9.     arr.show(1,3,1,3);
    10.  
    11.    
    12.  
    13.     system("PAUSE");
    14.    
    15.     return 0;
    16. }
     
  5. ibafluss

    ibafluss Erfahrenes Mitglied

    Du erstellst das Objekt mit 3 Zeilen und 3 Spalten, das bedeutet für den Index bei der Arrayschreibweise ist eine Zahl von 0 bis 2 möglich. Du rufst ein() aber mit 1 und 3 auf, und in der Füllschleife in ein() steht i <= ax1, das bedeutet er arbeitet mit Index 3 auch noch, das ist der Fehler.
    Am besten machst du bei dieser Funktion gleich am Anfang eine Überprüfung, ob die Grenzen beim Array nicht überschritten werden.

    Lg
     
  6. Termos

    Termos Mitglied

    So n dummer Fehler ;( Vielen Dank - Wird zeit für Frühstück :(
     
  7. Termos

    Termos Mitglied

    hab es jetzt bis auf die ganzen Fehler abfragen soweit fertig, getestet mit

    3x3 Matrix ... Problem macht immer noch die Ausgabe.
    Code (Text):
    1. /*
    2. Zeilen Anzahl: 3
    3. Spalten Anzahl: 3
    4. Element [3][3] = 0 //Eigentlich sollte hier 100 stehen auf Index [2][2] im Speicher ... in der Tabelle steht es richtig da ... Spreche ich mit meinem Aufruf ein falsches Element an?
    5.  
    6. 0       0       0
    7. 0       0       0
    8. 0       0       100
    9. Eingabe [0],[0] = 1
    10.  
    11. Eingabe [0],[1] = 2
    12.  
    13. Eingabe [0],[2] = 3
    14.  
    15.  
    16. Eingabe [1],[0] = 4
    17.  
    18. Eingabe [1],[1] = 5
    19.  
    20. Eingabe [1],[2] = 6
    21.  
    22.  
    23. Eingabe [2],[0] = 7
    24.  
    25. Eingabe [2],[1] = 8
    26.  
    27. Eingabe [2],[2] = 9
    28.  
    29.  
    30. 1       2       3
    31. 4       5       6
    32. 7       8       9
    33. gesamtsumme = 45
    34. Drücken Sie eine beliebige Taste . . .
    35. */
    Code (Text):
    1. #include "stdafx.h"
    2. #include <iostream>
    3. #include <cstring>
    4.  
    5. using namespace std;
    6.  
    7.  
    8. class zweidim
    9. {
    10.     private:
    11.         double **d;
    12.         size_t ax, ay;
    13.     public:
    14.         zweidim(int Zeilen=1, int Spalten=1)
    15.         {
    16.             ax = Zeilen;
    17.             ay = Spalten;
    18.            
    19.             if(ax>0&&ay>0)
    20.             {
    21.                 d = new double*[ax];
    22.                 for(size_t co=0; co<ax; co++)
    23.                 {
    24.                         d[co] = new double[ay];
    25.                 }
    26.  
    27.                 for(size_t i=0;i<ax;i++)
    28.                 {
    29.                     for(size_t n=0;n<ay;n++)
    30.                     {
    31.                         d[i][n]=0.0;                   
    32.                     }
    33.                 }
    34.  
    35.             }
    36.             else
    37.             {
    38.                     cout<<"Falsche Eingabe "<<ax<<","<<ay<<"\n";
    39.                     d = NULL;
    40.             }
    41.         }
    42.        double *& operator[] (int ax)
    43.         {
    44.             return  d[ax];
    45.         }
    46.        
    47.  
    48.        /* ~zweidim()
    49.         {
    50.             int i;
    51.             for (i = 0; i < ax; i++)
    52.                     delete [] d[i];
    53.             delete [] d;
    54.         }*/
    55.  
    56.         ~zweidim() {
    57.             if(d != NULL) {
    58.                     for(size_t i=0; i < ax; i++)
    59.                     for(size_t j=0; j < ay; j++)
    60.                         if(d[i] != NULL) {
    61.                             delete d[i];
    62.                             d[i] = NULL;
    63.                                 }
    64.                         delete [] d;
    65.                 d = NULL;
    66.                         }
    67.         }
    68.          
    69.  
    70.  
    71.  
    72.         double summe(size_t ax0=0, size_t ax1=0,size_t ay0=0,size_t ay1=0)
    73.         {
    74.             double summe=0;
    75.             size_t temp;
    76.  
    77.             //test ob ax/y0 > ax/y1
    78.             if(ax0>ax1)
    79.             {
    80.                 temp=ax0;
    81.                 ax0=ax1;
    82.                 ax1=temp;
    83.             }
    84.  
    85.             if(ay0>ay1)
    86.             {
    87.                 temp=ay0;
    88.                 ay0=ay1;
    89.                 ay1=temp;
    90.             }
    91.             //test ende
    92.  
    93.            
    94.  
    95.  
    96.  
    97.                        
    98.             for(size_t i=0;i<ax1;i++)
    99.                 {
    100.                     for(size_t n=0;n<ay1;n++)
    101.                     {
    102.                         summe=summe+d[i][n];
    103.                     }
    104.                 }
    105.            
    106.             return summe;
    107.         }
    108.  
    109.                
    110.        
    111.  
    112.        
    113.         //Rückgabe von ax
    114.         size_t get_ax()
    115.         {
    116.             return this->ax;
    117.         }
    118.         //Rückgabe von ay
    119.         size_t get_ay()
    120.         {
    121.             return this->ay;
    122.         }
    123.         //Wert an Position x,y setzen
    124.         void set(size_t ax=0, size_t ay=0, double wert=0.0)
    125.         {
    126.             if(d!=NULL && ax<this->ax && ay<this->ay)
    127.             {
    128.                 d[ax][ay]=wert;
    129.             }
    130.             else
    131.             {
    132.                 cout<<"Wert konnte nicht gesetzte werden, objekt nicht existent oder Position oberhalb des Arrays\n";
    133.             }
    134.  
    135.         }
    136.  
    137.         //Wert holen // Anzeigen
    138.         double get(size_t ax=0, size_t ay=0)
    139.         {
    140.             if(ax>this->ax || ay>this->ay)
    141.             {
    142.                 cout<<"Unzulässige Werte x y \n";
    143.                 return 0;
    144.             }else
    145.             {
    146.                
    147.             return  d[ax][ay];
    148.             }
    149.         }
    150.  
    151.         //Anzeigen Aller Elemente
    152.         void show(size_t ax0=0, size_t ax1=0, size_t ay0=0, size_t ay1=0)
    153.         {
    154.             //test auf 0>1 und auflösen bei fehler
    155.             int temp;
    156.             if(ax0>ax1)
    157.             {
    158.                 temp=ax0;
    159.                 ax0=ax1;
    160.                 ax1=temp;
    161.             }
    162.  
    163.             if(ay0>ay1)
    164.             {
    165.                 temp=ay0;
    166.                 ay0=ay1;
    167.                 ay1=temp;
    168.             }//test ende
    169.            
    170.             //Ausgabe Schleife
    171.             for(size_t i=0;i<ax1;i++)
    172.                 {
    173.                     for(size_t n=0;n<ay1;n++)
    174.                     {
    175.                         cout<<d[i][n]<<"\t";
    176.                     }
    177.                     cout<<"\n";
    178.                 }
    179.         }//Ende Show
    180.  
    181.         void ein(size_t ax0=0, size_t ax1=0, size_t ay0=0, size_t ay1=0)
    182.         {
    183.             int wert=0;
    184.                         //test auf 0>1 und auflösen bei fehler
    185.             int temp;
    186.             if(ax0>ax1)
    187.             {
    188.                 temp=ax0;
    189.                 ax0=ax1;
    190.                 ax1=temp;
    191.             }
    192.  
    193.             if(ay0>ay1)
    194.             {
    195.                 temp=ay0;
    196.                 ay0=ay1;
    197.                 ay1=temp;
    198.             }//test ende
    199.  
    200.  
    201.             if(d!=NULL && ax0>0 && ay0>0)
    202.             {
    203.                 for(size_t i = ax0-1;i <=ax1-1;i++)
    204.                     {
    205.                         for(size_t n = ay0-1;n <=ay1-1;n++)
    206.                         {
    207.                             cout<<"Eingabe ["<<i<<"],["<<n<<"] = ";
    208.                             cin>>wert;
    209.                             d[i][n]=wert;
    210.                             cout<<"\n";
    211.                         }
    212.                         cout<<"\n";
    213.                     }
    214.             }else
    215.             {cout<<"Fehler in Aufruf, ax ay dürfen nciht 0 sein\n";}
    216.  
    217.  
    218.         /*do {
    219.         cout<<"d = "; cin>>d
    220.         if(cin.eof()) break;
    221.         if(cin.fail() || (cin.peek() != '\n')){
    222.         cin.clear(); cin.ignore(INT_MAX,'\n');
    223.         continue;
    224.         }
    225.         break;
    226.         } while(true);
    227.         */
    228.         }
    229. };
    230.  
    231.  
    232. int main()
    233. {
    234.     int ax,ay;
    235.     cout<<"Zeilen Anzahl: ";
    236.     cin>>ax;
    237.     cout<<"Spalten Anzahl: ";
    238.     cin>>ay;
    239.     zweidim d(ax,ay);
    240.     d.set(ax-1,ay-1,100);
    241.     cout<<"Element ["<<ax<<"]["<<ay<<"] = "<<*d[ax-1,ay-1]<<"\n";
    242.     d.show(1,ax,1,ay);
    243.     d.ein(1,ax,1,ay);
    244.     d.show(1,ax,1,ay);
    245.     cout<<"gesamtsumme = "<<d.summe(1,ax,1,ay)<<"\n";
    246.     system("PAUSE");
    247.    
    248.     return 0;
    249. }
     
  8. ibafluss

    ibafluss Erfahrenes Mitglied

    Dein operator[] ist das Problem, er erwartet ja nur ein Argument, du musst den so schreiben, dass er 2 Argumente bekommt und genau das Element der Matrix zurückgibt, im Moment macht er ganz was anderes.

    Lg
     
  9. Termos

    Termos Mitglied

    hast einen tipp nach was ich genau googeln kann?

    habe jetzt alles zur Operator Überladung c++ gelesen, nirgends der Fall das ich 2 Argumente übergeben kann ... muss ich mit zeigern arbeiten um den 2ten wert zu bekommen?


    Oder sollte diese Form weiterhelfen?
    Code (Text):
    1. inline virtual const type& operator[](register const unsigned long index) const { return _pointer[index]; }
    2.       inline virtual type& operator[](register const unsigned long index) { return _pointer[index]; }
     
  10. ibafluss

    ibafluss Erfahrenes Mitglied

    Du könntest entweder den operator () überladen oder versuch es mal mit einem pair. Google mal pair.

    Lg
     
  11. Termos

    Termos Mitglied

    Hab vielen dank
     
  12. sheel

    sheel I love Asm Administrator

    Man kann [] nicht zwei Parameter geben (bzw. wenn man das Klassenobjekt
    mitzählt sinds zwei, aber sonst...)

    Oben hast du schon mal geschrieben, das [] funktioniert hat.
    Warum nicht so lassen?
    Falschzugriffe kommen vom übergebenen Index, da kann [] nichts dafür.
     
  13. Termos

    Termos Mitglied

    Da hatten alle Elemente den Wert 1 ... Ergo falsch getestet ... jetzt bekomme ich bei a[2][2] nur einen speicherwert bzw speichermüll raus ... (Dereferenziert)
     
  14. Termos

    Termos Mitglied

    Okay Fehler umgangen, ich benutze einfach die get() Funktion der Klasse.

    Vielen Dank für eure Hilfe******!