2

我试图让这段代码在 GNU C++ 编译器 (g++) 中编译，但它似乎不起作用。我用过 Vis Studio 和 Code::Blocks，它工作得很好。我知道编译器在某些方面有所不同，并且想知道是否有人可以帮助我找到我的错误。

#include <iostream>
using namespace std;

template <class T>
class Array
{
    private:
        T *m_array;
        int m_size;
    public:
        Array();
        Array(Array& other);
        Array(int size);
        ~Array();
        void setValue(int index, T val);
        T getValue(int index);
        int getSize();
        Array &operator=(Array &other);
        Array &operator+(Array &other);
        Array &operator+(T val);
        inline friend ostream &operator<<(ostream &other, Array<T> arr)
        {
            for (int i = 0; i < arr.getSize(); i++)
            {
                other << arr.getValue(i) << " ";
            }
        }
};

template<class T>
Array<T>::Array()
{
    m_array =  NULL;
    m_size = 0;
}
template<class T>
Array<T>::Array(int size)
{
    m_size = size;
    m_array = new T[size];
}
template<class T>
Array<T>::Array(Array& other)
{
    *this = other;
}
template<class T>
Array<T>::~Array()
{
    delete[] m_array;
}
template<class T>
void Array<T>::setValue( int index, T val )
{
    m_array[index] = val;
}
template<class T>
T Array<T>::getValue( int index )
{
    return m_array[index];
}
template<class T>
Array<T> &Array<T>::operator=( Array& other )
{
    if (m_array != NULL)
        delete[] m_array;

    m_size = other.getSize();

    m_array = new T[m_size];

    for (int i = 0; i < other.getSize(); i++)
    {
        m_array[i] = other.getValue(i);
    }

    return *this;
}
template<class T>
Array<T> &Array<T>::operator+( Array &other )
{
    for (int i = 0; i < m_size; i++)
    {
        m_array[i] += other.getValue(i);
    }

    return *this;
}
template<class T>
Array<T> &Array<T>::operator+( T val )
{
    for (int i = 0; i < m_size; i++)
    {
        m_array[i] += val;
    }

    return *this;
}
template<class T>
int Array<T>::getSize()
{
    return m_size;
}
4

3 回答 3

1

1) 你应该真正了解 const-correctness

2) 这段代码看起来很可疑

template<class T>
Array<T> &Array<T>::operator+( Array &other )
{
    for (int i = 0; i < m_size; i++)
    {
        m_array[i] += other.getValue(i);
    }

    return *this;
}

如果otherArray 的元素较少怎么办？您将有未定义的行为（可能包括分段错误）

3）你为什么使用 m_array[i] += other.getValue(i);？因为m_array是私人的？请记住，访问是在类级别而不是对象级别定义的，因此m_array[i] = other.m_arry[i]也可以。

4) 我建议你应该阅读一本好的 C++ 书籍

5) 只有在发布使用数组类的代码时才能确定段错误的确切原因。

于 2012-11-13T18:02:40.000 回答
1

我看到两个问题：

  1. 使用const &，否则您的数组将被复制：

    内联好友 ostream &operator<<(ostream &other, const Array &arr)

  2. 不要在没有初始化指针的情况下在构造函数中使用赋值运算符：

    Array::Array(Array& other) { *this = other; }

这至少应该是：

Array<T>::Array(const Array& other)
    : m_array(0)
{
    *this = other;
}

我猜这就是它崩溃的地方：

template<class T>
Array<T> &Array<T>::operator=( Array& other )
{
    if (m_array != NULL)
        delete[] m_array; // In copy constructor, deletes uninitialized pointer!

    m_size = other.getSize();

    m_array = new T[m_size];

    for (int i = 0; i < other.getSize(); i++)
    {
        m_array[i] = other.getValue(i);
    }

    return *this;
}
于 2012-11-13T18:06:52.167 回答
1

打开你的警告：

g++ -std=c++0x -pedantic -Wall -Werror -g    x.cc   -o x
cc1plus: warnings being treated as errors
x.cc: In function ‘std::ostream& operator<<(std::ostream&, Array<T>)’:
x.cc:27: error: no return statement in function returning non-void

不正确的功能是：

inline friend ostream &operator<<(ostream &other, Array<T> arr)
于 2012-11-13T18:05:14.740 回答