C++ Multidimensional array program shows garbage value in output

Question

I am writing this simple code for setting a matrix ourselves and displaying it. When I execute this program, it gives garbage value on the first row. How is that? Any errors in my program ?

#include<iostream>
using namespace std;

void setOneMatrix();
//void getOneMatrix(int mat[6][5]);
int display(int mat[6][5]);

int main() {

int setMat[6][5]={};
setOneMatrix();
display(setMat);


}

void setOneMatrix() {
/*int setMat[6][5] = {1,2,3,4,5,
                    6,7,8,9,10,
                    11,12,13,14,15,
                    16,17,18,19,20,
                    21,22,23,24,25,
                    26,27,28,29,30};*/

int setMat[6][5] = {{1,2,3,4,5},
                    {6,7,8,9,10},
                    {11,12,13,14,15},
                    {16,17,18,19,20},
                    {21,22,23,24,25},
                    {26,27,28,29,30}};

}

int display(int mat[6][5]) {
int i,j,setMat[6][5];
for(i=0;i<6;i++){
    for(j=0;j<5;j++) {
        cout << setMat[i][j] << "\t";
    }
    cout << endl;
}
}

Output:

4665744 4687848 6946296 4257625 0
1   2   3   4   5
6   7   8   9   10
11  12  13  14  15
16  17  18  19  20
21  22  23  24  25

Rakete1111 · Accepted Answer · 2016-07-10 04:02:02Z

3

Sorry to say that, but your whole program has undefined behavior! This is my output:

-858993460      -858993460      -858993460      -858993460      -858993460
-858993460      -858993460      -858993460      -858993460      -858993460
-858993460      -858993460      -858993460      -858993460      -858993460
-858993460      -858993460      -858993460      -858993460      -858993460
-858993460      -858993460      -858993460      -858993460      -858993460
-858993460      -858993460      -858993460      -858993460      -858993460

You are just lucky that the numbers get printed, except for the first row. That shouldn't happen, see my output.

Some facts:

setMat in main contains just 0.
setOneMatrix doesn't initialize setMat in main, it just initializes another setMat. This function basically does nothing.
displayMath is a bit better, because you are passing setMat to it, but then the function does even use setMat, it just creates another array, mat and prints that array out. That array isn't initialized, so it contains garbage values.

So when you print it, you can get anything!

(It also doesn't return an int, so the program is already ill-formed, why should it? Just make it return void.)

edited Jul 10, 2016 at 4:02

answered Jul 10, 2016 at 3:51

Rakete1111

49.6k19 gold badges133 silver badges173 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

SRT Over a year ago

can you write me a simple program in C++ to do this operation initialize the matrix of dimension 6 x 5 and make a function to display it thank you

Rakete1111 Over a year ago

@user5941106 I could do it, but then you wouldn't learn anything :) And I am absolutely sure you can do it, based on the code you posted. There are just some minor mistakes, and I'm sure you can handle them :) Here's a little tip to get you started for setOneMatrix. Good luck :)

phuclv Over a year ago

MSVC fills uninitialized memory with 0xCC to aid debugging, and -858993460 = 0xCCCCCCCC

Rakete1111 Over a year ago

@phuclv Interesting, I was only aware of 0xCDCDCDCD. Thanks :)

lampv · Accepted Answer · 2016-07-10 04:12:19Z

1

There are somethings wrong in your code.
1. The function "Display" must return a value. You just miss it. And instead of returning a value, just changing it into "void" function.
2. You redefine the matrix "setMat" in the function setOneMatrix(). It means the matrix "setMat" in the main() and the matrix "setMat" in the function setOneMatrix() are different about their address in the computer's memory. When you try to print out the matrix, you just print the matrix in the main(). And the value of its element are garbage.

answered Jul 10, 2016 at 4:12

lampv

374 bronze badges

Comments

SloCompTech · Accepted Answer · 2016-07-10 13:40:31Z

Here, i corrected your code

#include<iostream>
using namespace std;


int mat[6][5]; //Global variable, you can access this anywhere in your program

void init_mat();
void setOneMatrix();
void display(int mat[6][5]);

int main() {
    init_mat(); /* Init mat array */
    setOneMatrix();

    display(mat);//Pass array that is global as argument


}

void init_mat()
{
    //Give fields in array default value
    //Easyer to do field by field
    for (int i = 0; i < 6; i++)
        for (int j = 0; j < 5; j++)
            mat[i][j] = 0;
}
void setOneMatrix() {
    /*int setMat[6][5] = { { 1,2,3,4,5 },
    { 6,7,8,9,10 },
    { 11,12,13,14,15 },
    { 16,17,18,19,20 },
    { 21,22,23,24,25 },
    { 26,27,28,29,30 } };*/

    //Now you need to set value of fields field by field, 
    for (int i = 0; i < 6; i++)
        for (int j = 0; j < 5; j++)
            mat[i][j] = 1+i*5+j;
}

void display(int pMat[6][5]) {
    int i, j;
    for (i = 0; i<6; i++) {
        for (j = 0; j<5; j++) {
            cout << pMat[i][j] << "\t";
        }
        cout << endl;
    }
}

Collectives™ on Stack Overflow

C++ Multidimensional array program shows garbage value in output

3 Answers 3

4 Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

4 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related