鱼C论坛

 找回密码
 立即注册
查看: 1888|回复: 7

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

[复制链接]
发表于 2019-12-9 10:24:23 | 显示全部楼层 |阅读模式

马上注册,结交更多好友,享用更多功能^_^

您需要 登录 才可以下载或查看,没有账号?立即注册

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

  1. #include<iostream>

  2. using namespace std;

  3. class tes
  4. {
  5.         public:
  6.                 int* gen(int N, int M)//生成N*M的矩阵
  7.                 {
  8.                         int matrix[N*M] = {2};
  9.                         for(int i = 0; i < N; i++)
  10.                         {
  11.                                 for(int j = 0; j < M; j++)
  12.                                 {
  13.                                         matrix[i*M+j] = i*2+j;
  14.                                 }
  15.                         }
  16.                         return matrix;       
  17.                 }
  18.                 void show(int*l)//打印显示生成的N*M矩阵
  19.                 {
  20.                         for(int i = 0; i < 2; i++)
  21.                         {
  22.                                 for(int j = 0; j < 3; j++)
  23.                                 {
  24.                                         cout << *l++ << " ";
  25.                                 }
  26.                                 cout << "\n" << endl;
  27.                         }       
  28.                 }       
  29. };
  30. int main(void)
  31. {
  32.         tes t;
  33.         int* p = t.gen(2, 3);
  34.         t.show(p);
  35.         return 0;
  36. }
复制代码
最佳答案
2019-12-10 10:53:42
本帖最后由 jackz007 于 2019-12-10 10:55 编辑

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

  2. using namespace std;

  3. class tes
  4. {
  5.         public:
  6.                 int matrix[2][3] = {2}                             ;
  7.                 void gen(void)
  8.                 {
  9.                         for(int i = 0; i < 2; i++)
  10.                         {
  11.                                 for(int j = 0; j < 3 ; j++)
  12.                                 {
  13.                                         matrix[i][j] = i * 2 + j   ;
  14.                                 }
  15.                         }
  16.                 }

  17.                 void show(void)
  18.                 {
  19.                         for(int i = 0; i < 2; i++)
  20.                         {
  21.                                 for(int j = 0; j < 3; j++)
  22.                                 {
  23.                                         cout << matrix[i][j] << " ";
  24.                                 }
  25.                                 cout << "\n" << endl               ;
  26.                         }      
  27.                 }      
  28. };

  29. main(void)
  30. {
  31.         tes t     ;
  32.         t.gen()   ;
  33.         t.show()  ;
  34. }
复制代码

        编译、运行实况:
  1. C:\Bin>g++ -o x x.cpp

  2. C:\Bin>x
  3. 0 1 2

  4. 2 3 4


  5. C:\Bin>
复制代码
小甲鱼最新课程 -> https://ilovefishc.com
回复

使用道具 举报

 楼主| 发表于 2019-12-9 12:13:53 | 显示全部楼层
static应该加入这个关键词
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2019-12-10 10:53:42 | 显示全部楼层    本楼为最佳答案   
本帖最后由 jackz007 于 2019-12-10 10:55 编辑

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

  2. using namespace std;

  3. class tes
  4. {
  5.         public:
  6.                 int matrix[2][3] = {2}                             ;
  7.                 void gen(void)
  8.                 {
  9.                         for(int i = 0; i < 2; i++)
  10.                         {
  11.                                 for(int j = 0; j < 3 ; j++)
  12.                                 {
  13.                                         matrix[i][j] = i * 2 + j   ;
  14.                                 }
  15.                         }
  16.                 }

  17.                 void show(void)
  18.                 {
  19.                         for(int i = 0; i < 2; i++)
  20.                         {
  21.                                 for(int j = 0; j < 3; j++)
  22.                                 {
  23.                                         cout << matrix[i][j] << " ";
  24.                                 }
  25.                                 cout << "\n" << endl               ;
  26.                         }      
  27.                 }      
  28. };

  29. main(void)
  30. {
  31.         tes t     ;
  32.         t.gen()   ;
  33.         t.show()  ;
  34. }
复制代码

        编译、运行实况:
  1. C:\Bin>g++ -o x x.cpp

  2. C:\Bin>x
  3. 0 1 2

  4. 2 3 4


  5. C:\Bin>
复制代码
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

 楼主| 发表于 2019-12-10 11:00:15 | 显示全部楼层
jackz007 发表于 2019-12-10 10:53
你的问题出在,matrix 只是函数 int* gen(int N, int M) 的一个局部变量,其生存期只是在方法函数 ge ...

如果matrix的生存只在gen函数里面,那为什么下面这个show函数还可以读取matrix?
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2019-12-10 12:30:55 | 显示全部楼层
糖逗 发表于 2019-12-10 11:00
如果matrix的生存只在gen函数里面,那为什么下面这个show函数还可以读取matrix?


      你用的是什么编译器?我使用 GNU 的 TDM GCC 编译器,你的代码倒是可以编译,但是,会有 array 是局部变量的警告,程序根本无法运行,一启动就会死掉。原因就是在 show() 中访问了 array。
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

 楼主| 发表于 2019-12-10 15:44:33 | 显示全部楼层
jackz007 发表于 2019-12-10 12:30
你用的是什么编译器?我使用 GNU 的 TDM GCC 编译器,你的代码倒是可以编译,但是,会有 array  ...

是的,原程序有warning问题,我用DEV能跑出来,但是结果不对。后来我在数组声明的前面加上static后程序运行成功了。你的代码是正确的!我一开始的初衷是:用户输入一个二维数组的尺寸,用class类里定义的gen函数动态生成,然后用show函数打印结果。
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

 楼主| 发表于 2019-12-10 15:54:51 | 显示全部楼层
jackz007 发表于 2019-12-10 12:30
你用的是什么编译器?我使用 GNU 的 TDM GCC 编译器,你的代码倒是可以编译,但是,会有 array  ...

程序这样改也可以。
  1. #include<iostream>

  2. using namespace std;

  3. class test
  4. {
  5. public:
  6.         test(int N, int M)
  7.         {
  8.         this -> M = M;
  9.         this -> N = N;
  10.         }
  11.         ~test(){};
  12.        
  13.     int* gen(void)//生成N*M的矩阵
  14.     {
  15.         static int* matrix= new int [N*M] ;
  16.         for(int i = 0; i < N; i++)
  17.         {
  18.             for(int j = 0; j < M; j++)
  19.             {
  20.                     matrix[i*M+j] = i*2+j;
  21.             }
  22.         }
  23.         return matrix;        
  24.     }
  25.     void show(int*l)//打印显示生成的N*M矩阵
  26.     {
  27.         for(int i = 0; i < N; i++)
  28.         {
  29.             for(int j = 0; j < M; j++)
  30.             {
  31.                     cout << *l++ << " ";
  32.             }
  33.             cout << "\n" << endl;
  34.         }        
  35.     }
  36. private:
  37.         int N, M;      
  38. };




  39. int main(void)
  40. {
  41.         test t(2, 3);
  42.         int* p = t.gen();
  43.         t.show(p);
  44.         return 0;
  45. }

复制代码
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2019-12-10 16:03:22 | 显示全部楼层
糖逗 发表于 2019-12-10 15:54
程序这样改也可以。

          静态变量不在堆栈分配空间,等同于全局变量,这样修改当然也是可行的。
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 1 反对 0

使用道具 举报

您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

小黑屋|手机版|Archiver|鱼C工作室 ( 粤ICP备18085999号-1 | 粤公网安备 44051102000585号)

GMT+8, 2025-7-6 15:55

Powered by Discuz! X3.4

© 2001-2023 Discuz! Team.

快速回复 返回顶部 返回列表