This is a followup to:
Please review this pointer class.
template<typename T>
class Ptr {
public:
Ptr(T* t, int s = 1) {
sz = s;
p = new T[sz];
for (int i = 0; i < sz; ++i) {
p[i] = t[i];
}
}
Ptr(const Ptr& t) {
sz = t.sz;
p = new T[sz];
T* x = t.get();
for (int i = 0; i < sz; ++i) {
p[i] = x[i];
}
}
Ptr& operator=(const Ptr& t) {
Ptr copy{t};
std::swap(copy.sz, sz);
std::swap(copy.p, p);
return *this;
}
Ptr(Ptr &&t) :p{t.p}, sz{t.sz} {
t.p = nullptr;
t.sz = 0;
}
Ptr& operator=(Ptr &&t) {
std::swap(t.p,p);
std::swap(t.sz,sz);
return *this;
}
T& operator*() {
check_range();
return *p;
}
T& operator[](int i) {
check_range(i);
return p[i];
}
T* get() const {
return p;
}
void operator+=(int i) {
check_range(index+i);
index += i;
p+= i;
}
void operator-=(int i) {
check_range(index-i);
index -= i;
p -= i;
}
Ptr operator+(int i) {
Ptr old{*this};
old+=i;
return old;
}
Ptr operator-(int i) {
Ptr old{*this};
old-= i;
return old;
}
Ptr& operator++() {
operator+=(1);
return *this;
}
Ptr operator++(int) {
Ptr<T> old{p};
operator++();
return old;
}
Ptr& operator--() {
operator-=(1);
return *this;
}
Ptr operator--(int) {
Ptr<T> old{p};
operator--();
return old;
}
~Ptr() {
delete[] p;
}
private:
T* p;
int sz;
int index = 0;
void check_range(int i) {
if (i < 0 || i > sz-1) {
throw std::out_of_range("out of range");
}
}
void check_range() {
if (p == nullptr) {
throw std::out_of_range("null pointer");
}
}
};