Classes Matrix question

[Hypnotik]

Hello everyone. I'm writing a program which uses a class called matrix. I have written all of the different functions, constructor, etc.

When I run the program I receive "Constructo r", which I placed in the constructor, and then the program crashes. I have no clue where my problem is.

The matrix is for size 2x2 up to 10x10, and it must be square.

Below is the code:

#include <iostream>
#include <ctype.h>
using namespace std;
int innerprod(int s[][10],int t[][10],int r,int c,int size)
{
    int temp=0,i;
    for (i=0;i<=size;++i)
    {
        temp=temp+s[r][i]*t[i][c];
    }
    return temp;
}
class Matrix
{
public:
    ~Matrix(){};
    Matrix();
    void readm();
    void addm(Matrix);
    void multm(Matrix);
    void print();
private:
    int a[10][10];
    int size;
};
Matrix::Matrix()
{
    cout<<"Constructor"<<endl;
    int i=0, j=0;
    for (i=0;i<=10;++i)
        for (j=0;j<=10;++i)
        {
            a[i][j]=0;
        }
}
void Matrix::readm()
{
    int i=0,j=0;
    for (i=0;i<=10;++i)
        for (j=0;j<=10;++j)
        {
            cin>>a[i][j];
        }
}
void Matrix::addm(Matrix B)
{
    int i=0,j=0;
    for (i=0;i<=10;++i)
        for (j=0;j<=10;++j)
        {
            a[i][j]=a[i][j]+B.a[i][j];
        }
}
void Matrix::multm(Matrix B)
{
    int i=0,j=0,temp[10][10],n=2;
    for(i=0;i<=10;++i)
        for (j=0;j<=10;++j)
        {
            temp[i][j]=innerprod(a,B.a,i,j,n);
        }
    for (i=0;i<=10;++i)
        for (j=0;j<=10;++j)
        {
            a[i][j]=temp[i][j];
        }
}
void Matrix::print()
{
    int i=0,j=0;
    for (i=0;i<=10;++i)
    {
        cout<<endl<<endl;
        for (j=0;j<=10;++j)
        {
            cout<<"    "<<a[i][j];
        }
    }
    cout<<endl;
}
void main()
{
    char choice;
    Matrix x,y;
    cout<<"   Welcome to Matrix Manipulations."<<endl;
    x.print();
    cout<<"   You may add or multiply a matrix."<<endl;
    cout<<"   Enter choice: (a)dd or (m)multiply.  "<<endl;
    cin>>choice;
    choice>>tolower(choice);
    cout<<"   Enter a square matrix from 2x2 to 10x10 in row order:"<<endl;
    x.readm();
    switch(choice)
    {
    case 'a':cout<<"Enter a square matrix from size 2x2 to 10x10 in row order:"<<endl;
        y.readm();
        x.print();
        cout<<endl<<"    +";
        y.print();
        cout<<endl<<"    =";
        x.addm(y);
        x.print();
        break;
    case 'm':cout<<"Enter a square matrix from size 2xe to 10x10 in row order:"<<endl;
        y.readm();
        x.print();
        x.multm(y);
        cout<<endl<<"    *";
        y.print();
        cout<<endl<<"    =";
        x.print();
        break;
    default: cout<<"    INVALID CHOICE!";
        break;
    };
}
 
 

Thanks,

J

[JosAH]

Your loops all have an off by one error. When the maximum dimension of an array
equals 10, valid index values are 0, 1, 2, 3, 4, 5, 6 , 7, 8, and 9 (count them:
there are 10 different index values). Therefore your loops should all have a
terminating condition such as:

for (int i= 0; i < 10; i++) ...
 

kind regards,

Jos

[Hypnotik]

Back to my rescue again hun JosAH?

I understand what you're saying, and that is right...I wasn't thinking when I made them less than or equal to 10. The program still crashes, so the functions aren't going out of scope (or whatever it's called).

J

[JosAH]

Back to my rescue again hun JosAH?

I understand what you're saying, and that is right...I wasn't thinking when I made them less than or equal to 10. The program still crashes, so the functions aren't going out of scope (or whatever it's called).

J

I took a bit closer look at your code: have a look at the inner loop in your ctor:
it increments 'i' instead of 'j' and it will shoot you in your foot for it ;-)

kind regards,

Jos

[Hypnotik]

Awesome. I never used 'i' and 'j' before because they look so much alike! But seems that my instructor uses them so I've started.

I also removed the destructor function...whic h stopped the program from crashing. Do you know why that would be?

J

[Hypnotik]

I know need to set the size of my matrix to the input from the user from the default size of the matrix (the constructor). Do I need to create an entirely differnent function for this?

something like:

void SetMatrix() ?

[Hypnotik]

I know need to set the size of my matrix to the input from the user from the default size of the matrix (the constructor). Do I need to create an entirely differnent function for this?

something like:

void SetMatrix() ?

Is passing the size of the array to the functions just like passing any other info to a function?

Thanks,
J

[Hypnotik]

Well I have found my problem, however there is a problem with my multiplication.

J

[JosAH]

Well I have found my problem, however there is a problem with my multiplication.

J

What's the problem then? And please don't say "it produces the wrong results" ;-)

kind regards,

Jos

[Hypnotik]

Hah. Well it does give the wrong answer. Anyway, I think the problem is in the innerprod function. When I do addition the program works great. When I do the multiplication it returns a HUGE number for a matrix with all 1's as the input. I pass all types of information to the innnerprod function including both matrices and size. The size is correct, however the numbers within the matrix are incorrect.

I'm going to check my code and make sure the one posted in the thread is up to date.


Thanks,
J