c ++错误:重新定义矩阵类

时间:2012-12-17 08:34:10

标签: c++

我一直在努力让这段代码得到编译。 我正在尝试实现一个简单的Matrix类,以便在另一个计算中使用。

截至目前,我仍然遇到以下两个错误: 来自编译器:

g++ -g -Wall -c main.cpp
In file included from main.cpp:19:0:
    Matrix.hh:2:7: error: redefinition of ‘class Matrix’
    Matrix.hh:2:14: error: previous definition of ‘class Matrix’

我没有看到任何明显的重新定义。请帮忙

由于

以下是支持文件:

Matrix.hh:

// Matrix class of variable size
class Matrix {

private:
    int n_rows;
    int n_cols;
    double *array;

public:
    // Constructors
    Matrix(); // default constructor
    Matrix(int rows, int cols); // two-argument constructor
//  Matrix(const Matrix &arr); // copy constructor


    // Destructor
    ~Matrix();

    // Mutators
//  void add(Matrix m2);
//  void subtract(Matrix m2);
    void setelem(int r, int c, double val);

    // Accessors
    int getrows();
    int getcols();
    double getelem(int r, int c);
    bool equals(Matrix m2);
    char display();
    int WriteToArray(double *CopyOfArray);

};

Matrix.cc:

#include "Matrix.hh"
#include <cassert>
#include <math.h>
#include <string.h>
#include <stdio.h>

// CONSTRUCTORS
// default constructor
Matrix::Matrix() {
    n_rows = 0;
    n_cols = 0;
    array = NULL;
}

// two-argument constructor
Matrix::Matrix(int rows, int cols) {
    n_rows = rows;
    n_cols = cols;
    array = new double[rows * cols];

    for (int i = 0; i < n_rows; i++) {
        for (int j = 0; j < n_cols; j++) {
            array[i * n_cols + j] = 0;
        }
    }
}

/* copy constructor*/
//Matrix::Matrix(const Matrix &arr)  {
//  n_rows = arr.n_rows; // find proper size
//  n_cols = arr.n_cols; // find proper size
//  array = new double[n_rows * n_cols]; // allocate a deep-copy
//  for (int i = 0; i < n_rows; i++) {
//      for (int j = 0; j < n_cols; j++) {
//          array[i * n_cols + j] = arr.array[i * n_cols + j];
//      }
//  }
//  arr=&array;
//}


// DESTRUCTOR
Matrix::~Matrix() {
    assert(array[1]!=0);
    int ii=0;
    printf("array values in ~\n");
    for(ii=0;ii<n_rows;ii++){
        printf("%e\n",array[ii]);
    }
    delete[] array; // free up memory
}

// MUTATORS
// adds a second matrix to this matrix object
void Matrix::add(Matrix m2) {
    assert (m2.n_rows == n_rows); // assert dimensions match
    assert (m2.n_cols == n_cols); // assert dimensions match
    for (int i = 0; i < n_rows; i++) {
        for (int j = 0; j < n_cols; j++) {
            array[i * n_cols + j] = array[i * n_cols + j] + m2.array[i *n_cols + j];
        }
    }
}

// subtracts a second matrix to this matrix object
void Matrix::subtract(Matrix m2) {
assert (m2.n_rows == n_rows); // assert dimensions match
assert (m2.n_cols == n_cols); // assert dimensions match
for (int i = 0; i < n_rows; i++) {
    for (int j = 0; j < n_cols; j++) {
        array[i * n_cols + j] = array[i * n_cols + j] - m2.array[i *n_cols + j];
        }
    }
}

// change an element in the matrix
void Matrix::setelem(int r, int c, double val) {

    array[r * n_cols + c] = val;
}

// ACCESSORS
// get number of rows
int Matrix::getrows() {
    return n_rows;
}

// get number of columns
int Matrix::getcols() {
    return n_cols;
}

// get value of element at specified row, col
double Matrix::getelem(int r, int c) {
    printf("getelem value: %e\n", array[r*n_cols+c]);
    return array[r * n_cols + c];
}

// test if two matrices are equal
bool Matrix::equals(Matrix m2) {
    // if dimensions not equal, matrices not equal
    if (m2.n_rows != n_rows || m2.n_cols != n_cols) { 
        return false;
    }

    // test equality element by element
    for (int i = 0; i < n_rows; i++) {
        for (int j = 0; j < n_cols; j++) {
            if (array[i * n_cols + j] != m2.array[i * n_cols + j]) {
                return false; // if one val not equal, matrices not equal
            }
        }
    }

    return true; // has checked all vals, matrices match
}

char Matrix::display() {
    // Print out the contents of this matrix m2:
    char string[100], temp[1];
    int n;
    for(int r = 0; r < n_rows; r++) {
      for(int c = 0; c < n_cols; c++) {
        printf("Element (%d, %d) =  %e, \n", r,c,array[r * n_cols + c]);

      }
    }
    printf("\n");
    return *string;
}

int Matrix::WriteToArray(double *CopyOfArray){
    int i=0;
    while(i<n_rows){
        *(CopyOfArray+i)=array[i*n_cols];
        i++;
    }
    return *CopyOfArray;
}

1 个答案:

答案 0 :(得分:2)

通常的做法是在C和C ++头文件中使用“#include警卫”以防止重新定义。

例如,在您的情况下:

#ifndef MATRIX_H
#define MATRIX_H

class Matrix {
   // [...]
};

#endif

此构造意味着如果#include "Matrix.hh"在源文件(及其include d个文件)中出现多次,则class Matrix只会被定义一次。

某些C和C ++实现提供了非标准指令#pragma once(如注释中所示)。插入头文件顶部的该指令将确保该文件仅包含一次。出于可移植性的原因,我更喜欢上面演示的构造。