1
votes

Following code return this error:

main.cpp|80|error: passing ‘const matrix’ as ‘this’ argument of ‘T& matrix::at(size_t, size_t) [with T = int, size_t = long unsigned int]’ discards qualifiers [-fpermissive]’

Any thoughts why and how to repair it?

#include <vector>

template <typename T>
class matrix
{
    std::vector< std::vector<T> > data;
    size_t N, M;    
public:
        T& at(size_t x, size_t y);
        matrix<T> operator+(const matrix<T>& m2) const;
};

template<class T>
T& matrix<T>::at(size_t x, size_t y)
{ return data[x][y]; }

template<class T>
matrix<T> matrix<T>::operator+(const matrix<T>& m2) const
{    
    matrix<T> res; //initialization of internals not related to error
    for(unsigned int i=0; i<N; ++i)
        for(unsigned int j=0; j<M; ++j)
            res.at(i,j) = this->at(i, j) + m2.at(i, j);    
    return res;
}

int main()
{
    matrix<int> a, b; //initialization of internals not related to error
    a = a + b;
    return 0;
}

http://ideone.com/5OigTP

2
I think your *= and += need to return "matrix<T>&". Just as an aside. - Alex

2 Answers

7
votes

You need to overload at for const matrix<> as well:

template<class T>
const T& matrix<T>::at(size_t x, size_t y) const
{
    return data[x][y];
}
1
votes

have you tried adding a const overload for at()???

as in:

const T& at(size_t x, size_t y) const;