Fill 2 matrices arrays

[Mido14]

What is wrong with That code.The Program has Stopped Working !

  #include <iostream>
#include<string.h>

using namespace std;



int main()
{

    int **Matrix1,**Matrix2;
    int Rows,Cols;
    cout<<"Number of Rows : ";
    cin>>Rows;
    cout<<"Number of Columns : ";
    cin>>Cols;

    Matrix1= new int*[Rows];
    Matrix2= new int*[Rows];

    for(int i=0;i<Rows;i++){
        Matrix1[i]= new int[Cols];}

        cout<<"Matrix #1 : "<<endl;

    for(int i=0;i<Rows;i++){
        for(int j=0;j<Cols;j++)
        cin>>Matrix1[i][j];}

         cout<<"Matrix #2 : "<<endl;

         for(int i=0;i<Rows;i++){
        for(int j=0;j<Cols;j++)
        cin>>Matrix2[i][j];}



        for(int i=0; i<Rows;i++)
        {
            delete []Matrix1[i];

        }
            delete []Matrix1;
            for(int i=0; i<Rows;i++)
        {
            delete []Matrix2[i];

        }
            delete []Matrix2;



    return 0;
}

[PrivateRyan]

Matrix1 and Matrix2 are two dimensional arrays. You initialize the rows in both of them but forgot to initialize the columns in Matrix2.

for(int i=0;i<Rows;i++) //You added this first one
{
      Matrix1[i]= new int[Cols];
}

 for(int i=0;i<Rows;i++) //But you forgot this one
{
      Matrix2[i]= new int[Cols];
}

It helps when brackets are always used with loops even when there is only one statement being executed. Helps things be more readable.

Hope this helps!

[Mido14]

Thank You very much...This is very helpful to me