Question

我使用不同的函数在C ++中编程。我对本地和全局变量的声明和使用有问题。首先必须在main中使用许多变量，并且还需要在函数中使用。

所以，我宣布了一些全局变量。我还想在main()之外分配一些内存。

但现在代码似乎不起作用。请帮我解决这个问题。

我有：

int *mb = new int [c];  //declare variables before you use them
matrix *M = new matrix  [n+1+m];

当我使用上面两个变量作为全局变量时，我的代码不起作用。

以下是我的代码的一部分：

int p=4, n=5, m=5;
int num;
struct matrix{
public:
int row;
int col;
int val;
};

int P(int k){
return k-1;
}

void M_to_C(){

int *mb,*mc,*mi;
double *mv;
int i, cur_col;
matrix*M;
cur_col=-1;
for(i=0; i<= num; i++) {
    cur_col=M[i].col;
    mb[cur_col]=i;
    mc[cur_col]=1;
    mi[i]=M[i].row;
    mv[i]=M[i].val;
}}

int main (int argc, char **argv){
int status = 0;

int project = 4; 
int employee = 5;
int time = 5; 

int empl [] = {2, 2, 2, 3};
int meet [] = {2, 4, 3, 3};

int NUMCOLS=n+m;

double *lb = new double [NUMCOLS];
double *ub = new double [NUMCOLS];
double *x  = new double [NUMCOLS];
int    *mb= new int [NUMCOLS];
int    *mc= new int [NUMCOLS];
char   *ctype = new char [NUMCOLS];

int **F = new int*[employee]; 
for(int a = 1; a <= employee; a++){     
    F[a] = new int [time];
    for(int b = 1; b <= time; b++){
        F[a][b]=rand()%2;
    }}

int NUMROWS=0;  

for(int i=1; i<=n; i++){        
    for(int j=1; j<=m; j++){ 
        for(int l=1; l<=p; l++) {
            if(F[i][j] ==0){            
                NUMROWS++;
                                    num++;
            }
        } 
    }
}

int    *mi= new int [2*NUMROWS+m];
double *mv= new double [2*NUMROWS+m];
matrix_entry *M = new matrix_entry  [num+1+m];

num=-1;
int row=-1;
for(int i=1; i <= n; i++) {
    for(int j=1; j <= m; j++) {
        for(int k=1; k <= project; k++) {
            if(F[i][j] ==0) {
                row++;
                num++;
                M[num].col=P(k); 
                M[num].row=row;
                M[num].val=-1;
            }}}}

M_to_C();

Answer 1

您可以将它们定义为全局变量并在main填充它们：

int *mb;
matrix *M;

int main () {
    mb = new int [c]; 
    M = new matrix  [n+1+m];
    :
}

但是我应该警告你，全局变量（以及mb，lb和ub之类的短变量名称）通常被认为是个坏主意。

根据需要，最好将它们放在一个类中，或者将它们作为引用传递来限制它们的可访问性。

Answer 2

在M_to_C()函数内部，您对单位化指针M执行了一些操作，所以基本上这是未定义的行为：

matrix*M;
cur_col=-1;
for(i=0; i<= num; i++) {
    cur_col=M[i].col; // access local uninitialized pointer M

为了修复它，您应该在使用之前初始化M属性。在您的情况下，局部变量M隐藏全局变量。我建议将M传递给M_to_C()函数，以便它可以在现有对象上运行：

void M_to_C(matrix * M)

C ++全局变量和函数

2 个答案: