0

配列とそのサイズを受け取るクラスがあります。クラスには、オーバーロードされたインデックス演算子「[]」があり、その内容をクラスメンバー関数に渡し、その関数を返します。また、配列サイズの外側のコンテンツにアクセスしているかどうかもチェックします。

     this_type operator[] (int index)  
    {
        assert (index > 0 && index<=len);
        at(index);
        return c_arr()[index];
    }

そのためのコピーコンストラクタを作成しました

   //constructor
    my_array(this_type const arr[], int size)
    {
    len = size;
    assert(arr != NULL);
    assert(size >= 0);

    this->arr = new this_type[size];
    for (int i = 0; i < size; i++)
        (*this).arr[i] = arr[i];
    }
    //copy constructor
    my_array( const my_array<this_type>  & arr)
    {
    this->arr = new this_type[sizeof(arr)];
    memcpy(this->arr, arr.arr, sizeof(this_type)*arr.len);
    }
    my_array(int size)
    {
    len = size;
    assert(size >= 0);

    this->arr = new this_type[size];
    }

しかし、メンバー関数「len」に呼び出されたときに、配列サイズの値を渡していないようです。何かご意見は？

    #include <iostream>
    #include <cassert>
    #include <assert.h>
    using namespace std;

    template <class this_type> 
    class my_array
    {
    private:
    this_type *arr;
    int len;

    int sum()
    {
        int sum;
        for (int i = 0; i < len; i++)
            sum += arr[i];
    }

    public:


    int size() const
    {
    return len;
    }

    this_type &at(int index)
    {
    assert( index >= 0 && index < len);
    return arr[index];
    }

    my_array(this_type const arr[], int size)
    {
    len = size;
    assert(arr != NULL);
    assert(size >= 0);

    this->arr = new this_type[size];
    for (int i = 0; i < size; i++)
        (*this).arr[i] = arr[i];
    }
    my_array( const my_array<this_type>  & arr)
    {
    this->arr = new this_type[sizeof(arr)];
    memcpy(this->arr, arr.arr, sizeof(this_type)*arr.len);
    }
    my_array(int size)
    {
    len = size;
    assert(size >= 0);

    this->arr = new this_type[size];
    }
    ~my_array()
    {
    delete[] arr;
    }

    this_type* c_arr()
    {
        return arr;
    }

     this_type & operator[] (int index)
        {
        assert (index > 0 && index<=len);
         at(index);
         return c_arr()[index];
    }

     this_type operator[] (int index)  
    {
        assert (index > 0 && index<=len);
        at(index);
        return c_arr()[index];
    }
    friend istream & operator>>(istream &lhs, my_array<this_type> &rhs);

    friend ostream & operator<<(ostream &lhs, my_array<this_type> &rhs);
    };
    template <class this_type> 
    ostream & operator<<(ostream &lhs, my_array<this_type>&rhs)
    {
    for ( int i = 0; i < rhs.size(); i++)
        lhs << rhs.arr[i] << endl;
    return lhs;
    }
    template <class this_type> 
    istream & operator>>(istream &lhs, my_array<this_type> &rhs)
    {
    for ( int i = 0; i < rhs.size(); i++)
        lhs >> rhs.arr[i];
    return lhs;
    }




    int main()
    {
    char arra[5]={'c','o','d','e','s'};

    my_array<char> arr(arra,5), arr1(5), arr2(arr);
       for(int t=0;t< 9;t++)
      {
       //use the operator that is attached to the class instance
        cout << arr2[t];

   }
    for(int t=0;t< 9;t++)
    {
    cout<<arr2.c_arr()[t];
    }
    return 0;
    }
4

1 に答える 1

1 
this->arr = new this_type[sizeof(arr)];

コピー コンストラクターのこの行は正しくありません。 あなたのクラス arrのオブジェクトです。コンパイル時の定数であり、配列に割り当てられた要素の数とは完全に無関係です。これらの要素は、クラスにも含まれていません。それらはフリー ストアにあり、クラスはそれらへのポインターを保持します。代わりにこれが必要です：my_arraysizeof(arr)

this->arr = new this_type[arr.len];

また、構築しているオブジェクトの len メンバーに割り当てたいと考えています。

this->len = arr.len;

学習目的でこのクラスを作成している場合を除き、std::vector.

于 2013-11-08T03:04:48.033 に答える