Operator for array assignment.

I know how to overload the array operator to GET a value:

a = myClass[1];

But how do i would i set the value like this:

myClass[1] = a;

How does it make a difference ? You just have to make sure that myClass::operator [] returns a reference (which it should anyway).

Actually, there should be two versions of myClass::operator[]. One returns a reference, the other returns a const reference.

True. For const-correctness.

Edit: The one which returns a const reference should also be a const function.

Code:

---

T_& myClass::operator [] (unsigned int i) { return mystuff[i]; }
const T_& myClass::operator [] (unsigned int i) const { return mystuff[i]; }

---

Ok heres my new problem. I cant return a reference -_-.

Heres the code:

Code:

---

    T operator[](int pIndex)
    {
        if(pIndex >= 0 && pIndex < count)
            return items[pIndex];
        return T();
    }

---

If i change it to:

Code:

---


    T& operator[](int pIndex)
    {
        if(pIndex >= 0 && pIndex < count)
            return items[pIndex];
        return T();  //<<< this doesnt work
    }

---

error: invalid initialization of non-const reference of type 'int&' from a temporary of type 'int'

Where T is int

Explain "doesn't work". It's Chinese to me.

You have a choice: you can simply return the index and make it the caller's responsibility to provide a valid index, or you can check and throw an exception (e.g., std::out_of_range) if the index is invalid. The former is what operator[] for the standard containers do, the latter is what the at() member function for the standard containers do.

You certainly cannot return a reference to a local temporary.

Quote:

---

Originally Posted by 39ster

Code:

---


    T& operator[](int pIndex)
    {
        if(pIndex >= 0 && pIndex < count)
            return items[pIndex];
        return T();  //<<< this doesnt work
    }

---

error: invalid initialization of non-const reference of type 'int&' from a temporary of type 'int'

Where T is int

---

Right, because that way the compiler prevents you from doing this:

Code:

---

Myclass[-1]=a;

---

If you want to allow the above, then returning a temporary is not enough. You have to actually put the newly constructed object in your array.

Or you can decide on some other behavior for an invalid index as suggested by above.

Thanks. Also i just figured out what putting const at the end of const-safe functions does.

Code:

---

    const T& operator[](int pIndex) const
    {
        return items[pIndex];
    }

    T& operator[](int pIndex)
    {
        return items[pIndex];
    }

---

Quote:

---

Originally Posted by 39ster

Thanks. Also i just figured out what putting const at the end of const-safe functions does.

---

Ah yes, that const applies to the implicit 'this' parameter to non-static class functions. That's simply how you make 'this' a pointer to const.