论坛 › C\C++交流 › C++打印问题

糖逗 发表于 2019-12-9 10:24:23

C++打印问题

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

#include<iostream>

using namespace std;

class tes
{
        public:
                int* gen(int N, int M)//生成N*M的矩阵
                {
                        int matrix = {2};
                        for(int i = 0; i < N; i++)
                        {
                                for(int j = 0; j < M; j++)
                                {
                                        matrix = 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;
}

糖逗 发表于 2019-12-9 12:13:53

static应该加入这个关键词

jackz007 发表于 2019-12-10 10:53:42

本帖最后由 jackz007 于 2019-12-10 10:55 编辑

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

using namespace std;

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

                void show(void)
                {
                        for(int i = 0; i < 2; i++)
                        {
                              for(int j = 0; j < 3; j++)
                              {
                                        cout << matrix << " ";
                              }
                              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>

糖逗 发表于 2019-12-10 11:00:15

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

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

jackz007 发表于 2019-12-10 12:30:55

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

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

糖逗 发表于 2019-12-10 15:44:33

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

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

糖逗 发表于 2019-12-10 15:54:51

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 ;
      for(int i = 0; i < N; i++)
      {
            for(int j = 0; j < M; j++)
            {
                  matrix = 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 16:03:22

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

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

查看完整版本: C++打印问题