0

このコードをコンパイルするのに苦労しています。別の計算で使用する単純な Matrix クラスを実装しようとしています。

今のところ、次の 2 つのエラーで立ち往生しています: コンパイラから:

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’

明らかな再定義は見当たりません。助けてください

ありがとう

サポート ファイルは次のとおりです。

マトリックス.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);

};

マトリックス.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;
}
4

1 に答える 1

2

#include再定義を防ぐために、C および C++ ヘッダー ファイルで「ガード」を使用するのが一般的です。

たとえば、あなたの場合：

#ifndef MATRIX_H
#define MATRIX_H

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

#endif

この構文は、ソース ファイル (およびそのd ファイル) に#include "Matrix.hh"複数回出現する場合、一度しか定義されないことを意味します。includeclass Matrix

一部の C および C++ 実装は、非標準ディレクティブを提供します#pragma once(コメントで提案されているように)。このディレクティブをヘッダー ファイルの先頭に挿入すると、ファイルが 1 回だけインクルードされるようになります。移植性の理由から、上で示した構成を好みます。

于 2012-12-17T08:44:59.943 に答える