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 Grünschnabel

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

    sheel Mod # I love Asm Moderator

    Hi

    Ändere
    Code (C++):
    1.  
    2. double* operator[] (int ax)
    3. {
    4.     return & d[ax][ax];
    5. }
    6.  
    zu
    Code (C++):
    1.  
    2. double*&operator[] (int ax)
    3. {
    4.     return d[ax];
    5. }
    6.  
    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 Grünschnabel

    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. }
    17.  
  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 Grünschnabel

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

    Termos Grünschnabel

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

    3x3 Matrix ... Problem macht immer noch die Ausgabe.

    /*
    Zeilen Anzahl: 3
    Spalten Anzahl: 3
    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?

    [/COLOR]
    0 0 0
    0 0 0
    0 0 100
    Eingabe [0],[0] = 1

    Eingabe [0],[1] = 2

    Eingabe [0],[2] = 3


    Eingabe [1],[0] = 4

    Eingabe [1],[1] = 5

    Eingabe [1],[2] = 6


    Eingabe [2],[0] = 7

    Eingabe [2],[1] = 8

    Eingabe [2],[2] = 9


    1 2 3
    4 5 6
    7 8 9
    gesamtsumme = 45
    Drücken Sie eine beliebige Taste . . .
    */


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

    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 Grünschnabel

    Hab vielen dank
  12. sheel

    sheel Mod # I love Asm Moderator

    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 Grünschnabel

    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 Grünschnabel

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

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

Diese Seite empfehlen