[已解决]C++打印问题

糖逗

这段代码是先生成N*M矩阵然后打印生成的矩阵，但是为什么打印的结果不正确的问题？希望有明白的鱼友可以解答一下！

#include<iostream>

using namespace std;

class tes
{
        public:
                int* gen(int N, int M)//生成N*M的矩阵 
                {
                        int matrix[N*M] = {2};
                        for(int i = 0; i < N; i++)
                        {
                                for(int j = 0; j < M; j++)
                                {
                                        matrix[i*M+j] = i*2+j;
                                }
                        }
                        return matrix;        
                }
                void show(int*l)//打印显示生成的N*M矩阵 
                {
                        for(int i = 0; i < 2; i++)
                        {
                                for(int j = 0; j < 3; j++)
                                {
                                        cout << *l++ << " ";
                                }
                                cout << "\n" << endl;
                        }        
                }        
};
int main(void)
{
        tes t;
        int* p = t.gen(2, 3);
        t.show(p);
        return 0;
}

最佳答案

月排行榜 / 总排行榜

jackz007

2019-12-10 10:53:42

      你的问题出在，matrix 只是函数 int* gen(int N, int M) 的一个局部变量，其生存期只是在方法函数 gen() 的运行期间，一旦方法调用结束，局部变量 matrix 所占用的内存会被立即释放，供其它变量使用。就是说，在 gen() 以外，matrix 将不复存在。

#include<iostream>

using namespace std;

class tes
{
        public:
                int matrix[2][3] = {2}                             ;
                void gen(void)
                {
                        for(int i = 0; i < 2; i++)
                        {
                                for(int j = 0; j < 3 ; j++)
                                {
                                        matrix[i][j] = i * 2 + j   ;
                                }
                        }
                }

                void show(void)
                {
                        for(int i = 0; i < 2; i++)
                        {
                                for(int j = 0; j < 3; j++)
                                {
                                        cout << matrix[i][j] << " ";
                                }
                                cout << "\n" << endl               ;
                        }       
                }       
};

main(void)
{
        tes t     ;
        t.gen()   ;
        t.show()  ;
}

        编译、运行实况：

C:\Bin>g++ -o x x.cpp

C:\Bin>x
0 1 2

2 3 4


C:\Bin>

跳转到最佳答案楼层

糖逗

static应该加入这个关键词

jackz007

      你的问题出在，matrix 只是函数 int* gen(int N, int M) 的一个局部变量，其生存期只是在方法函数 gen() 的运行期间，一旦方法调用结束，局部变量 matrix 所占用的内存会被立即释放，供其它变量使用。就是说，在 gen() 以外，matrix 将不复存在。

#include<iostream>

using namespace std;

class tes
{
        public:
                int matrix[2][3] = {2}                             ;
                void gen(void)
                {
                        for(int i = 0; i < 2; i++)
                        {
                                for(int j = 0; j < 3 ; j++)
                                {
                                        matrix[i][j] = i * 2 + j   ;
                                }
                        }
                }

                void show(void)
                {
                        for(int i = 0; i < 2; i++)
                        {
                                for(int j = 0; j < 3; j++)
                                {
                                        cout << matrix[i][j] << " ";
                                }
                                cout << "\n" << endl               ;
                        }       
                }       
};

main(void)
{
        tes t     ;
        t.gen()   ;
        t.show()  ;
}

        编译、运行实况：

C:\Bin>g++ -o x x.cpp

C:\Bin>x
0 1 2

2 3 4


C:\Bin>

糖逗

jackz007 发表于 2019-12-10 10:53
你的问题出在，matrix 只是函数 int* gen(int N, int M) 的一个局部变量，其生存期只是在方法函数 ge ...

如果matrix的生存只在gen函数里面，那为什么下面这个show函数还可以读取matrix？

jackz007

糖逗 发表于 2019-12-10 11:00
如果matrix的生存只在gen函数里面，那为什么下面这个show函数还可以读取matrix？

      你用的是什么编译器？我使用 GNU 的 TDM GCC 编译器，你的代码倒是可以编译，但是，会有 array 是局部变量的警告，程序根本无法运行，一启动就会死掉。原因就是在 show() 中访问了 array。

糖逗

jackz007 发表于 2019-12-10 12:30
你用的是什么编译器？我使用 GNU 的 TDM GCC 编译器，你的代码倒是可以编译，但是，会有 array  ...

是的，原程序有warning问题，我用DEV能跑出来，但是结果不对。后来我在数组声明的前面加上static后程序运行成功了。你的代码是正确的！我一开始的初衷是：用户输入一个二维数组的尺寸，用class类里定义的gen函数动态生成，然后用show函数打印结果。

糖逗

jackz007 发表于 2019-12-10 12:30
你用的是什么编译器？我使用 GNU 的 TDM GCC 编译器，你的代码倒是可以编译，但是，会有 array  ...

程序这样改也可以。

#include<iostream>

using namespace std;

class test
{
public:
        test(int N, int M)
        {
        this -> M = M;
        this -> N = N;
        } 
        ~test(){};
        
    int* gen(void)//生成N*M的矩阵 
    {
        static int* matrix= new int [N*M] ;
        for(int i = 0; i < N; i++)
        {
            for(int j = 0; j < M; j++)
            {
                    matrix[i*M+j] = i*2+j;
            }
        }
        return matrix;        
    }
    void show(int*l)//打印显示生成的N*M矩阵 
    {
        for(int i = 0; i < N; i++)
        {
            for(int j = 0; j < M; j++)
            {
                    cout << *l++ << " ";
            }
            cout << "\n" << endl;
        }        
    } 
private:
        int N, M;       
};




int main(void)
{
        test t(2, 3);
        int* p = t.gen();
        t.show(p);
        return 0;
}

jackz007

糖逗 发表于 2019-12-10 15:54
程序这样改也可以。

          静态变量不在堆栈分配空间，等同于全局变量，这样修改当然也是可行的。